| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Date: Tue, 15 Sep 2020 10:14:57 +0530
From: Allen Pais <apais@...ux.microsoft.com>
To: Saeed Mahameed <saeedm@...dia.com>,
"davem@...emloft.net" <davem@...emloft.net>,
"allen.lkml@...il.com" <allen.lkml@...il.com>
Cc: "m.grzeschik@...gutronix.de" <m.grzeschik@...gutronix.de>,
"linux-usb@...r.kernel.org" <linux-usb@...r.kernel.org>,
"woojung.huh@...rochip.com" <woojung.huh@...rochip.com>,
"petkan@...leusys.com" <petkan@...leusys.com>,
"oliver@...kum.org" <oliver@...kum.org>,
"netdev@...r.kernel.org" <netdev@...r.kernel.org>,
"kuba@...nel.org" <kuba@...nel.org>,
"UNGLinuxDriver@...rochip.com" <UNGLinuxDriver@...rochip.com>,
"paulus@...ba.org" <paulus@...ba.org>,
"linux-ppp@...r.kernel.org" <linux-ppp@...r.kernel.org>
Subject: Re: [RESEND net-next v2 00/12]drivers: net: convert tasklets to use
new tasklet_setup() API
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
>
Powered by blists - more mailing lists