lkml.org 
[lkml]   [2022]   [Jun]   [23]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 4.19 226/234] usb: gadget: u_ether: fix regression in setting fixed MAC address
    Date
    From: Marian Postevca <posteuca@mutex.one>

    commit b337af3a4d6147000b7ca6b3438bf5c820849b37 upstream.

    In systemd systems setting a fixed MAC address through
    the "dev_addr" module argument fails systematically.
    When checking the MAC address after the interface is created
    it always has the same but different MAC address to the one
    supplied as argument.

    This is partially caused by systemd which by default will
    set an internally generated permanent MAC address for interfaces
    that are marked as having a randomly generated address.

    Commit 890d5b40908bfd1a ("usb: gadget: u_ether: fix race in
    setting MAC address in setup phase") didn't take into account
    the fact that the interface must be marked as having a set
    MAC address when it's set as module argument.

    Fixed by marking the interface with NET_ADDR_SET when
    the "dev_addr" module argument is supplied.

    Fixes: 890d5b40908bfd1a ("usb: gadget: u_ether: fix race in setting MAC address in setup phase")
    Cc: stable@vger.kernel.org
    Signed-off-by: Marian Postevca <posteuca@mutex.one>
    Link: https://lore.kernel.org/r/20220603153459.32722-1-posteuca@mutex.one
    Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
    ---
    drivers/usb/gadget/function/u_ether.c | 11 +++++++++--
    1 file changed, 9 insertions(+), 2 deletions(-)

    --- a/drivers/usb/gadget/function/u_ether.c
    +++ b/drivers/usb/gadget/function/u_ether.c
    @@ -772,9 +772,13 @@ struct eth_dev *gether_setup_name(struct
    dev->qmult = qmult;
    snprintf(net->name, sizeof(net->name), "%s%%d", netname);

    - if (get_ether_addr(dev_addr, net->dev_addr))
    + if (get_ether_addr(dev_addr, net->dev_addr)) {
    + net->addr_assign_type = NET_ADDR_RANDOM;
    dev_warn(&g->dev,
    "using random %s ethernet address\n", "self");
    + } else {
    + net->addr_assign_type = NET_ADDR_SET;
    + }
    if (get_ether_addr(host_addr, dev->host_mac))
    dev_warn(&g->dev,
    "using random %s ethernet address\n", "host");
    @@ -831,6 +835,9 @@ struct net_device *gether_setup_name_def
    INIT_LIST_HEAD(&dev->tx_reqs);
    INIT_LIST_HEAD(&dev->rx_reqs);

    + /* by default we always have a random MAC address */
    + net->addr_assign_type = NET_ADDR_RANDOM;
    +
    skb_queue_head_init(&dev->rx_frames);

    /* network device setup */
    @@ -868,7 +875,6 @@ int gether_register_netdev(struct net_de
    g = dev->gadget;

    memcpy(net->dev_addr, dev->dev_mac, ETH_ALEN);
    - net->addr_assign_type = NET_ADDR_RANDOM;

    status = register_netdev(net);
    if (status < 0) {
    @@ -908,6 +914,7 @@ int gether_set_dev_addr(struct net_devic
    if (get_ether_addr(dev_addr, new_addr))
    return -EINVAL;
    memcpy(dev->dev_mac, new_addr, ETH_ALEN);
    + net->addr_assign_type = NET_ADDR_SET;
    return 0;
    }
    EXPORT_SYMBOL_GPL(gether_set_dev_addr);

    \
     
     \ /
      Last update: 2022-06-23 20:21    [W:2.475 / U:0.024 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site