[ovs-dev] [PATCH 1/2] datapath: Compatibility code for __packed macro.
Ben Pfaff
blp at nicira.com
Mon Dec 13 18:16:14 UTC 2010
On Mon, Dec 13, 2010 at 10:13:17AM -0800, Jesse Gross wrote:
> On Mon, Dec 13, 2010 at 9:22 AM, Ben Pfaff <blp at nicira.com> wrote:
> > On Sun, Dec 12, 2010 at 12:49:39PM -0800, Jesse Gross wrote:
> >> The __packed macro for structure packing wasn't introduced until 2.6.24,
> >> so define it ourselves.
> >>
> >> Signed-off-by: Jesse Gross <jesse at nicira.com>
> >
> > Acked-by: Ben Pfaff <blp at nicira.com>
> >
> > Nits:
> >
> > Is the #ifndef necessary? Won't the #include_next take care of that anyway?
>
> The #include_next will take care of the #ifndef on current kernels.
> Older ones have a comment to that effect but no check.
>
> >
> > I always add include guards habitually, even when they are not
> > necessary. I don't care that much, though.
>
> The original Linux source file doesn't have any guards - just a check
> that compiler.h was included already, which acts as its guard. I kept
> it the same for consistency.
OK, that's all good reasoning, thanks.
More information about the dev
mailing list