[Settings] Add per-netdevice settings block

Add a configuration settings block for each net device.  This will
provide the parent scope for settings applicable only to that network
device (e.g. non-volatile options stored on the NIC, options obtained via
DHCP, etc.).

Expose the MAC address as a setting.

2 files changed, 109 insertions, 0 deletions

diff --git a/src/net/netdev_settings.c b/src/net/netdev_settings.c
new file mode 100644
index 000000000..9baad888b
--- /dev/null
+++ b/src/net/netdev_settings.c
@@ -0,0 +1,95 @@
+/*
+ * Copyright (C) 2008 Michael Brown <mbrown@fensystems.co.uk>.
+ *
+ * This program is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU General Public License as
+ * published by the Free Software Foundation; either version 2 of the
+ * License, or any later version.
+ *
+ * This program is distributed in the hope that it will be useful, but
+ * WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+ * General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
+ */
+
+#include <string.h>
+#include <errno.h>
+#include <gpxe/dhcp.h>
+#include <gpxe/settings.h>
+#include <gpxe/netdevice.h>
+
+/** @file
+ *
+ * Network device configuration settings
+ *
+ */
+
+/**
+ * Store value of network device setting
+ *
+ * @v settings		Settings block
+ * @v tag		Setting tag number
+ * @v data		Setting data, or NULL to clear setting
+ * @v len		Length of setting data
+ * @ret rc		Return status code
+ */
+static int netdev_store ( struct settings *settings, unsigned int tag,
+			  const void *data, size_t len ) {
+	struct net_device *netdev =
+		container_of ( settings, struct net_device, settings );
+
+	switch ( tag ) {
+	case DHCP_EB_MAC:
+		if ( len != netdev->ll_protocol->ll_addr_len )
+			return -EINVAL;
+		memcpy ( netdev->ll_addr, data, len );
+		return 0;
+	default :
+		return simple_settings_store ( settings, tag, data, len );
+	}
+}
+
+/**
+ * Fetch value of network device setting
+ *
+ * @v settings		Settings block
+ * @v tag		Setting tag number
+ * @v data		Setting data, or NULL to clear setting
+ * @v len		Length of setting data
+ * @ret rc		Return status code
+ */
+static int netdev_fetch ( struct settings *settings, unsigned int tag,
+			  void *data, size_t len ) {
+	struct net_device *netdev =
+		container_of ( settings, struct net_device, settings );
+
+	switch ( tag ) {
+	case DHCP_EB_MAC:
+		if ( len > netdev->ll_protocol->ll_addr_len )
+			len = netdev->ll_protocol->ll_addr_len;
+		memcpy ( data, netdev->ll_addr, len );
+		return netdev->ll_protocol->ll_addr_len;
+	default :
+		return simple_settings_fetch ( settings, tag, data, len );
+	}
+}
+
+/** Network device configuration settings operations */
+struct settings_operations netdev_settings_operations = {
+	.store = netdev_store,
+	.fetch = netdev_fetch,
+};
+
+/** Network device named settings */
+struct named_setting netdev_named_settings[] __named_setting = {
+	{
+		.name = "mac",
+		.description = "MAC address",
+		.tag = DHCP_EB_MAC,
+		.type = &setting_type_hex,
+	},
+};
diff --git a/src/net/netdevice.c b/src/net/netdevice.c
index f2778e880..323e91056 100644
--- a/src/net/netdevice.c
+++ b/src/net/netdevice.c
@@ -266,6 +266,9 @@ struct net_device * alloc_netdev ( size_t priv_size ) {
 		netdev->refcnt.free = free_netdev;
 		INIT_LIST_HEAD ( &netdev->tx_queue );
 		INIT_LIST_HEAD ( &netdev->rx_queue );
+		settings_init ( &netdev->settings,
+				&netdev_settings_operations, &netdev->refcnt,
+				netdev->name );
 		netdev->priv = ( ( ( void * ) netdev ) + sizeof ( *netdev ) );
 	}
 	return netdev;
@@ -282,11 +285,19 @@ struct net_device * alloc_netdev ( size_t priv_size ) {
  */
 int register_netdev ( struct net_device *netdev ) {
 	static unsigned int ifindex = 0;
+	int rc;
 
 	/* Create device name */
 	snprintf ( netdev->name, sizeof ( netdev->name ), "net%d",
 		   ifindex++ );
 
+	/* Register per-netdev configuration settings */
+	if ( ( rc = register_settings ( &netdev->settings, NULL ) ) != 0 ) {
+		DBGC ( netdev, "NETDEV %p could not register settings: %s\n",
+		       netdev, strerror ( rc ) );
+		return rc;
+	}
+
 	/* Add to device list */
 	netdev_get ( netdev );
 	list_add_tail ( &netdev->list, &net_devices );
@@ -357,6 +368,9 @@ void unregister_netdev ( struct net_device *netdev ) {
 	/* Ensure device is closed */
 	netdev_close ( netdev );
 
+	/* Unregister per-netdev configuration settings */
+	unregister_settings ( &netdev->settings );
+
 	/* Remove from device list */
 	list_del ( &netdev->list );
 	netdev_put ( netdev );