Re: [edk2] [PATCH 0/2] Fixe out-of-sync issue between GCD and CPU driver

2017-09-21 Thread Wang, Jian J
Subject: Re: [edk2] [PATCH 0/2] Fixe out-of-sync issue between GCD and CPU driver On Wed, Sep 20, 2017 at 07:53:36AM +, Wang, Jian J wrote: > Jiewen, > > Yes. The patch has passed the validation of the UEFI booting Windows 10, > Windows 7 and Fedora 25. > > If no comments f

Re: [edk2] [PATCH 0/2] Fixe out-of-sync issue between GCD and CPU driver

2017-09-21 Thread Anthony PERARD
On Wed, Sep 20, 2017 at 07:53:36AM +, Wang, Jian J wrote: > Jiewen, > > Yes. The patch has passed the validation of the UEFI booting Windows 10, > Windows 7 and Fedora 25. > > If no comments from you and others, I'll push the patch to upstream soon. > Hi, I think this patch series does no

Re: [edk2] [PATCH 0/2] Fixe out-of-sync issue between GCD and CPU driver

2017-09-20 Thread Yao, Jiewen
Thanks. I have no other comments. From: Wang, Jian J Sent: Wednesday, September 20, 2017 3:54 PM To: Yao, Jiewen ; edk2-devel@lists.01.org Cc: Kinney, Michael D ; Laszlo Ersek ; Dong, Eric ; Zeng, Star Subject: RE: [edk2] [PATCH 0/2] Fixe out-of-sync issue between GCD and CPU driver Jiewen

Re: [edk2] [PATCH 0/2] Fixe out-of-sync issue between GCD and CPU driver

2017-09-20 Thread Wang, Jian J
PM To: Wang, Jian J ; edk2-devel@lists.01.org Cc: Kinney, Michael D ; Laszlo Ersek ; Dong, Eric ; Zeng, Star Subject: RE: [edk2] [PATCH 0/2] Fixe out-of-sync issue between GCD and CPU driver Hi I take a look at the driver. I think the update is good. Would you please share what test you have done

Re: [edk2] [PATCH 0/2] Fixe out-of-sync issue between GCD and CPU driver

2017-09-19 Thread Yao, Jiewen
20, 2017 1:12 PM > To: Wang, Jian J ; edk2-devel@lists.01.org > Cc: Kinney, Michael D ; Laszlo Ersek > ; Yao, Jiewen ; Dong, Eric > ; Zeng, Star > Subject: RE: [edk2] [PATCH 0/2] Fixe out-of-sync issue between GCD and CPU > driver > > Jiewen, > > Any comments on th

Re: [edk2] [PATCH 0/2] Fixe out-of-sync issue between GCD and CPU driver

2017-09-19 Thread Wang, Jian J
Subject: [edk2] [PATCH 0/2] Fixe out-of-sync issue between GCD and CPU driver There're two issues here actually. >From GCD perspective, its SetMemorySpaceAttributes() method doesn't accept page related attributes. That means users cannot use it to change page attributes, and have t

[edk2] [PATCH 0/2] Fixe out-of-sync issue between GCD and CPU driver

2017-09-18 Thread Jian J Wang
There're two issues here actually. >From GCD perspective, its SetMemorySpaceAttributes() method doesn't accept page related attributes. That means users cannot use it to change page attributes, and have to turn to CPU arch protocol to do it, which is not be allowed by PI spec. >From CpuDxe driver

Re: [edk2] [PATCH 0/2] Fixe out-of-sync issue between GCD and CPU driver

2017-09-18 Thread Wang, Jian J
; Yao, Jiewen ; Zeng, Star Subject: [edk2] [PATCH 0/2] Fixe out-of-sync issue between GCD and CPU driver There're two issues here actually. >From GCD perspective, its SetMemorySpaceAttributes() method doesn't accept page related attributes. That means users cannot use it to change pa

[edk2] [PATCH 0/2] Fixe out-of-sync issue between GCD and CPU driver

2017-09-17 Thread Jian J Wang
There're two issues here actually. >From GCD perspective, its SetMemorySpaceAttributes() method doesn't accept page related attributes. That means users cannot use it to change page attributes, and have to turn to CPU arch protocol to do it, which is not be allowed by PI spec. >From CpuDxe driver