Thanks for looking into this. At this time we are not sure when we can have someone look into your issues, so please send us patches so that we may test those. Some of the code execution paths may not be applicable and we will ACK/NAK the patches as necessary.
Please make sure to test your patches against the necessary hardware. Some of the drivers (e100 in particular) have been out in the field for a long time and we haven't heard of many recurring issues. Todd Fujinaka Software Application Engineer Networking Division (ND) Intel Corporation todd.fujin...@intel.com (503) 712-4565 -----Original Message----- From: linux-nics-boun...@isotope.jf.intel.com [mailto:linux-nics-boun...@isotope.jf.intel.com] On Behalf Of Jia-Ju Bai Sent: Sunday, December 14, 2014 5:52 PM To: Linux NICS; e1000-devel@lists.sourceforge.net Subject: [linux-nics] Potential bugs found in e100 Recently I test linux device drivers in Linux 3.17.2, and find some potential bugs. e100 driver: The target file is drivers/net/ethernet/intel/e100.c, which is used to build e100.ko. I hope you can help me check my findings: [1] The function pci_pool_create is called by e100_probe when initializing the ethernet card driver. But when pci_pool_create is failed, which means that it returns NULL to nic->cbs_pool, the system crash will happen. Because pci_pool_alloc (in e100_alloc_cbs in e100_up in e100_open) need to use nic->cbs_pool to allocate the resource, but it is NULL. I suggest that a check can be added in the code to detect whether pci_pool_create returns NULL. [2] In the normal process, netif_napi_add is called in e100_probe, but netif_napi_del is not called in e100_remove. However, many other ethernet card drivers call them in pairs, even in the error handling paths, such as r8169 and igb. Could you help me check these findings? Thank you very much, and I'm looking forward to your reply. -- Jia-Ju Bai _______________________________________________ Linux-nics mailing list linux-n...@intel.com ------------------------------------------------------------------------------ Download BIRT iHub F-Type - The Free Enterprise-Grade BIRT Server from Actuate! Instantly Supercharge Your Business Reports and Dashboards with Interactivity, Sharing, Native Excel Exports, App Integration & more Get technology previously reserved for billion-dollar corporations, FREE http://pubads.g.doubleclick.net/gampad/clk?id=164703151&iu=/4140/ostg.clktrk _______________________________________________ E1000-devel mailing list E1000-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/e1000-devel To learn more about Intel® Ethernet, visit http://communities.intel.com/community/wired
