From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga09.intel.com (mga09.intel.com [134.134.136.24]) (using TLSv1 with cipher CAMELLIA256-SHA (256/256 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id 16C211A1E47 for ; Thu, 25 Aug 2016 02:49:02 -0700 (PDT) Received: from orsmga003.jf.intel.com ([10.7.209.27]) by orsmga102.jf.intel.com with ESMTP; 25 Aug 2016 02:49:01 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.28,575,1464678000"; d="png'150?scan'150,208,150";a="870888103" Received: from fmsmsx104.amr.corp.intel.com ([10.18.124.202]) by orsmga003.jf.intel.com with ESMTP; 25 Aug 2016 02:49:01 -0700 Received: from fmsmsx111.amr.corp.intel.com (10.18.116.5) by fmsmsx104.amr.corp.intel.com (10.18.124.202) with Microsoft SMTP Server (TLS) id 14.3.248.2; Thu, 25 Aug 2016 02:49:00 -0700 Received: from shsmsx103.ccr.corp.intel.com (10.239.4.69) by fmsmsx111.amr.corp.intel.com (10.18.116.5) with Microsoft SMTP Server (TLS) id 14.3.248.2; Thu, 25 Aug 2016 02:48:59 -0700 Received: from shsmsx102.ccr.corp.intel.com ([169.254.2.147]) by SHSMSX103.ccr.corp.intel.com ([169.254.4.181]) with mapi id 14.03.0248.002; Thu, 25 Aug 2016 17:48:58 +0800 From: "Zhang, Lubo" To: Santhapur Naveen CC: "edk2-devel@lists.01.org" Thread-Topic: [edk2] 2nd HttpBoot fails even after upgrading to revision 24729 Thread-Index: AdH92eeTmmRENzHdTIac5lMehDH2YAACSFaA//99hID//2xmkIAAueaA//6I6ECAAue+AP//dDBA Date: Thu, 25 Aug 2016 09:48:57 +0000 Message-ID: <7619447B08B8F74DA4FF2A813B79803B01A53984@shsmsx102.ccr.corp.intel.com> References: <625A2455CC232F40B0F38F05ACED6D976C541797@VENUS1.in.megatrends.com> <7619447B08B8F74DA4FF2A813B79803B01A53538@shsmsx102.ccr.corp.intel.com> <625A2455CC232F40B0F38F05ACED6D976C541869@VENUS1.in.megatrends.com> <7619447B08B8F74DA4FF2A813B79803B01A53649@shsmsx102.ccr.corp.intel.com> <625A2455CC232F40B0F38F05ACED6D976C541A16@VENUS1.in.megatrends.com> <7619447B08B8F74DA4FF2A813B79803B01A5380E@shsmsx102.ccr.corp.intel.com> <625A2455CC232F40B0F38F05ACED6D976C542191@VENUS1.in.megatrends.com> In-Reply-To: <625A2455CC232F40B0F38F05ACED6D976C542191@VENUS1.in.megatrends.com> Accept-Language: en-US X-MS-Has-Attach: yes X-MS-TNEF-Correlator: x-originating-ip: [10.239.127.40] MIME-Version: 1.0 X-Content-Filtered-By: Mailman/MimeDel 2.1.21 Subject: Re: 2nd HttpBoot fails even after upgrading to revision 24729 X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 25 Aug 2016 09:49:02 -0000 Content-Language: en-US Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable Yes, the SVN version number I provide is not consistent with you.=20 The attachment is latest patch log details of the GitHub, you can make a = contrast when you download the code. And then do some test. =20 Best Regards Lubo =20 =20 -----Original Message----- From: Santhapur Naveen [mailto:naveens@amiindia.co.in]=20 Sent: Thursday, August 25, 2016 5:09 PM To: Zhang, Lubo Cc: edk2-devel@lists.01.org Subject: RE: [edk2] 2nd HttpBoot fails even after upgrading to revision 247= 29 Hi Lubo, I believe the way we refer the version numbers seem different. I use torto= ise SVN to get the details of the edk2. When I checked the revision numbers= 22104 and 21740 for HttpBootDxe and HttpDxe drivers respectively, I couldn= 't find anything. Please refer the attachments. By any means is it possible= for you to provide the way you refer the revision details? However, I've downloaded the latest source from the URL you have provided = and I'm trying to verify the same now. Will update you my result. Thank you, Naveen -----Original Message----- From: Zhang, Lubo [mailto:lubo.zhang@intel.com]=20 Sent: Thursday, August 25, 2016 7:19 AM To: Santhapur Naveen Cc: edk2-devel@lists.01.org Subject: RE: [edk2] 2nd HttpBoot fails even after upgrading to revision 247= 29 Hi Naveen I update the SVN and now the revision is 22467. The HttpBootDxe revision is 22104,patch is - Fix IPv6 HTTPClient ven= dor class data The HttpDxe revision is 21740, patch is - NetworkPkg: Fix typos in c= omments. But I suggest you to download the https://github.com/tianocore/edk2= , it is latest edk2 code. Best Regards Lubo -----Original Message----- From: Santhapur Naveen [mailto:naveens@amiindia.co.in]=20 Sent: Wednesday, August 24, 2016 7:10 PM To: Zhang, Lubo Cc: edk2-devel@lists.01.org Subject: RE: [edk2] 2nd HttpBoot fails even after upgrading to revision 247= 29 Hi Lubo, Can you please provide me any reference revision number to download and ch= eck the same? Thank you, Naveen -----Original Message----- From: Zhang, Lubo [mailto:lubo.zhang@intel.com]=20 Sent: Wednesday, August 24, 2016 4:23 PM To: Santhapur Naveen Cc: edk2-devel@lists.01.org Subject: RE: [edk2] 2nd HttpBoot fails even after upgrading to revision 247= 29 Hi Naveen I have checked the issue you mentioned. On my side, I updated the cod= e to latest and do some test on NT32 and Denlow platform, both the first ht= tp boot and 2nd are successful.=20 Until now , we have fixed few bugs which may lead the 2nd http boot f= ail. As for the patch you mentioned is one case to fix the bug, So I sugges= t you to update the code firstly and at least make sure the HttpDxe , HttpB= ootDxe and DxeHttpLib are latest.=20 =20 If any result, Pls let me know. Best Regards Lubo=20 =20 -----Original Message----- From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of Sant= hapur Naveen Sent: Wednesday, August 24, 2016 4:53 PM To: Zhang, Lubo Cc: edk2-devel@lists.01.org Subject: Re: [edk2] 2nd HttpBoot fails even after upgrading to revision 247= 29 Hi Lubo, Please find the following details that may help. NetworkPkg:Fix a bug the 2nd httpboot fail issue. Httpboot over Ipv4 or Ipv6 stack,for both Identity and chunked transfer mod= e,when the last data has been parsed by HttpLib, the HttpInstance->NextMsg = pointer should point a correct location.Now after the first successful htt= pboot for ipv4 or ipv6,the HttpInstance->NextMsgpoint the character after t= he last byte, it may be a bad buffer if we don't receive another HttpHeader= , so if call a 2nd httpboot, the wrong NextMsg pointer will cause the httpb= oot fail, so we need to check this case in HttpBodyParserCallback function = in the first http boot process. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Zhang Lubo Reviewed-by: Fu Siyuan Reviewed-by: Ye Ting Reviewed-by: Wu Jiaxin git-svn-id: https://svn.code.sf.net/p/edk2/code/trunk/edk2@19423 6f19259b-4= bc3-4df7-8a09-765794883524 Thanks, Naveen -----Original Message----- From: Zhang, Lubo [mailto:lubo.zhang@intel.com]=20 Sent: Wednesday, August 24, 2016 2:13 PM To: Santhapur Naveen Cc: edk2-devel@lists.01.org Subject: RE: [edk2] 2nd HttpBoot fails even after upgrading to revision 247= 29 Hi Naveen Could you list the patch name or Git SHA value which fixed the 2nd Htt= pBootfail issue. On my side, the SVN version 24729 is not exist. Thanks=20 Lubo=20 -----Original Message----- From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of Sant= hapur Naveen Sent: Wednesday, August 24, 2016 3:40 PM To: edk2-devel@lists.01.org Subject: Re: [edk2] 2nd HttpBoot fails even after upgrading to revision 247= 29 Hello, Revision 24729 of edk2 says 2nd HttpBoot fail issue has been fixe= d. So, I have taken the changes from the file NetworkPkg/HttpDxe/HttpImpl.c= of the revsision 24729 and still the 2nd HttpBoot fails. Please let me kn= ow if I have missed any. P.S: The first http boot happens successfully though. (downloading a Shell.= efi) Thanks, Naveen _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel