From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from list by lists.gnu.org with archive (Exim 4.71) id 1dUs8H-00006T-4I for mharc-grub-devel@gnu.org; Tue, 11 Jul 2017 06:11:05 -0400 Received: from eggs.gnu.org ([2001:4830:134:3::10]:54423) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1dUs8E-00006B-Hj for grub-devel@gnu.org; Tue, 11 Jul 2017 06:11:03 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1dUs8A-0007Ur-Gq for grub-devel@gnu.org; Tue, 11 Jul 2017 06:11:02 -0400 Received: from smtp.nue.novell.com ([195.135.221.5]:44335) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1dUs8A-0007Uc-64 for grub-devel@gnu.org; Tue, 11 Jul 2017 06:10:58 -0400 Received: from emea4-mta.ukb.novell.com ([10.120.13.87]) by smtp.nue.novell.com with ESMTP (TLS encrypted); Tue, 11 Jul 2017 12:10:56 +0200 Received: from linux-9gqx.suse.de (nwb-a10-snat.microfocus.com [10.120.13.202]) by emea4-mta.ukb.novell.com with ESMTP (TLS encrypted); Tue, 11 Jul 2017 10:49:04 +0100 Date: Tue, 11 Jul 2017 17:48:56 +0800 From: Michael Chang To: The development of GNU GRUB Cc: "Knippers, Linda" , "arvidjaar@gmail.com" , "Chang, Clay (HPS OE-Linux TDC)" , "Ruan, Michael (HPS OE-Linux/VMware TDC)" , "phcoder@gmail.com" Subject: Re: [PATCH v2 0/9] Add UEFI HTTP Boot support for IPv4 and IPv6 Message-ID: <20170711094856.GF26571@linux-9gqx.suse.de> Mail-Followup-To: The development of GNU GRUB , "Knippers, Linda" , "arvidjaar@gmail.com" , "Chang, Clay (HPS OE-Linux TDC)" , "Ruan, Michael (HPS OE-Linux/VMware TDC)" , "phcoder@gmail.com" References: <1482483252-8710-1-git-send-email-kengyu@hpe.com> <20170116101219.GO23864@olila.local.net-space.pl> <20170710101034.GE26571@linux-9gqx.suse.de> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.24 (2015-08-30) X-detected-operating-system: by eggs.gnu.org: GNU/Linux 3.x [fuzzy] X-Received-From: 195.135.221.5 X-BeenThere: grub-devel@gnu.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: The development of GNU GRUB List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 11 Jul 2017 10:11:03 -0000 On Tue, Jul 11, 2017 at 07:39:16AM +0000, Lin, Keng-Yu wrote: > > How many test case did you perform on this patch. More specifically, did you > > verify how does it work with existing network commands > > > > https://www.gnu.org/software/grub/manual/grub.html#Networking- > > commands > > > > And also environment variables related to network > > > > https://www.gnu.org/software/grub/manual/grub.html#Network > > > > I think the patch is good at running grub booted directly by UEFI http load file > > protocol, assuming the UEFI Handle has configured network protocol down > > the road before handing over to grub. But that may not satisfied the user as > > they get used to have more than that already. > > > > Practically grub can work out more (sophisticated) scenario by exercising the > > network commands to configure the network interface one way or the other, > > and obviously the patch did not route the same commands to UEFI network > > stack, which is regarded as functional regression as it did not apply on UEFI > > protocol. > > Hi Michael: > Thanks for the comments. > Are you saying that for this patchset, the following environment variables should be supported in the UEFI HTTP Boot mode too? Yes. And it should not be limited to http but also PXE mode of the UEFI. Afterall, the UEFI should be acting like alternative module of networking at user's disposal and should not notice any difference. Thanks, Michael > > * 'net__ip' > *'net__mac' > * 'net__hostname' > * 'net__domain' > * 'net__rootpath' > * 'net__extensionspath' > * 'net__boot_file' > * 'net__dhcp_server_name' > * 'net__next_server' > * 'net_default_interface' > * 'net_default_ip' > * 'net_default_mac' > * 'net_default_server' > > And the following commands: > > * net_add_addr: Add a network address > * net_add_dns: Add a DNS server > * net_add_route: Add routing entry > * net_bootp: Perform a bootp autoconfiguration > * net_del_addr: Remove IP address from interface > * net_del_dns: Remove a DNS server > * net_del_route: Remove a route entry > * net_get_dhcp_option: Retrieve DHCP options > * net_ipv6_autoconf: Perform IPv6 autoconfiguration > * net_ls_addr: List interfaces > * net_ls_cards: List network cards > * net_ls_dns: List DNS servers > * net_ls_routes: List routing entries > * net_nslookup: Perform a DNS lookup > > _______________________________________________ > Grub-devel mailing list > Grub-devel@gnu.org > https://lists.gnu.org/mailman/listinfo/grub-devel