Re: [PATCH 2/2] xen: remove bm_rld_set of xen_processor_flags

From: Jan Beulich
Date: Tue Jun 04 2013 - 04:13:46 EST


>>> On 04.06.13 at 10:05, liguang <lig.fnst@xxxxxxxxxxxxxx> wrote:
> bm_rld_set seems obsolete now
>
> Signed-off-by: liguang <lig.fnst@xxxxxxxxxxxxxx>
> ---
> 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.

Jan

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/