Messages in this thread | ![/](/images/icornerl.gif) | | From | Yoann Padioleau <> | Subject | dev->priv to netdev_priv(dev) | Date | Fri, 03 Aug 2007 20:03:05 +0200 |
| |
Jeff Garzik <jeff@garzik.org> writes:
>> PS: I have performed the same transformation on the whole kernel >> and it affects around 70 files, most of them in drivers/net/. >> Should I split my patch for each subnet directories ? (wireless/, wan/, etc) > > applied. splitting up by sub-directory would be helpful.
Done. I was not always able to find the maintainer for all the subdirectories so for those cases I have considered you as the maintainer :)
I have run another semantic patch (more like a semantic grep in fact) to find the places where our tool didn't transform the dev->priv to netdev_priv. The reason is that our tool was not able to find the call to alloc_dev_xxx(sizeof(T),...) in the driver or there was an assignation to dev->priv which means that transforming to netdev_priv would do something wrong. I have printed the list of "bad driver" below.
In very few cases the driver was in fact "good" but because of the way the code was written, our tool was not able to find the alloc_xxx function.
drivers/net/82596.c drivers/net/chelsio/cxgb2.c drivers/net/chelsio/sge.c drivers/net/cxgb3/cxgb3_main.c drivers/net/cxgb3/sge.c drivers/net/e1000/e1000_main.c drivers/net/ehea/ehea_main.c drivers/net/hamradio/dmascc.c drivers/net/hamradio/scc.c drivers/net/lance.c drivers/net/mace.c
for mace.c the code is written this way:
#define PRIV_BYTES (sizeof(struct mace_data) \ + (N_RX_RING + NCMDS_TX * N_TX_RING + 3) * sizeof(struct dbdma_cmd))
dev = alloc_etherdev(PRIV_BYTES);
So our semantic patch can't find a alloc_etherdev(sizeof(T)). If I inline the definition of PRIV_BYTES, then our tool will be able to transform it.
drivers/net/ni65.c drivers/net/pcmcia/com20020_cs.c
dev = alloc_arcdev("");
drivers/net/ppp_generic.c drivers/net/spider_net_ethtool.c
again, the code is written this way:
alloc_size = sizeof(struct spider_net_card) + (tx_descriptors + rx_descriptors) * sizeof(struct spider_net_descr); netdev = alloc_etherdev(alloc_size);
If I inline the definition of alloc_size, then we can transform the file.
drivers/net/tulip/xircom_cb.c drivers/net/wan/cosa.c drivers/net/wan/cycx_x25.c drivers/net/wan/hdlc_fr.c drivers/net/wan/hdlc_ppp.c drivers/net/wan/hostess_sv11.c drivers/net/wan/pc300_drv.c drivers/net/wan/sbni.c drivers/net/wireless/airo.c drivers/net/wireless/libertas/ethtool.c drivers/net/wireless/libertas/main.c drivers/net/wireless/libertas/scan.c drivers/net/wireless/libertas/wext.c drivers/net/wireless/wavelan.c drivers/net/wireless/zd1201.c
drivers/net/tokenring/tms380tr.c This file is both used as a single module and used by other module. The other modules have a call to a alloc_xxx but not tms380tr.c when used as a single module.
- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| ![\](/images/icornerr.gif) |