[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <1370333967.17798.60.camel@liguang.fnst.cn.fujitsu.com>
Date: Tue, 04 Jun 2013 16:19:27 +0800
From: li guang <lig.fnst@...fujitsu.com>
To: Jan Beulich <JBeulich@...e.com>
Cc: Stefano Stabellini <stefano.stabellini@...citrix.com>,
Len Brown <lenb@...nel.org>,
Konrad Rzeszutek Wilk <konrad.wilk@...cle.com>,
"Rafael J. Wysocki" <rjw@...k.pl>,
Jeremy Fitzhardinge <jeremy@...p.org>,
Liu Jinsong <jinsong.liu@...el.com>,
"Paul E. McKenney" <paulmck@...ux.vnet.ibm.com>,
virtualization@...ts.linux-foundation.org,
xen-devel@...ts.xensource.com, linux-acpi@...r.kernel.org,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH 2/2] xen: remove bm_rld_set of xen_processor_flags
在 2013-06-04二的 09:13 +0100,Jan Beulich写道:
> >>> On 04.06.13 at 10:05, liguang <lig.fnst@...fujitsu.com> wrote:
> > bm_rld_set seems obsolete now
> >
> > Signed-off-by: liguang <lig.fnst@...fujitsu.com>
> > ---
> > include/xen/interface/platform.h | 1 -
> > 1 files changed, 0 insertions(+), 1 deletions(-)
> >
> > diff --git a/include/xen/interface/platform.h
> > b/include/xen/interface/platform.h
> > index c57d5f6..7ffff33 100644
> > --- a/include/xen/interface/platform.h
> > +++ b/include/xen/interface/platform.h
> > @@ -240,7 +240,6 @@ struct xen_processor_flags {
> > uint32_t bm_check:1;
> > uint32_t has_cst:1;
> > uint32_t power_setup_done:1;
> > - uint32_t bm_rld_set:1;
> > };
> >
> > struct xen_processor_power {
>
> Any such patch would need to be submitted against the master copy
> of the header (in the Xen repo), and by recognizing that you'd also
> notice that this is part of a public ABI, and hence can't be removed,
> but at best can be documented as obsolete. Of course you'd first
> need to check whether the hypervisor makes any use of that bit
> when passed down from Dom0.
>
Right, this patch mostly likes a reminder,
hoping someone will try it and verify it's fail or OK,
since I can't test this for now.
Sorry and Thanks!
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists