Subject: Re: CVS commit: src/sys
To: Tom Spindler <dogcow@NetBSD.org>
From: David Laight <david@l8s.co.uk>
List: source-changes
Date: 05/04/2007 07:56:58
On Fri, May 04, 2007 at 12:00:34AM +0000, Tom Spindler wrote:
> > Module Name: src
> > Committed By: dyoung
> > Date: Wed May 2 20:40:29 UTC 2007
> >
> > Modified Files:
> [...]
> > src/sys/netiso: clnp.h clnp_er.c clnp_frag.c clnp_output.c clnp_raw.c
> > clnp_subr.c eonvar.h if_eon.c iso.c iso.h iso_pcb.c iso_pcb.h
> > iso_proto.c iso_var.h tp_iso.c
>
> As noted on current-users, this inline code breaks ipf on 64-bit architectures,
> due to src/sys/dist/ipf/netinet/ip_fil.h:
> #define offsetof(t,m) (int)((&((t *)0L)->m))
>
> As also pointed out on current-users, either #include'ing <stddef.h>,
> fixing the #define, or making the sockaddr_iso_init1 sub (as defined
> in sys/netiso/iso.h) non-inlined would all probably suffice to make
> things work again.
How big are these inlined lumps of code?
If non-trivial they probably don't want inlining anyway.
David
--
David Laight: david@l8s.co.uk