Re: [PATCH 6/7] x86/traps: Fix up invalid PASID

2020-04-27 Thread Raj, Ashok
Hi Thomas, On Tue, Apr 28, 2020 at 02:54:59AM +0200, Thomas Gleixner wrote: > Ashok, > > "Raj, Ashok" writes: > > On Sun, Apr 26, 2020 at 05:25:06PM +0200, Thomas Gleixner wrote: > >> Just for the record I also suggested to have a proper errorcode in the > >> #GP for ENQCMD and I surely did not

Re: [PATCH 6/7] x86/traps: Fix up invalid PASID

2020-04-27 Thread Thomas Gleixner
Ashok, "Raj, Ashok" writes: > On Sun, Apr 26, 2020 at 05:25:06PM +0200, Thomas Gleixner wrote: >> Just for the record I also suggested to have a proper errorcode in the >> #GP for ENQCMD and I surely did not suggest to avoid decoding the user >> instructions. > > We certainly discussed the

RE: [PATCH 6/7] x86/traps: Fix up invalid PASID

2020-04-27 Thread Thomas Gleixner
"Luck, Tony" writes: >> Just for the record I also suggested to have a proper errorcode in the >> #GP for ENQCMD and I surely did not suggest to avoid decoding the user >> instructions. > > Is the heuristic to avoid decoding the user instructions OK (you are just > pointing > out that you should

Re: [PATCH 6/7] x86/traps: Fix up invalid PASID

2020-04-27 Thread Thomas Gleixner
Fenghua Yu writes: > On Sun, Apr 26, 2020 at 05:25:06PM +0200, Thomas Gleixner wrote: >> > @@ -499,6 +510,12 @@ dotraplinkage void do_general_protection(struct >> > pt_regs *regs, long error_code) >> >int ret; >> > >> >RCU_LOCKDEP_WARN(!rcu_is_watching(), "entry code didn't wake RCU");

RE: [PATCH 6/7] x86/traps: Fix up invalid PASID

2020-04-27 Thread Luck, Tony
> Just for the record I also suggested to have a proper errorcode in the > #GP for ENQCMD and I surely did not suggest to avoid decoding the user > instructions. Thomas, Is the heuristic to avoid decoding the user instructions OK (you are just pointing out that you should not be given credit

Re: [PATCH 6/7] x86/traps: Fix up invalid PASID

2020-04-27 Thread Raj, Ashok
Hi Thomas On Sun, Apr 26, 2020 at 05:25:06PM +0200, Thomas Gleixner wrote: > Fenghua Yu writes: > > A #GP fault is generated when ENQCMD instruction is executed without > > a valid PASID value programmed in. > > Programmed in what? > > > The #GP fault handler will initialize the current

Re: [PATCH 6/7] x86/traps: Fix up invalid PASID

2020-04-27 Thread Fenghua Yu
On Sun, Apr 26, 2020 at 05:25:06PM +0200, Thomas Gleixner wrote: > Fenghua Yu writes: > > A #GP fault is generated when ENQCMD instruction is executed without > > a valid PASID value programmed in. > > Programmed in what? Will change to "...programmed in the PASID MSR". > > > The #GP fault

Re: [PATCH 6/7] x86/traps: Fix up invalid PASID

2020-04-26 Thread Thomas Gleixner
Fenghua Yu writes: > A #GP fault is generated when ENQCMD instruction is executed without > a valid PASID value programmed in. Programmed in what? > The #GP fault handler will initialize the current thread's PASID MSR. > > The following heuristic is used to avoid decoding the user instructions