<html xmlns:v="urn:schemas-microsoft-com:vml" xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:w="urn:schemas-microsoft-com:office:word" xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" xmlns="http://www.w3.org/TR/REC-html40">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=us-ascii">
<meta name="Generator" content="Microsoft Word 15 (filtered medium)">
<!--[if !mso]><style>v\:* {behavior:url(#default#VML);}
o\:* {behavior:url(#default#VML);}
w\:* {behavior:url(#default#VML);}
.shape {behavior:url(#default#VML);}
</style><![endif]--><style><!--
/* Font Definitions */
@font-face
{font-family:"Cambria Math";
panose-1:2 4 5 3 5 4 6 3 2 4;}
@font-face
{font-family:Calibri;
panose-1:2 15 5 2 2 2 4 3 2 4;}
/* Style Definitions */
p.MsoNormal, li.MsoNormal, div.MsoNormal
{margin:0in;
margin-bottom:.0001pt;
font-size:11.0pt;
font-family:"Calibri","sans-serif";}
a:link, span.MsoHyperlink
{mso-style-priority:99;
color:#0563C1;
text-decoration:underline;}
a:visited, span.MsoHyperlinkFollowed
{mso-style-priority:99;
color:#954F72;
text-decoration:underline;}
p
{mso-style-priority:99;
mso-margin-top-alt:auto;
margin-right:0in;
mso-margin-bottom-alt:auto;
margin-left:0in;
font-size:12.0pt;
font-family:"Times New Roman","serif";}
p.msochpdefault, li.msochpdefault, div.msochpdefault
{mso-style-name:msochpdefault;
mso-margin-top-alt:auto;
margin-right:0in;
mso-margin-bottom-alt:auto;
margin-left:0in;
font-size:12.0pt;
font-family:"Calibri","sans-serif";}
span.emailstyle17
{mso-style-name:emailstyle17;
font-family:"Calibri","sans-serif";
color:windowtext;}
span.EmailStyle20
{mso-style-type:personal-reply;
font-family:"Calibri","sans-serif";
color:#1F497D;}
.MsoChpDefault
{mso-style-type:export-only;
font-size:10.0pt;
font-family:"Calibri","sans-serif";}
@page WordSection1
{size:8.5in 11.0in;
margin:1.0in 1.0in 1.0in 1.0in;}
div.WordSection1
{page:WordSection1;}
--></style><!--[if gte mso 9]><xml>
<o:shapedefaults v:ext="edit" spidmax="1026" />
</xml><![endif]--><!--[if gte mso 9]><xml>
<o:shapelayout v:ext="edit">
<o:idmap v:ext="edit" data="1" />
</o:shapelayout></xml><![endif]-->
</head>
<body lang="EN-US" link="#0563C1" vlink="#954F72">
<div class="WordSection1">
<p class="MsoNormal"><span style="color:#1F497D">Jiaxin:<o:p></o:p></span></p>
<p class="MsoNormal"><span style="color:#1F497D"><o:p> </o:p></span></p>
<p class="MsoNormal"><span style="color:#1F497D">Did you get chance to review the patch?<o:p></o:p></span></p>
<p class="MsoNormal"><span style="color:#1F497D"><o:p> </o:p></span></p>
<p class="MsoNormal"><span style="color:#1F497D">-Siva<o:p></o:p></span></p>
<div>
<div style="border:none;border-top:solid #E1E1E1 1.0pt;padding:3.0pt 0in 0in 0in">
<p class="MsoNormal"><b>From:</b> devel@edk2.groups.io [mailto:devel@edk2.groups.io]
<b>On Behalf Of </b>Sivaraman Nainar<br>
<b>Sent:</b> Thursday, August 13, 2020 12:21 PM<br>
<b>To:</b> devel@edk2.groups.io<br>
<b>Cc:</b> liming.gao@intel.com; jiaxin.wu@intel.com<br>
<b>Subject:</b> [edk2-devel] [Patch ] IP is getting Configured when IP Configuration is disabled<o:p></o:p></p>
</div>
</div>
<p class="MsoNormal"><o:p> </o:p></p>
<div>
<p class="MsoNormal">BZ: <a href="https://bugzilla.tianocore.org/show_bug.cgi?id=2873">
https://bugzilla.tianocore.org/show_bug.cgi?id=2873</a><o:p></o:p></p>
<p class="MsoNormal"> <o:p></o:p></p>
<p class="MsoNormal">*** E:\Work\NWStack\EDK2Tickets\IPConfigFix\Mod\Ip4Config2Impl.c 2020-08-04 18:38:06.000000000 +05-30<o:p></o:p></p>
<p class="MsoNormal">--- E:\Work\NWStack\EDK2Tickets\IPConfigFix\Org\Ip4Config2Impl.c 2020-05-07 22:54:46.000000000 +05-30<o:p></o:p></p>
<p class="MsoNormal">***************<o:p></o:p></p>
<p class="MsoNormal">*** 1965,1978 ****<o:p></o:p></p>
<p class="MsoNormal"> IP4_CONFIG2_INSTANCE *TmpInstance;<o:p></o:p></p>
<p class="MsoNormal"> LIST_ENTRY *Entry;<o:p></o:p></p>
<p class="MsoNormal"> EFI_STATUS Status;<o:p></o:p></p>
<p class="MsoNormal"> UINTN Index;<o:p></o:p></p>
<p class="MsoNormal"> UINT16 IfIndex;<o:p></o:p></p>
<p class="MsoNormal"> IP4_CONFIG2_DATA_ITEM *DataItem;<o:p></o:p></p>
<p class="MsoNormal">! BOOLEAN IPConfigEnabled = FALSE;<o:p></o:p></p>
<p class="MsoNormal">! IP4_CONFIG2_IFR_NVDATA *IfrFromNvData = NULL;<o:p></o:p></p>
<p class="MsoNormal"> <o:p></o:p></p>
<p class="MsoNormal"> IpSb = IP4_SERVICE_FROM_IP4_CONFIG2_INSTANCE (Instance);<o:p></o:p></p>
<p class="MsoNormal"> <o:p></o:p></p>
<p class="MsoNormal"> Instance->Signature = IP4_CONFIG2_INSTANCE_SIGNATURE;<o:p></o:p></p>
<p class="MsoNormal"> <o:p></o:p></p>
<p class="MsoNormal"> <o:p></o:p></p>
<p class="MsoNormal">--- 1965,1977 ----<o:p></o:p></p>
<p class="MsoNormal"> IP4_CONFIG2_INSTANCE *TmpInstance;<o:p></o:p></p>
<p class="MsoNormal"> LIST_ENTRY *Entry;<o:p></o:p></p>
<p class="MsoNormal"> EFI_STATUS Status;<o:p></o:p></p>
<p class="MsoNormal"> UINTN Index;<o:p></o:p></p>
<p class="MsoNormal"> UINT16 IfIndex;<o:p></o:p></p>
<p class="MsoNormal"> IP4_CONFIG2_DATA_ITEM *DataItem;<o:p></o:p></p>
<p class="MsoNormal">! <o:p></o:p></p>
<p class="MsoNormal"> <o:p></o:p></p>
<p class="MsoNormal"> IpSb = IP4_SERVICE_FROM_IP4_CONFIG2_INSTANCE (Instance);<o:p></o:p></p>
<p class="MsoNormal"> <o:p></o:p></p>
<p class="MsoNormal"> Instance->Signature = IP4_CONFIG2_INSTANCE_SIGNATURE;<o:p></o:p></p>
<p class="MsoNormal"> <o:p></o:p></p>
<p class="MsoNormal"> <o:p></o:p></p>
<p class="MsoNormal">***************<o:p></o:p></p>
<p class="MsoNormal">*** 2038,2063 ****<o:p></o:p></p>
<p class="MsoNormal"> <o:p></o:p></p>
<p class="MsoNormal"> //<o:p></o:p></p>
<p class="MsoNormal"> // Try to read the config data from NV variable.<o:p></o:p></p>
<p class="MsoNormal"> // If not found, write initialized config data into NV variable<o:p></o:p></p>
<p class="MsoNormal"> // as a default config data.<o:p></o:p></p>
<p class="MsoNormal"> //<o:p></o:p></p>
<p class="MsoNormal">! Status = GetVariable2(IpSb->MacString, &gIp4Config2NvDataGuid, (VOID**)&IfrFromNvData, NULL);<o:p></o:p></p>
<p class="MsoNormal">! if(!EFI_ERROR(Status)){<o:p></o:p></p>
<p class="MsoNormal">! if (IfrFromNvData->Configure == TRUE) {<o:p></o:p></p>
<p class="MsoNormal">! Status = Ip4Config2ReadConfigData (IpSb->MacString, Instance);<o:p></o:p></p>
<p class="MsoNormal">! if (Status == EFI_NOT_FOUND) {<o:p></o:p></p>
<p class="MsoNormal">! Status = Ip4Config2WriteConfigData (IpSb->MacString, Instance);<o:p></o:p></p>
<p class="MsoNormal">! }<o:p></o:p></p>
<p class="MsoNormal">! IPConfigEnabled = TRUE;<o:p></o:p></p>
<p class="MsoNormal">! }<o:p></o:p></p>
<p class="MsoNormal">! FreePool(IfrFromNvData);<o:p></o:p></p>
<p class="MsoNormal">! }<o:p></o:p></p>
<p class="MsoNormal">! if(!IPConfigEnabled){<o:p></o:p></p>
<p class="MsoNormal">! Status = Ip4Config2WriteConfigData (IpSb->MacString, Instance);<o:p></o:p></p>
<p class="MsoNormal">! }<o:p></o:p></p>
<p class="MsoNormal"> <o:p></o:p></p>
<p class="MsoNormal"> if (EFI_ERROR (Status)) {<o:p></o:p></p>
<p class="MsoNormal"> return Status;<o:p></o:p></p>
<p class="MsoNormal"> }<o:p></o:p></p>
<p class="MsoNormal"> <o:p></o:p></p>
<p class="MsoNormal"> Instance->Ip4Config2.SetData = EfiIp4Config2SetData;<o:p></o:p></p>
<p class="MsoNormal">--- 2037,2052 ----<o:p></o:p></p>
<p class="MsoNormal"> <o:p></o:p></p>
<p class="MsoNormal"> //<o:p></o:p></p>
<p class="MsoNormal"> // Try to read the config data from NV variable.<o:p></o:p></p>
<p class="MsoNormal"> // If not found, write initialized config data into NV variable<o:p></o:p></p>
<p class="MsoNormal"> // as a default config data.<o:p></o:p></p>
<p class="MsoNormal"> //<o:p></o:p></p>
<p class="MsoNormal">! Status = Ip4Config2ReadConfigData (IpSb->MacString, Instance);<o:p></o:p></p>
<p class="MsoNormal">! if (Status == EFI_NOT_FOUND) {<o:p></o:p></p>
<p class="MsoNormal">! Status = Ip4Config2WriteConfigData (IpSb->MacString, Instance);<o:p></o:p></p>
<p class="MsoNormal">! }<o:p></o:p></p>
<p class="MsoNormal"> <o:p></o:p></p>
<p class="MsoNormal"> if (EFI_ERROR (Status)) {<o:p></o:p></p>
<p class="MsoNormal"> return Status;<o:p></o:p></p>
<p class="MsoNormal"> }<o:p></o:p></p>
<p class="MsoNormal"> <o:p></o:p></p>
<p class="MsoNormal"> Instance->Ip4Config2.SetData = EfiIp4Config2SetData;<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"></o:p></span></p>
</div>
<p class="MsoNormal"><span style="font-size:12.0pt;font-family:"Times New Roman","serif"">This e-mail is intended for the use of the addressee only and may contain privileged, confidential, or proprietary information that is exempt from disclosure under law.
If you have received this message in error, please inform us promptly by reply e-mail, then delete the e-mail and destroy any printed copy. Thank you.
<o:p></o:p></span></p>
</div>
</body>
</html>
<div width="1" style="color:white;clear:both">_._,_._,_</div>
<hr>
Groups.io Links:<p>
You receive all messages sent to this group.
<p>
<a target="_blank" href="https://edk2.groups.io/g/devel/message/64509">View/Reply Online (#64509)</a> |
|
<a target="_blank" href="https://groups.io/mt/76163208/1813853">Mute This Topic</a>
| <a href="https://edk2.groups.io/g/devel/post">New Topic</a><br>
<br>
<a href="https://edk2.groups.io/g/devel/editsub/1813853">Your Subscription</a> |
<a href="mailto:devel+owner@edk2.groups.io">Contact Group Owner</a> |
<a href="https://edk2.groups.io/g/devel/unsub">Unsubscribe</a>
[edk2-devel-archive@redhat.com]<br>
<div width="1" style="color:white;clear:both">_._,_._,_</div>