Re: CVS commit: src/sys/netinet

To: source-changes-d%NetBSD.org@localhost, Martin Husemann <martin%duskware.de@localhost>
Subject: Re: CVS commit: src/sys/netinet
From: Roy Marples <roy%marples.name@localhost>
Date: Tue, 16 Feb 2021 08:26:40 +0000

On 16/02/2021 05:44, Martin Husemann wrote:

Module Name:	src
Committed By:	martin
Date:		Tue Feb 16 05:44:14 UTC 2021

Modified Files:
	src/sys/netinet: if_arp.c

Log Message:
Undo previous backout: alignment is needed here.
The reason for the previous backout was a misunderstanding (POINTER_ALIGNED_P
was broken, but the assertion fired even after it got fixed).


Is that because ARP_HDR_ALIGNMENT is forcing 4 byte alignment?

For the purposes of using just the header we define I'm pretty sure we can use 2byte alignment and set ARP_HDR_ALIGNMENT to 1.


ARP_HDR_ALIGNED_P can now be removed from if_arp.c as well.

Roy

Follow-Ups:
- Re: CVS commit: src/sys/netinet
  - From: Martin Husemann

Prev by Date: Re: CVS commit: src/sys
Next by Date: Re: CVS commit: src/sys/netinet
Previous by Thread: Re: CVS commit: src/lib/libc/locale
Next by Thread: Re: CVS commit: src/sys/netinet
Indexes:

Home | Main Index | Thread Index | Old Index