Re: [PATCH nftables 5/9] src: add host byte order integer type

2017-02-07 Thread Pablo Neira Ayuso
On Tue, Feb 07, 2017 at 12:58:56PM +0100, Pablo Neira Ayuso wrote: > On Mon, Feb 06, 2017 at 11:33:01PM +0100, Florian Westphal wrote: > > Pablo Neira Ayuso wrote: > > > On Fri, Feb 03, 2017 at 01:35:52PM +0100, Florian Westphal wrote: > > > > diff --git a/include/datatype.h

Re: [PATCH nftables 5/9] src: add host byte order integer type

2017-02-07 Thread Pablo Neira Ayuso
On Mon, Feb 06, 2017 at 11:33:01PM +0100, Florian Westphal wrote: > Pablo Neira Ayuso wrote: > > On Fri, Feb 03, 2017 at 01:35:52PM +0100, Florian Westphal wrote: > > > diff --git a/include/datatype.h b/include/datatype.h > > > index 9f127f2954e3..8c1c827253be 100644 > > >

Re: [PATCH nftables 5/9] src: add host byte order integer type

2017-02-06 Thread Florian Westphal
Pablo Neira Ayuso wrote: > On Fri, Feb 03, 2017 at 01:35:52PM +0100, Florian Westphal wrote: > > diff --git a/include/datatype.h b/include/datatype.h > > index 9f127f2954e3..8c1c827253be 100644 > > --- a/include/datatype.h > > +++ b/include/datatype.h > > @@ -82,6 +82,7 @@

Re: [PATCH nftables 5/9] src: add host byte order integer type

2017-02-06 Thread Pablo Neira Ayuso
On Mon, Feb 06, 2017 at 06:31:10PM +0100, Pablo Neira Ayuso wrote: > On Fri, Feb 03, 2017 at 01:35:52PM +0100, Florian Westphal wrote: > > diff --git a/include/datatype.h b/include/datatype.h > > index 9f127f2954e3..8c1c827253be 100644 > > --- a/include/datatype.h > > +++ b/include/datatype.h > >

Re: [PATCH nftables 5/9] src: add host byte order integer type

2017-02-06 Thread Pablo Neira Ayuso
On Fri, Feb 03, 2017 at 01:35:52PM +0100, Florian Westphal wrote: > diff --git a/include/datatype.h b/include/datatype.h > index 9f127f2954e3..8c1c827253be 100644 > --- a/include/datatype.h > +++ b/include/datatype.h > @@ -82,6 +82,7 @@ enum datatypes { > TYPE_DSCP, > TYPE_ECN, >

[PATCH nftables 5/9] src: add host byte order integer type

2017-02-03 Thread Florian Westphal
This is needed once we add support to set a zone, as in ct zone set 42 Using integer_type makes nft use big-endian representation of the zone id instead of the required host byte order. When using 'ct zone 1', things will work because the (implicit) relational operation makes sure that the left