Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751628Ab3FDIU2 (ORCPT ); Tue, 4 Jun 2013 04:20:28 -0400 Received: from cn.fujitsu.com ([222.73.24.84]:1118 "EHLO song.cn.fujitsu.com" rhost-flags-OK-FAIL-OK-OK) by vger.kernel.org with ESMTP id S1751170Ab3FDIUX convert rfc822-to-8bit (ORCPT ); Tue, 4 Jun 2013 04:20:23 -0400 X-IronPort-AV: E=Sophos;i="4.87,798,1363104000"; d="scan'208";a="7455606" Subject: Re: [PATCH 2/2] xen: remove bm_rld_set of xen_processor_flags From: li guang To: Jan Beulich Cc: Stefano Stabellini , Len Brown , Konrad Rzeszutek Wilk , "Rafael J. Wysocki" , Jeremy Fitzhardinge , Liu Jinsong , "Paul E. McKenney" , virtualization@lists.linux-foundation.org, xen-devel@lists.xensource.com, linux-acpi@vger.kernel.org, linux-kernel@vger.kernel.org In-Reply-To: <51ADBDC402000078000DAF36@nat28.tlf.novell.com> References: <1370333128-11412-1-git-send-email-lig.fnst@cn.fujitsu.com> <1370333128-11412-2-git-send-email-lig.fnst@cn.fujitsu.com> <51ADBDC402000078000DAF36@nat28.tlf.novell.com> Date: Tue, 04 Jun 2013 16:19:27 +0800 Message-ID: <1370333967.17798.60.camel@liguang.fnst.cn.fujitsu.com> Mime-Version: 1.0 X-Mailer: Evolution 2.30.3 X-MIMETrack: Itemize by SMTP Server on mailserver/fnst(Release 8.5.3|September 15, 2011) at 2013/06/04 16:18:41, Serialize by Router on mailserver/fnst(Release 8.5.3|September 15, 2011) at 2013/06/04 16:18:42, Serialize complete at 2013/06/04 16:18:42 Content-Transfer-Encoding: 8BIT Content-Type: text/plain; charset="UTF-8" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1535 Lines: 43 在 2013-06-04二的 09:13 +0100，Jan Beulich写道： > >>> On 04.06.13 at 10:05, liguang wrote: > > bm_rld_set seems obsolete now > > > > Signed-off-by: liguang > > --- > > 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@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/