[PATCH v4 1/3] net: ipv6: Add support for default gateway discovery.
Sergei Antonov
saproj at gmail.com
Thu May 4 16:30:15 CEST 2023
On Sat, 22 Apr 2023 at 03:08, <emohandesi at linux.microsoft.com> wrote:
>
> From: Ehsan Mohandesi <emohandesi at linux.microsoft.com>
Hello, Ehsan! Good patch, but one little change is needed.
> +struct icmp6_ra_prefix_info {
> + u8 type; /* Type is 3 for Prefix Information. */
> + u8 len; /* Len is 4 for Prefix Information. */
> + /* The number of leading bits in the Prefix that are valid. */
> + u8 prefix_len;
> + u8 reserved1:6, /* MUST be ignored by the receiver. */
> + aac:1, /* autonomous address-configuration flag */
> + /* Indicates that this prefix can be used for on-link determination. */
> + on_link:1;
> + /*
> + * The length of time in seconds that the prefix is valid for the
> + * purpose of on-link determination.
> + */
> + __be32 valid_lifetime;
> + /* The length of time addresses remain preferred. */
> + __be32 preferred_lifetime;
> + __be32 reserved2; /* MUST be ignored by the receiver. */
> + /*
> + * Prefix is an IP address or a prefix of an IP address. The Prefix
> + * Length field contains the number of valid leading bits in the prefix.
> + * The bits in the prefix after the prefix length are reserved and MUST
> + * be initialized to zero by the sender and ignored by the receiver.
> + */
> + struct in6_addr prefix;
> +};
Here it should end with:
} __packed;
Because this structure may be placed at a badly aligned offset within
a packet. For example, at offset 0x46 within Ethernet packet.
Other than that:
Tested-by: Sergei Antonov <saproj at gmail.com>
Reviewed-by: Sergei Antonov <saproj at gmail.com>
More information about the U-Boot
mailing list