Re: [edk2] MdeModulePkg/UefiBootManagerLib: recursive call of BmRepairAllControllers

2018-02-24 Thread Guo Heyi
Sure. On Sat, Feb 24, 2018 at 08:40:20AM +, Ni, Ruiyu wrote: > Will you submit a patch for this change? > > Thanks/Ray > > > -Original Message- > > From: Guo Heyi [mailto:heyi@linaro.org] > > Sent: Saturday, February 24, 2018 4:29 PM > > To: Ni, Ruiyu > >

Re: [edk2] MdeModulePkg/UefiBootManagerLib: recursive call of BmRepairAllControllers

2018-02-24 Thread Ni, Ruiyu
Will you submit a patch for this change? Thanks/Ray > -Original Message- > From: Guo Heyi [mailto:heyi@linaro.org] > Sent: Saturday, February 24, 2018 4:29 PM > To: Ni, Ruiyu > Cc: Guo Heyi ; edk2-devel@lists.01.org; Zeng, Star >

Re: [edk2] MdeModulePkg/UefiBootManagerLib: recursive call of BmRepairAllControllers

2018-02-24 Thread Guo Heyi
On Sat, Feb 24, 2018 at 04:20:52PM +0800, Ni, Ruiyu wrote: > On 2/24/2018 2:23 PM, Guo Heyi wrote: > >Hi folks, > > > >In BmDriverHealth.c, function BmRepairAllControllers may recursively call > >itself > >if some driver health protocol returns > >EfiDriverHealthStatusReconnectRequired. >

Re: [edk2] MdeModulePkg/UefiBootManagerLib: recursive call of BmRepairAllControllers

2018-02-24 Thread Ni, Ruiyu
On 2/24/2018 2:23 PM, Guo Heyi wrote: Hi folks, In BmDriverHealth.c, function BmRepairAllControllers may recursively call itself if some driver health protocol returns EfiDriverHealthStatusReconnectRequired. However, if there is something wrong in some 3rd party driver (e.g. PCI oprom), the

[edk2] MdeModulePkg/UefiBootManagerLib: recursive call of BmRepairAllControllers

2018-02-23 Thread Guo Heyi
Hi folks, In BmDriverHealth.c, function BmRepairAllControllers may recursively call itself if some driver health protocol returns EfiDriverHealthStatusReconnectRequired. However, if there is something wrong in some 3rd party driver (e.g. PCI oprom), the driver health protocol of that driver may