Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 471abeab authored by David S. Miller's avatar David S. Miller
Browse files

Merge branch 'net-sched-convert-cls-ndo_setup_tc-offload-calls-to-per-block-callbacks' 



Jiri Pirko says:

====================
net: sched: convert cls ndo_setup_tc offload calls to per-block callbacks

This patchset is a bit bigger, but most of the patches are doing the
same changes in multiple classifiers and drivers. I could do some
squashes, but I think it is better split.

This is another dependency on the way to shared block implementation.
The goal is to remove use of tp->q in classifiers code.

Also, this provides drivers possibility to track binding of blocks to
qdiscs. Legacy drivers which do not support shared block offloading.
register one callback per binding. That maintains the current
functionality we have with ndo_setup_tc. Drivers which support block
sharing offload register one callback per block which safes overhead.

Patches 1-4 introduce the binding notifications and per-block callbacks
Patches 5-8 add block callbacks calls to classifiers
Patches 9-17 do convert from ndo_setup_tc calls to block callbacks for
             classifier offloads in drivers
Patches 18-20 do cleanup

v1->v2:
- patch1:
  - move new enum value to the end
====================

Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parents b65f164d fa71212e

drivers/net/ethernet/broadcom/bnxt/bnxt.c

+31 −6
Original line number Diff line number Diff line
@@ -7295,23 +7295,48 @@ int bnxt_setup_mq_tc(struct net_device *dev, u8 tc) 
	return 0;

}



static int bnxt_setup_flower(struct net_device *dev,

			     struct tc_cls_flower_offload *cls_flower)

static int bnxt_setup_tc_block_cb(enum tc_setup_type type, void *type_data,

				  void *cb_priv)

{

	struct bnxt *bp = netdev_priv(dev);

	struct bnxt *bp = cb_priv;



	if (BNXT_VF(bp))

		return -EOPNOTSUPP;



	return bnxt_tc_setup_flower(bp, bp->pf.fw_fid, cls_flower);

	switch (type) {

	case TC_SETUP_CLSFLOWER:

		return bnxt_tc_setup_flower(bp, bp->pf.fw_fid, type_data);

	default:

		return -EOPNOTSUPP;

	}

}



static int bnxt_setup_tc_block(struct net_device *dev,

			       struct tc_block_offload *f)

{

	struct bnxt *bp = netdev_priv(dev);



	if (f->binder_type != TCF_BLOCK_BINDER_TYPE_CLSACT_INGRESS)

		return -EOPNOTSUPP;



	switch (f->command) {

	case TC_BLOCK_BIND:

		return tcf_block_cb_register(f->block, bnxt_setup_tc_block_cb,

					     bp, bp);

	case TC_BLOCK_UNBIND:

		tcf_block_cb_unregister(f->block, bnxt_setup_tc_block_cb, bp);

		return 0;

	default:

		return -EOPNOTSUPP;

	}

}



static int bnxt_setup_tc(struct net_device *dev, enum tc_setup_type type,

			 void *type_data)

{

	switch (type) {

	case TC_SETUP_CLSFLOWER:

		return bnxt_setup_flower(dev, type_data);

	case TC_SETUP_BLOCK:

		return bnxt_setup_tc_block(dev, type_data);

	case TC_SETUP_MQPRIO: {

		struct tc_mqprio_qopt *mqprio = type_data;

drivers/net/ethernet/broadcom/bnxt/bnxt_tc.c

+1 −2
Original line number Diff line number Diff line
@@ -748,8 +748,7 @@ int bnxt_tc_setup_flower(struct bnxt *bp, u16 src_fid, 
{

	int rc = 0;



	if (!is_classid_clsact_ingress(cls_flower->common.classid) ||

	    cls_flower->common.chain_index)

	if (cls_flower->common.chain_index)

		return -EOPNOTSUPP;



	switch (cls_flower->command) {

drivers/net/ethernet/broadcom/bnxt/bnxt_vfr.c

+38 −3
Original line number Diff line number Diff line
@@ -115,10 +115,11 @@ bnxt_vf_rep_get_stats64(struct net_device *dev, 
	stats->tx_bytes = vf_rep->tx_stats.bytes;

}



static int bnxt_vf_rep_setup_tc(struct net_device *dev, enum tc_setup_type type,

				void *type_data)

static int bnxt_vf_rep_setup_tc_block_cb(enum tc_setup_type type,

					 void *type_data,

					 void *cb_priv)

{

	struct bnxt_vf_rep *vf_rep = netdev_priv(dev);

	struct bnxt_vf_rep *vf_rep = cb_priv;

	struct bnxt *bp = vf_rep->bp;

	int vf_fid = bp->pf.vf[vf_rep->vf_idx].fw_fid;
@@ -130,6 +131,40 @@ static int bnxt_vf_rep_setup_tc(struct net_device *dev, enum tc_setup_type type, 
	}

}



static int bnxt_vf_rep_setup_tc_block(struct net_device *dev,

				      struct tc_block_offload *f)

{

	struct bnxt_vf_rep *vf_rep = netdev_priv(dev);



	if (f->binder_type != TCF_BLOCK_BINDER_TYPE_CLSACT_INGRESS)

		return -EOPNOTSUPP;



	switch (f->command) {

	case TC_BLOCK_BIND:

		return tcf_block_cb_register(f->block,

					     bnxt_vf_rep_setup_tc_block_cb,

					     vf_rep, vf_rep);

		return 0;

	case TC_BLOCK_UNBIND:

		tcf_block_cb_unregister(f->block,

					bnxt_vf_rep_setup_tc_block_cb, vf_rep);

		return 0;

	default:

		return -EOPNOTSUPP;

	}

}



static int bnxt_vf_rep_setup_tc(struct net_device *dev, enum tc_setup_type type,

				void *type_data)

{

	switch (type) {

	case TC_SETUP_BLOCK:

		return bnxt_vf_rep_setup_tc_block(dev, type_data);

	default:

		return -EOPNOTSUPP;

	}

}



struct net_device *bnxt_get_vf_rep(struct bnxt *bp, u16 cfa_code)

{

	u16 vf_idx;

drivers/net/ethernet/chelsio/cxgb4/cxgb4_main.c

+36 −6
Original line number Diff line number Diff line
@@ -2889,8 +2889,7 @@ static int cxgb_set_tx_maxrate(struct net_device *dev, int index, u32 rate) 
static int cxgb_setup_tc_flower(struct net_device *dev,

				struct tc_cls_flower_offload *cls_flower)

{

	if (!is_classid_clsact_ingress(cls_flower->common.classid) ||

	    cls_flower->common.chain_index)

	if (cls_flower->common.chain_index)

		return -EOPNOTSUPP;



	switch (cls_flower->command) {
@@ -2908,8 +2907,7 @@ static int cxgb_setup_tc_flower(struct net_device *dev, 
static int cxgb_setup_tc_cls_u32(struct net_device *dev,

				 struct tc_cls_u32_offload *cls_u32)

{

	if (!is_classid_clsact_ingress(cls_u32->common.classid) ||

	    cls_u32->common.chain_index)

	if (cls_u32->common.chain_index)

		return -EOPNOTSUPP;



	switch (cls_u32->command) {
@@ -2923,9 +2921,10 @@ static int cxgb_setup_tc_cls_u32(struct net_device *dev, 
	}

}



static int cxgb_setup_tc(struct net_device *dev, enum tc_setup_type type,

			 void *type_data)

static int cxgb_setup_tc_block_cb(enum tc_setup_type type, void *type_data,

				  void *cb_priv)

{

	struct net_device *dev = cb_priv;

	struct port_info *pi = netdev2pinfo(dev);

	struct adapter *adap = netdev2adap(dev);
@@ -2946,6 +2945,37 @@ static int cxgb_setup_tc(struct net_device *dev, enum tc_setup_type type, 
	}

}



static int cxgb_setup_tc_block(struct net_device *dev,

			       struct tc_block_offload *f)

{

	struct port_info *pi = netdev2pinfo(dev);



	if (f->binder_type != TCF_BLOCK_BINDER_TYPE_CLSACT_INGRESS)

		return -EOPNOTSUPP;



	switch (f->command) {

	case TC_BLOCK_BIND:

		return tcf_block_cb_register(f->block, cxgb_setup_tc_block_cb,

					     pi, dev);

	case TC_BLOCK_UNBIND:

		tcf_block_cb_unregister(f->block, cxgb_setup_tc_block_cb, pi);

		return 0;

	default:

		return -EOPNOTSUPP;

	}

}



static int cxgb_setup_tc(struct net_device *dev, enum tc_setup_type type,

			 void *type_data)

{

	switch (type) {

	case TC_SETUP_BLOCK:

		return cxgb_setup_tc_block(dev, type_data);

	default:

		return -EOPNOTSUPP;

	}

}



static netdev_features_t cxgb_fix_features(struct net_device *dev,

					   netdev_features_t features)

{

drivers/net/ethernet/intel/ixgbe/ixgbe_main.c

+38 −7
Original line number Diff line number Diff line
@@ -9365,13 +9365,10 @@ static int ixgbe_configure_clsu32(struct ixgbe_adapter *adapter, 
	return err;

}



static int ixgbe_setup_tc_cls_u32(struct net_device *dev,

static int ixgbe_setup_tc_cls_u32(struct ixgbe_adapter *adapter,

				  struct tc_cls_u32_offload *cls_u32)

{

	struct ixgbe_adapter *adapter = netdev_priv(dev);



	if (!is_classid_clsact_ingress(cls_u32->common.classid) ||

	    cls_u32->common.chain_index)

	if (cls_u32->common.chain_index)

		return -EOPNOTSUPP;



	switch (cls_u32->command) {
@@ -9390,6 +9387,40 @@ static int ixgbe_setup_tc_cls_u32(struct net_device *dev, 
	}

}



static int ixgbe_setup_tc_block_cb(enum tc_setup_type type, void *type_data,

				   void *cb_priv)

{

	struct ixgbe_adapter *adapter = cb_priv;



	switch (type) {

	case TC_SETUP_CLSU32:

		return ixgbe_setup_tc_cls_u32(adapter, type_data);

	default:

		return -EOPNOTSUPP;

	}

}



static int ixgbe_setup_tc_block(struct net_device *dev,

				struct tc_block_offload *f)

{

	struct ixgbe_adapter *adapter = netdev_priv(dev);



	if (f->binder_type != TCF_BLOCK_BINDER_TYPE_CLSACT_INGRESS)

		return -EOPNOTSUPP;



	switch (f->command) {

	case TC_BLOCK_BIND:

		return tcf_block_cb_register(f->block, ixgbe_setup_tc_block_cb,

					     adapter, adapter);

	case TC_BLOCK_UNBIND:

		tcf_block_cb_unregister(f->block, ixgbe_setup_tc_block_cb,

					adapter);

		return 0;

	default:

		return -EOPNOTSUPP;

	}

}



static int ixgbe_setup_tc_mqprio(struct net_device *dev,

				 struct tc_mqprio_qopt *mqprio)

{
@@ -9401,8 +9432,8 @@ static int __ixgbe_setup_tc(struct net_device *dev, enum tc_setup_type type, 
			    void *type_data)

{

	switch (type) {

	case TC_SETUP_CLSU32:

		return ixgbe_setup_tc_cls_u32(dev, type_data);

	case TC_SETUP_BLOCK:

		return ixgbe_setup_tc_block(dev, type_data);

	case TC_SETUP_MQPRIO:

		return ixgbe_setup_tc_mqprio(dev, type_data);

	default: