On Sun, Jun 14, 2015 at 08:43:51PM +0200, Felix Janda wrote:
> Pablo Neira Ayuso wrote:
> > On Sat, May 16, 2015 at 01:41:49PM +0200, Felix Janda wrote:
> > > Signed-off-by: Felix Janda <felix.janda@xxxxxxxxx>
> > > ---
> > > include/libnetfilter_log/linux_nfnetlink_log.h | 42 +++++++++++++-------------
> > > 1 file changed, 21 insertions(+), 21 deletions(-)
> > >
> > > diff --git a/include/libnetfilter_log/linux_nfnetlink_log.h b/include/libnetfilter_log/linux_nfnetlink_log.h
> > > index 9f38277..c647581 100644
> > > --- a/include/libnetfilter_log/linux_nfnetlink_log.h
> > > +++ b/include/libnetfilter_log/linux_nfnetlink_log.h
> > > @@ -20,31 +20,31 @@ enum nfulnl_msg_types {
> > > };
> > >
> > > struct nfulnl_msg_packet_hdr {
> > > - __be16 hw_protocol; /* hw protocol (network order) */
> > > - __u8 hook; /* netfilter hook */
> > > - __u8 _pad;
> > > + uint16_t hw_protocol; /* hw protocol (network order) */
> > > + uint8_t hook; /* netfilter hook */
> > > + uint8_t _pad;
> > > };
> >
> > I have skipped this patch in this series. It is mangling a cached
> > header from the kernel, I guess there must be a way to make musl happy
> > without this change given that we follow the same approach in other
> > libraries.
> >
> > Thanks.
>
> Yes, the conversion of types is not strictly necessary. As long as
> <linux/types.h> is included, everything is fine for musl. I have likely
> misunderstood your previous mail. Feel free to ignore patches mangling
> kernel headers after updating them.
This is what I did. I mangled the patches to skip the chunks that
modify the kernel headers that we cache in the trees.
> Thanks for your review and pushing of patches so far!
Thanks for confirming and for the rather large patchset to address the
musl problems.
--
To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
