netdev - [RFC] Running single tests via ethtool

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-next>] [day] [month] [year] [list]

Date:	Fri, 30 May 2008 23:08:20 +0100
From:	Ben Hutchings <bhutchings@...arflare.com>
To:	netdev@...r.kernel.org
Cc:	linux-net-drivers@...arflare.com
Subject: [RFC] Running single tests via ethtool

We have had a customer request to be able to run individual self-tests in
a future version of our driver.  Now we could do this any way we want in
an out-of-tree driver, but I'd rather find some way of doing this that
will also be acceptable in-tree.  I think that would mean extending the
ethtool self-test API in some way.

One possibility might be to add a new flag and to assign the reserved field
as a test index:

--- a/include/linux/ethtool.h
+++ b/include/linux/ethtool.h
@@ -235,13 +235,14 @@ struct ethtool_gstrings {
 enum ethtool_test_flags {
 	ETH_TEST_FL_OFFLINE	= (1 << 0),	/* online / offline */
 	ETH_TEST_FL_FAILED	= (1 << 1),	/* test passed / failed */
+	ETH_TEST_FL_SINGLE	= (1 << 2),	/* run single test */
 };

 /* for requesting NIC test and getting results*/
 struct ethtool_test {
 	__u32	cmd;		/* ETHTOOL_TEST */
 	__u32	flags;		/* ETH_TEST_FL_xxx */
-	__u32	reserved;
+	__u32	index;		/* test index if ETH_TEST_FL_SINGLE */
 	__u32	len;		/* result length, in number of u64 elements */
 	__u64	data[0];
 };

Clients would need to check len on return since the ethtool core and driver
ethtool functions ignore unknown flags.  I am assuming that running all
online self-tests instead of a single self-test will be harmless.

Ideally the ethtool core would allocate only the necessary memory for the
results of the selected test set or single test, but it would need to know
whether the driver also supports this extension.

Ben.

-- 
Ben Hutchings, Senior Software Engineer, Solarflare Communications
Not speaking for my employer; that's the marketing department's job.
--
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