Re: [PATCH] tpm: Pass unknown error as non-fatal, but debug print the error we got

2019-10-29 Thread Max Tottenham via Grub-devel
On 10/25, Javier Martinez Canillas wrote: > Hello Mathieu, > > On 10/25/19 4:48 PM, Mathieu Trudel-Lapierre wrote: > > On Fri, Oct 25, 2019 at 10:28 AM Mathieu Trudel-Lapierre > > wrote: > >> > >> Signed-off-by: Mathieu Trudel-Lapierre > >> > >> Patch-Name: ubuntu-tpm-unknown-error-non-fatal.pa

Re: [PATCH] tpm: Pass unknown error as non-fatal, but debug print the error we got

2019-10-29 Thread Javier Martinez Canillas
Hello Max, On 10/29/19 11:49 AM, Max Tottenham via Grub-devel wrote: > On 10/25, Javier Martinez Canillas wrote: [snip] >>> >> >> I think that we should go even further and make all the TPM measurement >> errors to be non-fatal. For example something like the following patch [0]. >> > > This po

Re: [PATCH v2 2/2] Make editenv chase symlinks including those across devices

2019-10-29 Thread Javier Martinez Canillas
Hello Daniel, On 10/28/19 5:12 PM, Daniel Kiper wrote: > On Mon, Oct 28, 2019 at 02:11:10PM +0100, Javier Martinez Canillas wrote: >> From: Peter Jones [snip] >> grub_util_create_envblk_file (const char *name) >> { >> + int rc; >>FILE *fp; >>char *buf; >>char *namenew; >> + ssiz

[PATCH v5 1/5] Add %X option to printf functions

2019-10-29 Thread Javier Martinez Canillas
From: Paulo Flabiano Smorigo The printf(3) function has support for the %X format specifier, to output an unsigned hexadecimal integer in uppercase. This can be achived in GRUB using the %x format specifier in grub_printf() and calling grub_toupper(), but it is more convenient if there is suppor

[PATCH v5 0/5] Search for specific config files using UUID, MAC and IP

2019-10-29 Thread Javier Martinez Canillas
Hello, This is a v5 of a series that have been carried in Fedora (and other distros) for a long time. It allows to fetch GRUB configurations that are customized to each machine by using the client UUID, MAC and IP as grub.cfg suffixes. The procedure is similar to what is used by pxelinux and yabo

[PATCH v5 2/5] net/dhcp: Consistently use decimal numbers for DHCP/BOOTP options enum

2019-10-29 Thread Javier Martinez Canillas
The DHCP Options and BOOTP Vendor Extensions enum values are a mixture of decimal and hexadecimal numbers. Change this to consistently use decimal numbers for all since that is how these values are defined by RFC 2132. Suggested-by: Daniel Kiper Signed-off-by: Javier Martinez Canillas Reviewed-b

[PATCH v5 5/5] docs: Update info with grub.cfg netboot selection order

2019-10-29 Thread Javier Martinez Canillas
From: Robert Marshall Add documentation to the grub info page that specifies the order netboot clients use to select a grub configuration file. Signed-off-by: Robert Marshall Signed-off-by: Javier Martinez Canillas --- Changes in v5: - Add Reviewed-by tag from Daniel Kiper to patch #5. Chan

[PATCH v5 4/5] normal/main: Search for specific config files for netboot

2019-10-29 Thread Javier Martinez Canillas
From: Paulo Flabiano Smorigo This patch implements a search for a specific configuration when the config file is on a remoteserver. It uses the following order: 1) DHCP client UUID option. 2) MAC address (in lower case hexadecimal with dash separators); 3) IP (in upper case hexadecimal)

[PATCH v5 3/5] net/dhcp: Set net__client{id, uuid} variables from DHCP options

2019-10-29 Thread Javier Martinez Canillas
From: Paulo Flabiano Smorigo This patch sets a net__clientid and net__clientuuid GRUB environment variables, using the DHCP client ID and UUID options if these are found. In the same way than net__ variables are set for other options such domain name, boot file, next server, etc. Signed-off-by:

Re: [PATCH v5 4/5] normal/main: Search for specific config files for netboot

2019-10-29 Thread Vladimir 'phcoder' Serbinenko
What is the problem with main grub.cfg just including the files based on environment variables? On Tue, 29 Oct 2019, 15:16 Javier Martinez Canillas, wrote: > From: Paulo Flabiano Smorigo > > This patch implements a search for a specific configuration when the config > file is on a remoteserver.