Re: [RESEND net-next v2 00/12]drivers: net: convert tasklets to use new tasklet_setup() API

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




Saeed,
ommit 12cc923f1ccc ("tasklet: Introduce new initialization API")'
introduced a new tasklet initialization API. This series converts
all the net/* drivers to use the new tasklet_setup() API
       ^^^
this is not all drivers ..

Right, my bad, I should not have said net/*.


see below

This series is based on v5.9-rc5

Allen Pais (12):
    net: mvpp2: Prepare to use the new tasklet API
    net: arcnet: convert tasklets to use new tasklet_setup() API
    net: caif: convert tasklets to use new tasklet_setup() API
    net: ifb: convert tasklets to use new tasklet_setup() API
    net: ppp: convert tasklets to use new tasklet_setup() API
    net: cdc_ncm: convert tasklets to use new tasklet_setup() API
    net: hso: convert tasklets to use new tasklet_setup() API
    net: lan78xx: convert tasklets to use new tasklet_setup() API
    net: pegasus: convert tasklets to use new tasklet_setup() API
    net: r8152: convert tasklets to use new tasklet_setup() API
    net: rtl8150: convert tasklets to use new tasklet_setup() API
    net: usbnet: convert tasklets to use new tasklet_setup() API



You are only converting drivers which are passing the taskelt
struct as
data ptr, most of other drivers are passing the container of the
tasklet as data, why not convert them as well, and let them use
container_of to find their data ? it is really straight forward and
will help convert most of net driver if not all.


from_tasklet uses container_of internally. use of container_of is
avoided cause it end being really long.

I understand that, what I meant, you didn't really convert all drivers,
as you claim in the cover letter, all you did is converting __some__
drivers which are passing the tasklet ptr as data ptr. all other
drivers that use tasklet_init differently are not converted, and it
should be relatively easy as i explained above.

The list of drivers using tasklet_init is longer than what you touched
in your series:

  drivers/net/arcnet/arcnet.c                     |  7 +++----
  drivers/net/caif/caif_virtio.c                  |  8 +++-----
  drivers/net/ethernet/marvell/mvpp2/mvpp2.h      |  1 +
  drivers/net/ethernet/marvell/mvpp2/mvpp2_main.c |  1 +
  drivers/net/ifb.c                               |  7 +++----
  drivers/net/ppp/ppp_async.c                     |  8 ++++----
  drivers/net/ppp/ppp_synctty.c                   |  8 ++++----
  drivers/net/usb/cdc_ncm.c                       |  8 ++++----
  drivers/net/usb/hso.c                           | 10 +++++-----
  drivers/net/usb/lan78xx.c                       |  6 +++---
  drivers/net/usb/pegasus.c                       |  6 +++---
  drivers/net/usb/r8152.c                         |  8 +++-----
  drivers/net/usb/rtl8150.c                       |  6 +++---
  drivers/net/usb/usbnet.c                        |  3 +--
  14 files changed, 41 insertions(+), 46 deletions(-)

The full file/driver list :

Yes, drivers/net/ethernet/* drivers/net/wireless/* were submitted as a separate series. The rest of them were part of the series above.

I should break it down so that it's easier to bisect.

Thanks.



$ git grep -l tasklet_init drivers/net/
drivers/net/arcnet/arcnet.c
drivers/net/caif/caif_virtio.c
drivers/net/ethernet/alteon/acenic.c
drivers/net/ethernet/amd/xgbe/xgbe-drv.c
drivers/net/ethernet/amd/xgbe/xgbe-i2c.c
drivers/net/ethernet/amd/xgbe/xgbe-mdio.c
drivers/net/ethernet/broadcom/cnic.c
drivers/net/ethernet/cadence/macb_main.c
drivers/net/ethernet/cavium/liquidio/lio_main.c
drivers/net/ethernet/cavium/octeon/octeon_mgmt.c
drivers/net/ethernet/cavium/thunder/nicvf_main.c
drivers/net/ethernet/chelsio/cxgb/sge.c
drivers/net/ethernet/chelsio/cxgb3/sge.c
drivers/net/ethernet/chelsio/cxgb4/cxgb4_tc_mqprio.c
drivers/net/ethernet/chelsio/cxgb4/sge.c
drivers/net/ethernet/dlink/sundance.c
drivers/net/ethernet/huawei/hinic/hinic_hw_eqs.c
drivers/net/ethernet/ibm/ehea/ehea_main.c
drivers/net/ethernet/ibm/ibmvnic.c
drivers/net/ethernet/jme.c
drivers/net/ethernet/marvell/skge.c
drivers/net/ethernet/mellanox/mlx4/eq.c
drivers/net/ethernet/mellanox/mlx5/core/eq.c
drivers/net/ethernet/mellanox/mlx5/core/fpga/conn.c
drivers/net/ethernet/mellanox/mlxsw/pci.c
drivers/net/ethernet/micrel/ks8842.c
drivers/net/ethernet/micrel/ksz884x.c
drivers/net/ethernet/natsemi/ns83820.c
drivers/net/ethernet/netronome/nfp/nfp_net_common.c
drivers/net/ethernet/ni/nixge.c
drivers/net/ethernet/qlogic/qed/qed_int.c
drivers/net/ethernet/silan/sc92031.c
drivers/net/ethernet/smsc/smc91x.c
drivers/net/ifb.c
drivers/net/ppp/ppp_async.c
drivers/net/ppp/ppp_synctty.c
drivers/net/usb/cdc_ncm.c
drivers/net/usb/hso.c
drivers/net/usb/lan78xx.c
drivers/net/usb/pegasus.c
drivers/net/usb/r8152.c
drivers/net/usb/rtl8150.c
drivers/net/wireless/ath/ath11k/pci.c
drivers/net/wireless/mediatek/mt76/mac80211.c
drivers/net/wireless/mediatek/mt76/mt7603/init.c
drivers/net/wireless/mediatek/mt76/mt7615/mmio.c
drivers/net/wireless/mediatek/mt76/mt76x02_dfs.c
drivers/net/wireless/mediatek/mt76/mt76x02_mmio.c
drivers/net/wireless/mediatek/mt76/usb.c
drivers/net/wireless/mediatek/mt7601u/dma.c




[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux