From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga14.intel.com (mga14.intel.com [192.55.52.115]) (using TLSv1 with cipher CAMELLIA256-SHA (256/256 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id EA78F1A1E21 for ; Mon, 29 Aug 2016 00:18:46 -0700 (PDT) Received: from orsmga002.jf.intel.com ([10.7.209.21]) by fmsmga103.fm.intel.com with ESMTP; 29 Aug 2016 00:18:46 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.28,595,1464678000"; d="scan'208";a="1042772207" Received: from fmsmsx105.amr.corp.intel.com ([10.18.124.203]) by orsmga002.jf.intel.com with ESMTP; 29 Aug 2016 00:18:46 -0700 Received: from fmsmsx119.amr.corp.intel.com (10.18.124.207) by FMSMSX105.amr.corp.intel.com (10.18.124.203) with Microsoft SMTP Server (TLS) id 14.3.248.2; Mon, 29 Aug 2016 00:18:45 -0700 Received: from shsmsx101.ccr.corp.intel.com (10.239.4.153) by FMSMSX119.amr.corp.intel.com (10.18.124.207) with Microsoft SMTP Server (TLS) id 14.3.248.2; Mon, 29 Aug 2016 00:18:45 -0700 Received: from shsmsx104.ccr.corp.intel.com ([169.254.5.116]) by SHSMSX101.ccr.corp.intel.com ([169.254.1.8]) with mapi id 14.03.0301.000; Mon, 29 Aug 2016 15:18:42 +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//dDBAANT6TwD//248EA== Date: Mon, 29 Aug 2016 07:18:42 +0000 Message-ID: <7619447B08B8F74DA4FF2A813B79803B37836806@SHSMSX104.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> <7619447B08B8F74DA4FF2A813B79803B01A53984@shsmsx102.ccr.corp.intel.com> <625A2455CC232F40B0F38F05ACED6D976C5448B0@VENUS1.in.megatrends.com> In-Reply-To: <625A2455CC232F40B0F38F05ACED6D976C5448B0@VENUS1.in.megatrends.com> Accept-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.239.127.40] MIME-Version: 1.0 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: Mon, 29 Aug 2016 07:18:47 -0000 Content-Language: en-US Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable What do you mean control goes to next boot option? Will it go to PXE boot = when you select the 2nd http boot. If was, you just need to update the whol= e edk2 code. This issue related to the BDS and we already fixed long before= , Best regards Lubo=20 -----Original Message----- From: Santhapur Naveen [mailto:naveens@amiindia.co.in]=20 Sent: Monday, August 29, 2016 2:27 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've taken the source from the location you have specified and checked. Bu= t I still get the same failure. When I select the same HTTP boot option the= 2nd time, control goes to next boot option available. I guess the problem = could be on my side only. I'll look into it. Thank you very much for your time and effort. Thanks, Naveen -----Original Message----- From: Zhang, Lubo [mailto:lubo.zhang@intel.com]=20 Sent: Thursday, August 25, 2016 3:19 PM To: Santhapur Naveen Cc: edk2-devel@lists.01.org Subject: RE: [edk2] 2nd HttpBoot fails even after upgrading to revision 247= 29 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