RE: [PATCH] net: export netdev_txq_to_tc to allow sch_mqprio to compile as module

-----Original Message-----
From: Eric Dumazet [mailto:edumazet@google.com]
Sent: Tuesday, October 17, 2017 4:21 AM
To: Henrik Austad <redacted>
Cc: netdev <redacted>; David S . Miller
[off-list ref]; Daniel Borkmann [off-list ref]; David
Ahern [off-list ref]; Duyck, Alexander H
[off-list ref]; Willem de Bruijn [off-list ref];
John Fastabend [off-list ref]; tcharding [off-list ref]; LKML
[off-list ref]; Henrik Austad [off-list ref]; Sanchez-
Palencia, Jesus [off-list ref]
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 [off-list ref] wrote:

quoted

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 <redacted>
Cc: Jesus Sanchez-Palencia <redacted>
Cc: David S. Miller <davem@davemloft.net>
Signed-off-by: Henrik Austad <redacted>

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 <redacted>