From mboxrd@z Thu Jan 1 00:00:00 1970 From: Borislav Petkov Subject: Re: [PATCH 3/3] X86, mpx: Intel MPX xstate feature definition Date: Fri, 6 Dec 2013 19:55:34 +0100 Message-ID: <20131206185534.GA7528@pd.tnic> References: <1386355976-11732-1-git-send-email-qiaowei.ren@intel.com> <1386355976-11732-3-git-send-email-qiaowei.ren@intel.com> <20131206134631.GD6694@pd.tnic> <52A2080A.9010208@zytor.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Cc: Qiaowei Ren , Paolo Bonzini , Ingo Molnar , Thomas Gleixner , x86@kernel.org, linux-kernel@vger.kernel.org, qemu-devel@nongnu.org, kvm@vger.kernel.org, Xudong Hao , Liu Jinsong To: "H. Peter Anvin" Return-path: Content-Disposition: inline In-Reply-To: <52A2080A.9010208@zytor.com> Sender: linux-kernel-owner@vger.kernel.org List-Id: kvm.vger.kernel.org On Fri, Dec 06, 2013 at 09:23:22AM -0800, H. Peter Anvin wrote: > On 12/06/2013 05:46 AM, Borislav Petkov wrote: > > I'm guessing this and the struct lwp_struct above is being added so that > > you can have the LWP XSAVE area size? If so, you don't need it: LWP > > XSAVE area is 128 bytes at offset 832 according to my manuals so I'd > > guess having a u8 lwp_area[128] should be fine. > > > Sure, but any reason to *not* document the internal structure? Only that you might start getting remove-this-unused-struct patches. :-) -- Regards/Gruss, Boris. Sent from a fat crate under my desk. Formatting is fine. --