[edk2-devel] [PATCH] MdeModulePkg/ScsiDisk: Change TPL to NOTIFY

2021-12-14 Thread Jeff Brasen via groups.io
Increase TPL to TPL_NOTIFY to allow for use if caller is > TPL_CALLBACK. This allows services like variable services that run at TPL_NOTIFY to be hosted on ScsiDisks (i.e. UFS) Aligns with the eMMC driver that also uses a higher TPL. This change was made in 3b1d8241d0dac25c5e678c364fa2754ac1731060

Re: [edk2-devel] [PATCH] MdeModulePkg/ScsiDisk: Change TPL to NOTIFY

2022-03-10 Thread Jeff Brasen via groups.io
Resuming this PATCH to see if there are any additional thoughts on this. In response to the query about DXE/BDS services we have some internal connection logic that runs in DXE to connect the devices that are needed for arch services that have to be connected prior the end of dxe. Thanks, Jeff

Re: [edk2-devel] [PATCH] MdeModulePkg/ScsiDisk: Change TPL to NOTIFY

2022-06-17 Thread Jeff Brasen via groups.io
Resending this as I replying via edk2.groups.io doesn't seem to copy maintainers. Resuming this patch to see if there are any additional thoughts on this. In response to the query about DXE/BDS services we have some internal connection logic that runs in DXE to connect the devices that are need

Re: [edk2-devel] [PATCH] MdeModulePkg/ScsiDisk: Change TPL to NOTIFY

2022-06-22 Thread Wu, Hao A
[PATCH] MdeModulePkg/ScsiDisk: Change TPL to > NOTIFY > > Resending this as I replying via edk2.groups.io doesn't seem to copy > maintainers. > > Resuming this patch to see if there are any additional thoughts on this. > > In response to the query about DXE/BDS serv

Re: [edk2-devel] [PATCH] MdeModulePkg/ScsiDisk: Change TPL to NOTIFY

2022-06-22 Thread Wu, Hao A
To: devel@edk2.groups.io; jbra...@nvidia.com > Cc: Ni, Ray ; Kinney, Michael D > ; Gao, Liming ; > Zeng, Star > Subject: RE: [edk2-devel] [PATCH] MdeModulePkg/ScsiDisk: Change TPL to > NOTIFY > > Hello, > > Several concerns: > 1. Raising the TPL to NOTIFY level for bloc

Re: [edk2-devel] [PATCH] MdeModulePkg/ScsiDisk: Change TPL to NOTIFY

2021-12-14 Thread Wu, Hao A
> -Original Message- > From: Jeff Brasen > Sent: Wednesday, December 15, 2021 1:59 AM > To: devel@edk2.groups.io > Cc: Wu, Hao A ; Ni, Ray ; Jeff > Brasen > Subject: [PATCH] MdeModulePkg/ScsiDisk: Change TPL to NOTIFY > > Increase TPL to TPL_NOTIFY to allow for use if caller is > TPL_CAL

Re: [edk2-devel] [PATCH] MdeModulePkg/ScsiDisk: Change TPL to NOTIFY

2021-12-14 Thread Jeff Brasen via groups.io
> -Original Message- > From: Wu, Hao A > Sent: Tuesday, December 14, 2021 8:00 PM > To: Jeff Brasen ; devel@edk2.groups.io > Cc: Ni, Ray > Subject: RE: [PATCH] MdeModulePkg/ScsiDisk: Change TPL to NOTIFY > > External email: Use caution opening links or attachments > > > > -Origi

Re: [edk2-devel] [PATCH] MdeModulePkg/ScsiDisk: Change TPL to NOTIFY

2021-12-14 Thread Wu, Hao A
(Add more people) Hello Mike, Liming and Star, Do you have suggestions for the below question raised from Jeff Brasen: " The core of the issue I am trying to solve it support variable services on a UFS device. When the UFS blockIO is invoked from variable services it is not allowed (which does

Re: [edk2-devel] [PATCH] MdeModulePkg/ScsiDisk: Change TPL to NOTIFY

2021-12-16 Thread Zeng, Star
Jeff's statement about the TPLs in variable and PCD drivers seems correct, not sure the impact to change them. No good idea from me about the TPL change in ScsiDisk. 😊 Side question: Variable service is an arch service which should be ready at DXE phase, ScsiDisk is a driver model driver which