Re: [PATCH v2 0/2] NetworkPkg/iScsiDxe: Update iScsiStart Policy

["Wu, Jiaxin" <jiaxin.wu@intel.com>]

Series Reviewed-by: Wu Jiaxin <jiaxin.wu@intel.com>



> -----Original Message-----
> From: hegdenag [mailto:nagaraj-p.hegde@hpe.com]
> Sent: Monday, February 13, 2017 3:20 PM
> To: edk2-devel@lists.01.org
> Cc: Wu, Jiaxin <jiaxin.wu@intel.com>; Ye, Ting <ting.ye@intel.com>; Zhang,
> Lubo <lubo.zhang@intel.com>; Fu, Siyuan <siyuan.fu@intel.com>; sriram-
> s@hpe.com
> Subject: [PATCH v2 0/2] NetworkPkg/iScsiDxe: Update iScsiStart Policy
> 
> v2: Updated list of reviewers.
> 
> Today, iSCSI driver checks whether there are AIP instances
> installed by iSCSI HBA adapter and if yes, the iSCSI driver
> will return EFI_ABORTED in its driver binding Start.
> We can not guarantee the existence of AIP instances always during
> iSCSI DBStart. Hence, we might end up doing a iScsiStart even if
> the intent of the customer was to use only iSCSI on HBA.
> The series of patch will define a new PCD value for the PCD
> PcdIScsiAIPNetworkBootPolicy, "Always use iSCSI HBA and ignore
> UEFI iSCSI", update iScsiStart to check this PCD value and return
> EFI_ABORT in this case even before checking for AIP instances.
> 
> hegdenag (2):
>   NetworkPkg/iSCSIDxe: Update the condition for IScsiStart Abort
>   NetworkPkg/NetworkPkg.dec: Update comments for the PCD
> 
>  NetworkPkg/IScsiDxe/IScsiDriver.c |  8 +++++++-
>  NetworkPkg/IScsiDxe/IScsiDriver.h | 19 +++++++++++--------
>  NetworkPkg/NetworkPkg.dec         |  5 +++--
>  3 files changed, 21 insertions(+), 11 deletions(-)
> 
> --
> 2.8.3.windows.1