* [Patch] NetworkPkg/IScsiDxe: Display InitiatorInfo in attempt page even DHCP enabled. @ 2017-10-13 6:01 Jiaxin Wu 2017-10-13 6:24 ` Fu, Siyuan 0 siblings, 1 reply; 4+ messages in thread From: Jiaxin Wu @ 2017-10-13 6:01 UTC (permalink / raw) To: edk2-devel; +Cc: Karunakar P, Ye Ting, Fu Siyuan, Wu Jiaxin Cc: Karunakar P <karunakarp@amiindia.co.in> Cc: Ye Ting <ting.ye@intel.com> Cc: Fu Siyuan <siyuan.fu@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Wu Jiaxin <jiaxin.wu@intel.com> --- NetworkPkg/IScsiDxe/IScsiConfigVfr.vfr | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/NetworkPkg/IScsiDxe/IScsiConfigVfr.vfr b/NetworkPkg/IScsiDxe/IScsiConfigVfr.vfr index d401419..35e8f9a 100644 --- a/NetworkPkg/IScsiDxe/IScsiConfigVfr.vfr +++ b/NetworkPkg/IScsiDxe/IScsiConfigVfr.vfr @@ -189,13 +189,14 @@ formset flags = INTERACTIVE, key = KEY_DHCP_ENABLE, endcheckbox; endif; - suppressif ideqval ISCSI_CONFIG_IFR_NVDATA.InitiatorInfoFromDhcp == 0x01 OR - ideqval ISCSI_CONFIG_IFR_NVDATA.IpMode == IP_MODE_IP6 OR + suppressif ideqval ISCSI_CONFIG_IFR_NVDATA.IpMode == IP_MODE_IP6 OR ideqval ISCSI_CONFIG_IFR_NVDATA.IpMode == IP_MODE_AUTOCONFIG; + + grayoutif ideqval ISCSI_CONFIG_IFR_NVDATA.InitiatorInfoFromDhcp == 0x01; string varid = ISCSI_CONFIG_IFR_NVDATA.LocalIp, prompt = STRING_TOKEN(STR_ISCSI_LOCAL_IP_ADDRESS), help = STRING_TOKEN(STR_ISCSI_IP_ADDRESS_HELP), flags = INTERACTIVE, key = KEY_LOCAL_IP, @@ -218,10 +219,11 @@ formset flags = INTERACTIVE, key = KEY_GATE_WAY, minsize = IP4_MIN_SIZE, maxsize = IP4_MAX_SIZE, endstring; + endif; endif; suppressif ideqval ISCSI_CONFIG_IFR_NVDATA.IpMode == IP_MODE_AUTOCONFIG; subtitle text = STRING_TOKEN(STR_NULL); -- 1.9.5.msysgit.1 ^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [Patch] NetworkPkg/IScsiDxe: Display InitiatorInfo in attempt page even DHCP enabled. 2017-10-13 6:01 [Patch] NetworkPkg/IScsiDxe: Display InitiatorInfo in attempt page even DHCP enabled Jiaxin Wu @ 2017-10-13 6:24 ` Fu, Siyuan 2017-10-13 11:48 ` Karunakar P 0 siblings, 1 reply; 4+ messages in thread From: Fu, Siyuan @ 2017-10-13 6:24 UTC (permalink / raw) To: Wu, Jiaxin, edk2-devel@lists.01.org; +Cc: Karunakar P, Ye, Ting Reviewed-by: Fu Siyuan <siyuan.fu@intel.com> -----Original Message----- From: Wu, Jiaxin Sent: Friday, October 13, 2017 2:01 PM To: edk2-devel@lists.01.org Cc: Karunakar P <karunakarp@amiindia.co.in>; Ye, Ting <ting.ye@intel.com>; Fu, Siyuan <siyuan.fu@intel.com>; Wu, Jiaxin <jiaxin.wu@intel.com> Subject: [Patch] NetworkPkg/IScsiDxe: Display InitiatorInfo in attempt page even DHCP enabled. Cc: Karunakar P <karunakarp@amiindia.co.in> Cc: Ye Ting <ting.ye@intel.com> Cc: Fu Siyuan <siyuan.fu@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Wu Jiaxin <jiaxin.wu@intel.com> --- NetworkPkg/IScsiDxe/IScsiConfigVfr.vfr | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/NetworkPkg/IScsiDxe/IScsiConfigVfr.vfr b/NetworkPkg/IScsiDxe/IScsiConfigVfr.vfr index d401419..35e8f9a 100644 --- a/NetworkPkg/IScsiDxe/IScsiConfigVfr.vfr +++ b/NetworkPkg/IScsiDxe/IScsiConfigVfr.vfr @@ -189,13 +189,14 @@ formset flags = INTERACTIVE, key = KEY_DHCP_ENABLE, endcheckbox; endif; - suppressif ideqval ISCSI_CONFIG_IFR_NVDATA.InitiatorInfoFromDhcp == 0x01 OR - ideqval ISCSI_CONFIG_IFR_NVDATA.IpMode == IP_MODE_IP6 OR + suppressif ideqval ISCSI_CONFIG_IFR_NVDATA.IpMode == IP_MODE_IP6 OR ideqval ISCSI_CONFIG_IFR_NVDATA.IpMode == IP_MODE_AUTOCONFIG; + + grayoutif ideqval ISCSI_CONFIG_IFR_NVDATA.InitiatorInfoFromDhcp == 0x01; string varid = ISCSI_CONFIG_IFR_NVDATA.LocalIp, prompt = STRING_TOKEN(STR_ISCSI_LOCAL_IP_ADDRESS), help = STRING_TOKEN(STR_ISCSI_IP_ADDRESS_HELP), flags = INTERACTIVE, key = KEY_LOCAL_IP, @@ -218,10 +219,11 @@ formset flags = INTERACTIVE, key = KEY_GATE_WAY, minsize = IP4_MIN_SIZE, maxsize = IP4_MAX_SIZE, endstring; + endif; endif; suppressif ideqval ISCSI_CONFIG_IFR_NVDATA.IpMode == IP_MODE_AUTOCONFIG; subtitle text = STRING_TOKEN(STR_NULL); -- 1.9.5.msysgit.1 ^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [Patch] NetworkPkg/IScsiDxe: Display InitiatorInfo in attempt page even DHCP enabled. 2017-10-13 6:24 ` Fu, Siyuan @ 2017-10-13 11:48 ` Karunakar P 2017-10-16 6:25 ` Wu, Jiaxin 0 siblings, 1 reply; 4+ messages in thread From: Karunakar P @ 2017-10-13 11:48 UTC (permalink / raw) To: 'Fu, Siyuan', Wu, Jiaxin, edk2-devel@lists.01.org; +Cc: Ye, Ting Hi Jiaxin/Siyuan, I've verified Initiator details with this changes. I found an issue like Initiator Details are shown for Connection less NIC Please find below details for more details [Issue details and steps to reproduce] 1. SUT have 2 LAN ports MAC XX:XX:XX:XX:XX:AA -> LAN Cable NOT connected to this port, i.e. Link Status is Disconnected MAC XX:XX:XX:XX:XX:BB -> LAN Cable Connected to this port, i.e. Link Status is Connected 2. I've added two attempts a. Attempt1 added to connection less NIC (MAC XX:XX:XX:XX:XX:AA ) b. Attempt2 added to connected NIC (MAC XX:XX:XX:XX:XX:BB ) [Observations] 1. Initiator IP Address and Subnet Mask displayed for both Attempt1 and Attempt2 Could you please comment on this observation? Thanks, Karunakar -----Original Message----- From: Fu, Siyuan [mailto:siyuan.fu@intel.com] Sent: Friday, October 13, 2017 11:54 AM To: Wu, Jiaxin; edk2-devel@lists.01.org Cc: Karunakar P; Ye, Ting Subject: RE: [Patch] NetworkPkg/IScsiDxe: Display InitiatorInfo in attempt page even DHCP enabled. Reviewed-by: Fu Siyuan <siyuan.fu@intel.com> -----Original Message----- From: Wu, Jiaxin Sent: Friday, October 13, 2017 2:01 PM To: edk2-devel@lists.01.org Cc: Karunakar P <karunakarp@amiindia.co.in>; Ye, Ting <ting.ye@intel.com>; Fu, Siyuan <siyuan.fu@intel.com>; Wu, Jiaxin <jiaxin.wu@intel.com> Subject: [Patch] NetworkPkg/IScsiDxe: Display InitiatorInfo in attempt page even DHCP enabled. Cc: Karunakar P <karunakarp@amiindia.co.in> Cc: Ye Ting <ting.ye@intel.com> Cc: Fu Siyuan <siyuan.fu@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Wu Jiaxin <jiaxin.wu@intel.com> --- NetworkPkg/IScsiDxe/IScsiConfigVfr.vfr | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/NetworkPkg/IScsiDxe/IScsiConfigVfr.vfr b/NetworkPkg/IScsiDxe/IScsiConfigVfr.vfr index d401419..35e8f9a 100644 --- a/NetworkPkg/IScsiDxe/IScsiConfigVfr.vfr +++ b/NetworkPkg/IScsiDxe/IScsiConfigVfr.vfr @@ -189,13 +189,14 @@ formset flags = INTERACTIVE, key = KEY_DHCP_ENABLE, endcheckbox; endif; - suppressif ideqval ISCSI_CONFIG_IFR_NVDATA.InitiatorInfoFromDhcp == 0x01 OR - ideqval ISCSI_CONFIG_IFR_NVDATA.IpMode == IP_MODE_IP6 OR + suppressif ideqval ISCSI_CONFIG_IFR_NVDATA.IpMode == IP_MODE_IP6 OR ideqval ISCSI_CONFIG_IFR_NVDATA.IpMode == IP_MODE_AUTOCONFIG; + + grayoutif ideqval ISCSI_CONFIG_IFR_NVDATA.InitiatorInfoFromDhcp == 0x01; string varid = ISCSI_CONFIG_IFR_NVDATA.LocalIp, prompt = STRING_TOKEN(STR_ISCSI_LOCAL_IP_ADDRESS), help = STRING_TOKEN(STR_ISCSI_IP_ADDRESS_HELP), flags = INTERACTIVE, key = KEY_LOCAL_IP, @@ -218,10 +219,11 @@ formset flags = INTERACTIVE, key = KEY_GATE_WAY, minsize = IP4_MIN_SIZE, maxsize = IP4_MAX_SIZE, endstring; + endif; endif; suppressif ideqval ISCSI_CONFIG_IFR_NVDATA.IpMode == IP_MODE_AUTOCONFIG; subtitle text = STRING_TOKEN(STR_NULL); -- 1.9.5.msysgit.1 ^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [Patch] NetworkPkg/IScsiDxe: Display InitiatorInfo in attempt page even DHCP enabled. 2017-10-13 11:48 ` Karunakar P @ 2017-10-16 6:25 ` Wu, Jiaxin 0 siblings, 0 replies; 4+ messages in thread From: Wu, Jiaxin @ 2017-10-16 6:25 UTC (permalink / raw) To: Karunakar P, Fu, Siyuan, edk2-devel@lists.01.org; +Cc: Ye, Ting Hi Karunakar, Thanks the verification, I agree it's another iSCSI bug that exposed by this patch. In IScsiGetConfigData() function, The existing attempt will always trigger the DHCP process if Initiator DHCP enabled no matter it associates with the current NIC or not. That's incorrect and unreasonable as you overserved if we display the InitiatorInfo in attempt page. I will send another separated but series patch to fix this issue. Thanks, Jiaxin > -----Original Message----- > From: Karunakar P [mailto:karunakarp@amiindia.co.in] > Sent: Friday, October 13, 2017 7:49 PM > To: Fu, Siyuan <siyuan.fu@intel.com>; Wu, Jiaxin <jiaxin.wu@intel.com>; > edk2-devel@lists.01.org > Cc: Ye, Ting <ting.ye@intel.com> > Subject: RE: [Patch] NetworkPkg/IScsiDxe: Display InitiatorInfo in attempt > page even DHCP enabled. > > Hi Jiaxin/Siyuan, > > I've verified Initiator details with this changes. > > I found an issue like Initiator Details are shown for Connection less NIC > > Please find below details for more details > > [Issue details and steps to reproduce] > 1. SUT have 2 LAN ports > MAC XX:XX:XX:XX:XX:AA -> LAN Cable NOT connected to this port, i.e. > Link Status is Disconnected > MAC XX:XX:XX:XX:XX:BB -> LAN Cable Connected to this port, i.e. Link > Status is Connected > > 2. I've added two attempts > a. Attempt1 added to connection less NIC (MAC XX:XX:XX:XX:XX:AA ) > b. Attempt2 added to connected NIC (MAC XX:XX:XX:XX:XX:BB ) > > [Observations] > 1. Initiator IP Address and Subnet Mask displayed for both Attempt1 and > Attempt2 > > Could you please comment on this observation? > > Thanks, > Karunakar > > -----Original Message----- > From: Fu, Siyuan [mailto:siyuan.fu@intel.com] > Sent: Friday, October 13, 2017 11:54 AM > To: Wu, Jiaxin; edk2-devel@lists.01.org > Cc: Karunakar P; Ye, Ting > Subject: RE: [Patch] NetworkPkg/IScsiDxe: Display InitiatorInfo in attempt > page even DHCP enabled. > > Reviewed-by: Fu Siyuan <siyuan.fu@intel.com> > > -----Original Message----- > From: Wu, Jiaxin > Sent: Friday, October 13, 2017 2:01 PM > To: edk2-devel@lists.01.org > Cc: Karunakar P <karunakarp@amiindia.co.in>; Ye, Ting <ting.ye@intel.com>; > Fu, Siyuan <siyuan.fu@intel.com>; Wu, Jiaxin <jiaxin.wu@intel.com> > Subject: [Patch] NetworkPkg/IScsiDxe: Display InitiatorInfo in attempt page > even DHCP enabled. > > Cc: Karunakar P <karunakarp@amiindia.co.in> > Cc: Ye Ting <ting.ye@intel.com> > Cc: Fu Siyuan <siyuan.fu@intel.com> > Contributed-under: TianoCore Contribution Agreement 1.0 > Signed-off-by: Wu Jiaxin <jiaxin.wu@intel.com> > --- > NetworkPkg/IScsiDxe/IScsiConfigVfr.vfr | 6 ++++-- > 1 file changed, 4 insertions(+), 2 deletions(-) > > diff --git a/NetworkPkg/IScsiDxe/IScsiConfigVfr.vfr > b/NetworkPkg/IScsiDxe/IScsiConfigVfr.vfr > index d401419..35e8f9a 100644 > --- a/NetworkPkg/IScsiDxe/IScsiConfigVfr.vfr > +++ b/NetworkPkg/IScsiDxe/IScsiConfigVfr.vfr > @@ -189,13 +189,14 @@ formset > flags = INTERACTIVE, > key = KEY_DHCP_ENABLE, > endcheckbox; > endif; > > - suppressif ideqval ISCSI_CONFIG_IFR_NVDATA.InitiatorInfoFromDhcp == > 0x01 OR > - ideqval ISCSI_CONFIG_IFR_NVDATA.IpMode == IP_MODE_IP6 OR > + suppressif ideqval ISCSI_CONFIG_IFR_NVDATA.IpMode == IP_MODE_IP6 > OR > ideqval ISCSI_CONFIG_IFR_NVDATA.IpMode == > IP_MODE_AUTOCONFIG; > + > + grayoutif ideqval ISCSI_CONFIG_IFR_NVDATA.InitiatorInfoFromDhcp == > 0x01; > string varid = ISCSI_CONFIG_IFR_NVDATA.LocalIp, > prompt = STRING_TOKEN(STR_ISCSI_LOCAL_IP_ADDRESS), > help = STRING_TOKEN(STR_ISCSI_IP_ADDRESS_HELP), > flags = INTERACTIVE, > key = KEY_LOCAL_IP, > @@ -218,10 +219,11 @@ formset > flags = INTERACTIVE, > key = KEY_GATE_WAY, > minsize = IP4_MIN_SIZE, > maxsize = IP4_MAX_SIZE, > endstring; > + endif; > > endif; > > suppressif ideqval ISCSI_CONFIG_IFR_NVDATA.IpMode == > IP_MODE_AUTOCONFIG; > subtitle text = STRING_TOKEN(STR_NULL); > -- > 1.9.5.msysgit.1 ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2017-10-16 6:21 UTC | newest] Thread overview: 4+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2017-10-13 6:01 [Patch] NetworkPkg/IScsiDxe: Display InitiatorInfo in attempt page even DHCP enabled Jiaxin Wu 2017-10-13 6:24 ` Fu, Siyuan 2017-10-13 11:48 ` Karunakar P 2017-10-16 6:25 ` Wu, Jiaxin
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox