From mboxrd@z Thu Jan 1 00:00:00 1970 From: "H. Peter Anvin" Subject: Re: [Xen-devel] Is: axe read_tscp pvops call. Was: Re: [RFC] ACPI S3 and Xen (suprisingly small\!). Date: Thu, 18 Oct 2012 08:28:48 -0700 Message-ID: <50802030.8070107@zytor.com> References: <1350481786-4969-1-git-send-email-konrad.wilk@oracle.com> <507ED6C0.4020503@zytor.com> <20121017161036.GA10691@phenom.dumpdata.com> <507EE1C3.7070300@zytor.com> <20121017165452.GA22740@phenom.dumpdata.com> <507EEC53.1010309@zytor.com> <84b3cbf9-7c84-4d7e-a2d7-46b0d1cc5975@default> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <84b3cbf9-7c84-4d7e-a2d7-46b0d1cc5975@default> Sender: linux-acpi-owner@vger.kernel.org To: Dan Magenheimer Cc: Konrad Wilk , linux-acpi@vger.kernel.org, x86@kernel.org, xen-devel@lists.xensource.com, linux-kernel@vger.kernel.org, lenb@kernel.org List-Id: xen-devel@lists.xenproject.org On 10/18/2012 08:22 AM, Dan Magenheimer wrote: > > It's a bit more complicated than that. The problem is that if > any patch is ever submitted to the kernel that uses the rdtscp > instruction *in kernel space* in some clever way, the resultant > kernel may not behave as expected (depending on how the instruction > is used) on a 32-bit[1] PV kernel running on Xen, up to and including > the possibility of data corruption. > > I don't know how one would implement it, but it's like a > BUILD_BUG_ON is needed if any kernel developer uses rdtscp > (one that never gets invoked by vdso code), that prints: > > "WARNING: Please do not use this instruction in the kernel > without notifying the Xen maintainer as there is a possibility > it may behave unpredictably in some Xen environments. > See Documentation/.../xen_pv_limitations for detail." > > The other virtualization-unsafe instructions may have similar > problems. > Good frakking God. This is the sort of things that makes me think that Xen PV should just be thrown out of the kernel once and for all. Do you notice that the document you just claimed doesn't even exist at this point, never mind being somehow enforced? In other word, there is ABSOLUTELY NO WAY a mainline kernel developer can have any idea what amount of violence Xen does to the architecture that it is parasiting on. -hpa -- H. Peter Anvin, Intel Open Source Technology Center I work for Intel. I don't speak on their behalf.