netdev - Re: [ofa-general] Re: [PATCH v3 01/13] QLogic VNIC: Driver

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

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Tue, 3 Jun 2008 17:44:23 +0530
From:	"Ramachandra K" <ramachandra.kuchimanchi@...gic.com>
To:	"Stephen Hemminger" <shemminger@...tta.com>
Cc:	netdev@...r.kernel.org, rdreier@...co.com,
	amar.mudrankit@...gic.com, general@...ts.openfabrics.org,
	poornima.kamath@...gic.com
Subject: Re: [ofa-general] Re: [PATCH v3 01/13] QLogic VNIC: Driver - netdev implementation

On Thu, May 29, 2008 at 10:57 PM, Stephen Hemminger
<shemminger@...tta.com> wrote:
> On Thu, 29 May 2008 15:24:23 +0530
> Ramachandra K <ramachandra.kuchimanchi@...gic.com> wrote:

<snip>
>
>> +module_param(vnic_debug, uint, 0444);
>> +MODULE_PARM_DESC(vnic_debug, "Enable debug tracing if > 0");
>
> maybe migrate this to ethtool msg_level?
>

We investigated this and found that we cannot map all of our Infiniband
level communication errors etc to the ethtool message levels. One option could
be to add support for Ethtool message levels in addition to the vnic_debug
controlled messages. The Ethtool support would make this driver conform
to ethtool similar to other Ethernet drivers and the vnic_debug messages will
help in debugging the Infiniband related errors.

Is ethtool support mandatory for a merge to 2.6.27 or can we consider
it as a future
enhancement while we first focus on moving this driver to use CONFIGFS ?

Regards,
Ram
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html