Message ID | 20170619213323.14220-2-joe@ovn.org |
---|---|
State | Accepted |
Headers | show |
On 06/19/2017 02:33 PM, Joe Stringer wrote: > Fix minor style variations and unnecessary includes. > > Signed-off-by: Joe Stringer <joe@ovn.org> > --- > lib/netdev-linux.c | 1 + > lib/netdev-tc-offloads.c | 11 ++++++----- > lib/tc.c | 6 ++++-- > lib/tc.h | 4 ++-- > 4 files changed, 13 insertions(+), 9 deletions(-) > > diff --git a/lib/netdev-linux.c b/lib/netdev-linux.c > index f5dc30fbc188..6978c44c796a 100644 > --- a/lib/netdev-linux.c > +++ b/lib/netdev-linux.c > @@ -29,6 +29,7 @@ > #include <linux/types.h> > #include <linux/ethtool.h> > #include <linux/mii.h> > +#include <linux/rtnetlink.h> > #include <linux/sockios.h> > #include <sys/types.h> > #include <sys/ioctl.h> > diff --git a/lib/netdev-tc-offloads.c b/lib/netdev-tc-offloads.c > index 940e20ff2759..621402303c8d 100644 > --- a/lib/netdev-tc-offloads.c > +++ b/lib/netdev-tc-offloads.c > @@ -16,24 +16,25 @@ > > #include <config.h> > #include "netdev-tc-offloads.h" > + > #include <errno.h> > #include <linux/if_ether.h> > + > +#include "dpif.h" > +#include "hash.h" > #include "openvswitch/hmap.h" > #include "openvswitch/match.h" > #include "openvswitch/ofpbuf.h" > #include "openvswitch/thread.h" > #include "openvswitch/types.h" > #include "openvswitch/vlog.h" > -#include "netdev-provider.h" > +#include "netdev-linux.h" > #include "netlink.h" > #include "netlink-socket.h" > #include "odp-netlink.h" > +#include "tc.h" > #include "unaligned.h" > #include "util.h" > -#include "hash.h" > -#include "dpif.h" > -#include "tc.h" > -#include "netdev-linux.h" > > VLOG_DEFINE_THIS_MODULE(netdev_tc_offloads); > > diff --git a/lib/tc.c b/lib/tc.c > index 9ca7b76ce27f..401690ec98c7 100644 > --- a/lib/tc.c > +++ b/lib/tc.c > @@ -1,5 +1,5 @@ > /* > - * Copyright (c) 2009, 2010, 2011, 2012, 2013, 2014, 2015, 2016, 2017 Nicira, Inc. > + * Copyright (c) 2009-2017 Nicira, Inc. > * Copyright (c) 2016 Mellanox Technologies, Ltd. > * > * Licensed under the Apache License, Version 2.0 (the "License"); > @@ -17,6 +17,7 @@ > > #include <config.h> > #include "tc.h" > + > #include <errno.h> > #include <linux/if_ether.h> > #include <linux/rtnetlink.h> > @@ -26,6 +27,7 @@ > #include <linux/tc_act/tc_vlan.h> > #include <linux/gen_stats.h> > #include <net/if.h> > + > #include "byte-order.h" > #include "netlink-socket.h" > #include "netlink.h" > @@ -33,7 +35,7 @@ > #include "openvswitch/vlog.h" > #include "packets.h" > #include "timeval.h" > -#include <unaligned.h> > +#include "unaligned.h" > > VLOG_DEFINE_THIS_MODULE(tc); > > diff --git a/lib/tc.h b/lib/tc.h > index 61188dd43ec3..1cc7362d59ae 100644 > --- a/lib/tc.h > +++ b/lib/tc.h > @@ -21,8 +21,8 @@ > #include <netinet/in.h> /* Must happen before linux/pkt_cls.h - Glibc #20215 */ > #include <linux/pkt_cls.h> > #include <linux/pkt_sched.h> > -#include <linux/rtnetlink.h> > -#include "lib/netlink-socket.h" > + > +#include "netlink-socket.h" > #include "odp-netlink.h" > #include "openvswitch/ofpbuf.h" > > Both patches in the series compile tested with glibc 2.17 and kernel 4.9. LGTM Tested-by: Greg Rose <gvrose8192@gmail.com> Acked-by: Greg Rose <gvrose8192@gmail.com>
On 19 June 2017 at 15:05, Greg Rose <gvrose8192@gmail.com> wrote: > On 06/19/2017 02:33 PM, Joe Stringer wrote: >> >> Fix minor style variations and unnecessary includes. >> >> Signed-off-by: Joe Stringer <joe@ovn.org> >> --- >> lib/netdev-linux.c | 1 + >> lib/netdev-tc-offloads.c | 11 ++++++----- >> lib/tc.c | 6 ++++-- >> lib/tc.h | 4 ++-- >> 4 files changed, 13 insertions(+), 9 deletions(-) >> >> diff --git a/lib/netdev-linux.c b/lib/netdev-linux.c >> index f5dc30fbc188..6978c44c796a 100644 >> --- a/lib/netdev-linux.c >> +++ b/lib/netdev-linux.c >> @@ -29,6 +29,7 @@ >> #include <linux/types.h> >> #include <linux/ethtool.h> >> #include <linux/mii.h> >> +#include <linux/rtnetlink.h> >> #include <linux/sockios.h> >> #include <sys/types.h> >> #include <sys/ioctl.h> >> diff --git a/lib/netdev-tc-offloads.c b/lib/netdev-tc-offloads.c >> index 940e20ff2759..621402303c8d 100644 >> --- a/lib/netdev-tc-offloads.c >> +++ b/lib/netdev-tc-offloads.c >> @@ -16,24 +16,25 @@ >> >> #include <config.h> >> #include "netdev-tc-offloads.h" >> + >> #include <errno.h> >> #include <linux/if_ether.h> >> + >> +#include "dpif.h" >> +#include "hash.h" >> #include "openvswitch/hmap.h" >> #include "openvswitch/match.h" >> #include "openvswitch/ofpbuf.h" >> #include "openvswitch/thread.h" >> #include "openvswitch/types.h" >> #include "openvswitch/vlog.h" >> -#include "netdev-provider.h" >> +#include "netdev-linux.h" >> #include "netlink.h" >> #include "netlink-socket.h" >> #include "odp-netlink.h" >> +#include "tc.h" >> #include "unaligned.h" >> #include "util.h" >> -#include "hash.h" >> -#include "dpif.h" >> -#include "tc.h" >> -#include "netdev-linux.h" >> >> VLOG_DEFINE_THIS_MODULE(netdev_tc_offloads); >> >> diff --git a/lib/tc.c b/lib/tc.c >> index 9ca7b76ce27f..401690ec98c7 100644 >> --- a/lib/tc.c >> +++ b/lib/tc.c >> @@ -1,5 +1,5 @@ >> /* >> - * Copyright (c) 2009, 2010, 2011, 2012, 2013, 2014, 2015, 2016, 2017 >> Nicira, Inc. >> + * Copyright (c) 2009-2017 Nicira, Inc. >> * Copyright (c) 2016 Mellanox Technologies, Ltd. >> * >> * Licensed under the Apache License, Version 2.0 (the "License"); >> @@ -17,6 +17,7 @@ >> >> #include <config.h> >> #include "tc.h" >> + >> #include <errno.h> >> #include <linux/if_ether.h> >> #include <linux/rtnetlink.h> >> @@ -26,6 +27,7 @@ >> #include <linux/tc_act/tc_vlan.h> >> #include <linux/gen_stats.h> >> #include <net/if.h> >> + >> #include "byte-order.h" >> #include "netlink-socket.h" >> #include "netlink.h" >> @@ -33,7 +35,7 @@ >> #include "openvswitch/vlog.h" >> #include "packets.h" >> #include "timeval.h" >> -#include <unaligned.h> >> +#include "unaligned.h" >> >> VLOG_DEFINE_THIS_MODULE(tc); >> >> diff --git a/lib/tc.h b/lib/tc.h >> index 61188dd43ec3..1cc7362d59ae 100644 >> --- a/lib/tc.h >> +++ b/lib/tc.h >> @@ -21,8 +21,8 @@ >> #include <netinet/in.h> /* Must happen before linux/pkt_cls.h - Glibc >> #20215 */ >> #include <linux/pkt_cls.h> >> #include <linux/pkt_sched.h> >> -#include <linux/rtnetlink.h> >> -#include "lib/netlink-socket.h" >> + >> +#include "netlink-socket.h" >> #include "odp-netlink.h" >> #include "openvswitch/ofpbuf.h" >> >> > Both patches in the series compile tested with glibc 2.17 and kernel 4.9. > > LGTM > > Tested-by: Greg Rose <gvrose8192@gmail.com> > Acked-by: Greg Rose <gvrose8192@gmail.com> Thanks, I applied the series to master.
diff --git a/lib/netdev-linux.c b/lib/netdev-linux.c index f5dc30fbc188..6978c44c796a 100644 --- a/lib/netdev-linux.c +++ b/lib/netdev-linux.c @@ -29,6 +29,7 @@ #include <linux/types.h> #include <linux/ethtool.h> #include <linux/mii.h> +#include <linux/rtnetlink.h> #include <linux/sockios.h> #include <sys/types.h> #include <sys/ioctl.h> diff --git a/lib/netdev-tc-offloads.c b/lib/netdev-tc-offloads.c index 940e20ff2759..621402303c8d 100644 --- a/lib/netdev-tc-offloads.c +++ b/lib/netdev-tc-offloads.c @@ -16,24 +16,25 @@ #include <config.h> #include "netdev-tc-offloads.h" + #include <errno.h> #include <linux/if_ether.h> + +#include "dpif.h" +#include "hash.h" #include "openvswitch/hmap.h" #include "openvswitch/match.h" #include "openvswitch/ofpbuf.h" #include "openvswitch/thread.h" #include "openvswitch/types.h" #include "openvswitch/vlog.h" -#include "netdev-provider.h" +#include "netdev-linux.h" #include "netlink.h" #include "netlink-socket.h" #include "odp-netlink.h" +#include "tc.h" #include "unaligned.h" #include "util.h" -#include "hash.h" -#include "dpif.h" -#include "tc.h" -#include "netdev-linux.h" VLOG_DEFINE_THIS_MODULE(netdev_tc_offloads); diff --git a/lib/tc.c b/lib/tc.c index 9ca7b76ce27f..401690ec98c7 100644 --- a/lib/tc.c +++ b/lib/tc.c @@ -1,5 +1,5 @@ /* - * Copyright (c) 2009, 2010, 2011, 2012, 2013, 2014, 2015, 2016, 2017 Nicira, Inc. + * Copyright (c) 2009-2017 Nicira, Inc. * Copyright (c) 2016 Mellanox Technologies, Ltd. * * Licensed under the Apache License, Version 2.0 (the "License"); @@ -17,6 +17,7 @@ #include <config.h> #include "tc.h" + #include <errno.h> #include <linux/if_ether.h> #include <linux/rtnetlink.h> @@ -26,6 +27,7 @@ #include <linux/tc_act/tc_vlan.h> #include <linux/gen_stats.h> #include <net/if.h> + #include "byte-order.h" #include "netlink-socket.h" #include "netlink.h" @@ -33,7 +35,7 @@ #include "openvswitch/vlog.h" #include "packets.h" #include "timeval.h" -#include <unaligned.h> +#include "unaligned.h" VLOG_DEFINE_THIS_MODULE(tc); diff --git a/lib/tc.h b/lib/tc.h index 61188dd43ec3..1cc7362d59ae 100644 --- a/lib/tc.h +++ b/lib/tc.h @@ -21,8 +21,8 @@ #include <netinet/in.h> /* Must happen before linux/pkt_cls.h - Glibc #20215 */ #include <linux/pkt_cls.h> #include <linux/pkt_sched.h> -#include <linux/rtnetlink.h> -#include "lib/netlink-socket.h" + +#include "netlink-socket.h" #include "odp-netlink.h" #include "openvswitch/ofpbuf.h"
Fix minor style variations and unnecessary includes. Signed-off-by: Joe Stringer <joe@ovn.org> --- lib/netdev-linux.c | 1 + lib/netdev-tc-offloads.c | 11 ++++++----- lib/tc.c | 6 ++++-- lib/tc.h | 4 ++-- 4 files changed, 13 insertions(+), 9 deletions(-)