> -----Original Message----- > From: Intel-wired-lan [mailto:intel-wired-lan-boun...@lists.osuosl.org] On > Behalf Of John Fastabend > Sent: Monday, February 29, 2016 11:26 AM > To: intel-wired-...@lists.osuosl.org; john.fastab...@gmail.com; Kirsher, > Jeffrey T > Cc: netdev@vger.kernel.org > Subject: [Intel-wired-lan] [net-next PATCH] net: relax setup_tc ndo op > handle restriction > > I added this check in setup_tc to multiple drivers, > > if (handle != TC_H_ROOT || tc->type != TC_SETUP_MQPRIO) > > Unfortunately restricting to TC_H_ROOT like this breaks the old > instantiation of mqprio to setup a hardware qdisc. This patch > relaxes the test to only check the type to make it equivalent > to the check before I broke it. With this the old instantiation > continues to work. > > A good smoke test is to setup mqprio with, > > # tc qdisc add dev eth4 root mqprio num_tc 8 \ > map 0 1 2 3 4 5 6 7 \ > queues 0@0 1@1 2@2 3@3 4@4 5@5 6@6 7@7 > > Fixes: e4c6734eaab9 ("net: rework ndo tc op to consume additional qdisc > handle paramete") > Reported-by: Singh Krishneil <krishneil.k.si...@intel.com> > Reported-by: Jake Keller <jacob.e.kel...@intel.com> > CC: Murali Karicheri <m-kariche...@ti.com> > CC: Shradha Shah <ss...@solarflare.com> > CC: Or Gerlitz <ogerl...@mellanox.com> > CC: Ariel Elior <ariel.el...@qlogic.com> > CC: Jeff Kirsher <jeffrey.t.kirs...@intel.com> > CC: Bruce Allan <bruce.w.al...@intel.com> > CC: Jesse Brandeburg <jesse.brandeb...@intel.com> > CC: Don Skidmore <donald.c.skidm...@intel.com> > Signed-off-by: John Fastabend <john.r.fastab...@intel.com> > --- > drivers/net/ethernet/amd/xgbe/xgbe-drv.c | 2 +- > drivers/net/ethernet/broadcom/bnx2x/bnx2x_cmn.c | 2 +- > drivers/net/ethernet/broadcom/bnxt/bnxt.c | 2 +- > drivers/net/ethernet/intel/fm10k/fm10k_netdev.c | 2 +- > drivers/net/ethernet/intel/ixgbe/ixgbe_main.c | 2 +- > drivers/net/ethernet/mellanox/mlx4/en_netdev.c | 2 +- > drivers/net/ethernet/sfc/tx.c | 2 +- > drivers/net/ethernet/ti/netcp_core.c | 2 +- > 8 files changed, 8 insertions(+), 8 deletions(-)
Jeff, please apply this to your next-queue tree/dev-queue branch ASAP as it is blocking the testing of another patch already in that queue. Thanks, Bruce.