Skip to contentSkip to conversation

xen-netback: allow changing the MAC address of the interface

PATCH Superseded Matt Wilson 2 participants 3 messages Oct 19, 2012 – Oct 23, 2012

Subsystems (2)

v1 of 2 Superseded Latest is v2 →

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 xen-netback: allow changing the MAC address of the interface

---
 drivers/net/xen-netback/interface.c |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)
drivers/net/xen-netback/interface.c +2 -0
index b7d41f8..f733cae 100644
@@ -238,6 +238,8 @@ static const struct net_device_ops xenvif_netdev_ops = {
 	.ndo_stop	= xenvif_close,
 	.ndo_change_mtu	= xenvif_change_mtu,
 	.ndo_fix_features = xenvif_fix_features,
+	.ndo_set_mac_address = eth_mac_addr,
+	.ndo_validate_addr   = eth_validate_addr,
 };
 
 struct xenvif *xenvif_alloc(struct device *parent, domid_t domid,
Signature · 1 line
1.7.4.5
Acked-by on v1 1/1
#1 lore raw
Quoted 3 lines · Matt Wilson

I can't remember that but it seems reasonable:

But, can you remind us of the usecase? Perhaps in the commit message?

Lastly, netback patches should be sent to [email protected] as per MAINTAINERS in order to be applied.

Thanks,
Ian.

Quoted 18 lines
---
 drivers/net/xen-netback/interface.c |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)
drivers/net/xen-netback/interface.c +2 -0
index b7d41f8..f733cae 100644
@@ -238,6 +238,8 @@ static const struct net_device_ops xenvif_netdev_ops = {
 	.ndo_stop	= xenvif_close,
 	.ndo_change_mtu	= xenvif_change_mtu,
 	.ndo_fix_features = xenvif_fix_features,
+	.ndo_set_mac_address = eth_mac_addr,
+	.ndo_validate_addr   = eth_validate_addr,
 };
 
 struct xenvif *xenvif_alloc(struct device *parent, domid_t domid,
#2 lore raw
Quoted 6 lines · Ian Campbell
Quoted 3 lines · Matt Wilson

I can't remember that but it seems reasonable:

http://lists.xen.org/archives/html/xen-devel/2011-01/msg01427.html

Quoted 3 lines

But, can you remind us of the usecase? Perhaps in the commit message?

It could be useful for some ebtables configurations, I think.

Quoted 2 lines

Lastly, netback patches should be sent to [email protected] as per MAINTAINERS in order to be applied.

Indeed. I wrongly assumed that Konrad would take care of bringing it into his tree and that it'd get pulled from there. I'll expand the commit message, add your Acked-by:, and send it on.

Matt

Quoted 19 lines
Quoted 18 lines
---
 drivers/net/xen-netback/interface.c |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)
drivers/net/xen-netback/interface.c +2 -0
index b7d41f8..f733cae 100644
@@ -238,6 +238,8 @@ static const struct net_device_ops xenvif_netdev_ops = {
 	.ndo_stop	= xenvif_close,
 	.ndo_change_mtu	= xenvif_change_mtu,
 	.ndo_fix_features = xenvif_fix_features,
+	.ndo_set_mac_address = eth_mac_addr,
+	.ndo_validate_addr   = eth_validate_addr,
 };
 
 struct xenvif *xenvif_alloc(struct device *parent, domid_t domid,