[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v3 2/5] net/dhcp: Consistently use hexadecimal numbers for op
From: |
Daniel Kiper |
Subject: |
Re: [PATCH v3 2/5] net/dhcp: Consistently use hexadecimal numbers for options enum |
Date: |
Tue, 22 Oct 2019 16:16:11 +0200 |
User-agent: |
NeoMutt/20170113 (1.7.2) |
On Tue, Oct 22, 2019 at 10:48:11AM +0200, Javier Martinez Canillas wrote:
> The bootp/dhcp options enum values are a mixture of hexadecimal and decimal
> numbers. Change this to consistently use hexadecimal numbers for all values.
>
> Suggested-by: Daniel Kiper <address@hidden>
> Signed-off-by: Javier Martinez Canillas <address@hidden>
>
> ---
>
> Changes in v3:
> - Add patch to cleanup bootp/dhcp options enum.
>
> Changes in v2: None
>
> include/grub/net.h | 14 +++++++-------
> 1 file changed, 7 insertions(+), 7 deletions(-)
>
> diff --git a/include/grub/net.h b/include/grub/net.h
> index 4a9069a1474..a6982990ab0 100644
> --- a/include/grub/net.h
> +++ b/include/grub/net.h
> @@ -462,13 +462,13 @@ enum
> GRUB_NET_BOOTP_DOMAIN = 0x0f,
> GRUB_NET_BOOTP_ROOT_PATH = 0x11,
> GRUB_NET_BOOTP_EXTENSIONS_PATH = 0x12,
> - GRUB_NET_DHCP_REQUESTED_IP_ADDRESS = 50,
> - GRUB_NET_DHCP_OVERLOAD = 52,
> - GRUB_NET_DHCP_MESSAGE_TYPE = 53,
> - GRUB_NET_DHCP_SERVER_IDENTIFIER = 54,
> - GRUB_NET_DHCP_PARAMETER_REQUEST_LIST = 55,
> - GRUB_NET_DHCP_TFTP_SERVER_NAME = 66,
> - GRUB_NET_DHCP_BOOTFILE_NAME = 67,
> + GRUB_NET_DHCP_REQUESTED_IP_ADDRESS = 0x32,
> + GRUB_NET_DHCP_OVERLOAD = 0x34,
> + GRUB_NET_DHCP_MESSAGE_TYPE = 0x35,
> + GRUB_NET_DHCP_SERVER_IDENTIFIER = 0x36,
> + GRUB_NET_DHCP_PARAMETER_REQUEST_LIST = 0x37,
> + GRUB_NET_DHCP_TFTP_SERVER_NAME = 0x42,
> + GRUB_NET_DHCP_BOOTFILE_NAME = 0x43,
Ugh... Checked RFC 2132. And it uses decimals instead of hexes.
So, could you convert all of them to decimals? If you do that
feel free to add my RB here. Sorry for the confusion. I should
consult the RFC earlier.
Daniel
- [PATCH v3 0/5] Search for specific config files using UUID, MAC and IP, Javier Martinez Canillas, 2019/10/22
- [PATCH v3 1/5] Add %X option to printf functions, Javier Martinez Canillas, 2019/10/22
- [PATCH v3 2/5] net/dhcp: Consistently use hexadecimal numbers for options enum, Javier Martinez Canillas, 2019/10/22
- Re: [PATCH v3 2/5] net/dhcp: Consistently use hexadecimal numbers for options enum,
Daniel Kiper <=
- [PATCH v3 3/5] net/dhcp: Set net_<interface>_client{id, uuid} variables from DHCP options, Javier Martinez Canillas, 2019/10/22
- [PATCH v3 4/5] normal/main: Search for specific config files for netboot, Javier Martinez Canillas, 2019/10/22
- [PATCH v3 5/5] docs: Update info with grub.cfg netboot selection order, Javier Martinez Canillas, 2019/10/22