Messages in this thread | | | From | "Duyck, Alexander H" <> | Subject | RE: [PATCH] net: export netdev_txq_to_tc to allow sch_mqprio to compile as module | Date | Tue, 17 Oct 2017 14:31:59 +0000 |
| |
> -----Original Message----- > From: Eric Dumazet [mailto:edumazet@google.com] > Sent: Tuesday, October 17, 2017 4:21 AM > To: Henrik Austad <henrik@austad.us> > Cc: netdev <netdev@vger.kernel.org>; David S . Miller > <davem@davemloft.net>; Daniel Borkmann <daniel@iogearbox.net>; David > Ahern <dsahern@gmail.com>; Duyck, Alexander H > <alexander.h.duyck@intel.com>; Willem de Bruijn <willemb@google.com>; > John Fastabend <john.fastabend@gmail.com>; tcharding <me@tobin.cc>; LKML > <linux-kernel@vger.kernel.org>; Henrik Austad <haustad@cisco.com>; Sanchez- > Palencia, Jesus <jesus.sanchez-palencia@intel.com> > Subject: Re: [PATCH] net: export netdev_txq_to_tc to allow sch_mqprio to > compile as module > > On Tue, Oct 17, 2017 at 3:10 AM, Henrik Austad <henrik@austad.us> wrote: > > In commit 32302902ff09 ("mqprio: Reserve last 32 classid values for HW > > traffic classes and misc IDs") sch_mqprio started using > > netdev_txq_to_tc to find the correct tc instead of dev->tc_to_txq[] > > > > However, when mqprio is compiled as a module, it cannot resolve the > > symbol, leading to this error: > > > > ERROR: "netdev_txq_to_tc" [net/sched/sch_mqprio.ko] undefined! > > > > This adds an EXPORT_SYMBOL() since the other user in the kernel > > (netif_set_xps_queue) is also EXPORT_SYMBOL() (and not _GPL) or in a > > sysfs-callback. > > > > Cc: Alexander Duyck <alexander.h.duyck@intel.com> > > Cc: Jesus Sanchez-Palencia <jesus.sanchez-palencia@intel.com> > > Cc: David S. Miller <davem@davemloft.net> > > Signed-off-by: Henrik Austad <haustad@cisco.com> > > > Reviewed-by: Eric Dumazet <edumazet@google.com>
This is identical to a patch I submitted yesterday when I got the report from the kbuild robot. I would say your description looks much better than mine though so I would be good with dropping my patch in favor of this one.
Acked-by: Alexander Duyck <alexander.h.duyck@intel.com>
| |