> -----Original Message----- > From: H. Peter Anvin [mailto:hpa@xxxxxxxxx] > Sent: Saturday, December 07, 2013 9:07 AM > To: Ren, Qiaowei; Liu, Jinsong; Paolo Bonzini > Cc: kvm@xxxxxxxxxxxxxxx; x86@xxxxxxxxxx; Xudong Hao; > qemu-devel@xxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx; Ingo Molnar; Thomas > Gleixner > Subject: Re: [Qemu-devel] [PATCH v2 3/3] X86, mpx: Intel MPX xstate feature > definition > > On 12/06/2013 04:23 PM, Ren, Qiaowei wrote: > >>> > >>> We need to either disable these features in lazy mode, or we need to > >>> force eager mode if these features are to be supported. The problem > >>> with the latter is that it means forcing eager mode regardless of if > >>> anything actually *uses* these features. > >>> > >>> A third option would be to require applications to use a prctl() or > >>> similar to enable eager-save features. > >>> > >> > >> The third option seems better -- how does native mpx patches work, > >> force eager? > >> > > It should be the second option, as you can see xsave.c which we remove > > from this patch. :) > > > > Ah yes... I missed the fact that that chunk had been dropped from this patch. > It really shouldn't be. > > I'll substitute the previous version of the patch. > Jinsong think that both kvm and host depend on these feature definition header file, so we firstly submit these files depended on. Thanks, Qiaowei ��.n��������+%������w��{.n�����o�^n�r������&��z�ޗ�zf���h���~����������_��+v���)ߣ�
