| 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
| ||
|
Message-ID: <458B86A7.6030402@chelsio.com> Date: Thu, 21 Dec 2006 23:17:59 -0800 From: Divy Le Ray <divy@...lsio.com> To: Arjan van de Ven <arjan@...radead.org> CC: Divy Le Ray <None@...lsio.com>, jeff@...zik.org, netdev@...r.kernel.org, linux-kernel@...r.kernel.org, swise@...ngridcomputing.com Subject: Re: [PATCH 2/10] cxgb3 - main source file Arjan van de Ven wrote: >> They are used to parameter the HW: >> register access, > > ethtool supports that, so shouldn't be an ioctl for sure > >> configuration of queue sets, on board memory >> configuration, > > I'm sure ethtool can do that too > >> firmware load, etc ... > > and for this we have request_firmware() interface. > > adding device specific ioctl that duplicate functionality that exists or > should exist in a generic way isn't really acceptable for 2.6 kernels > anymore.... > Arjan, The driver implements all the ethtool operations that apply to it. The GETREG ioctl is left for debug purposes: get_regs doesn't return clear-on-read registers while GETREG does. Using request_firmware assumes that the driver knows the FW file name and the driver initiates the load. That's not our model where we work with different FWs, don't know what the names are, and the user initiates the load. Cheers, Divy - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists