Received: by 2002:a05:6358:d09b:b0:dc:cd0c:909e with SMTP id jc27csp3182451rwb; Wed, 30 Nov 2022 16:52:59 -0800 (PST) X-Google-Smtp-Source: AA0mqf5Wd8Mwtr0tLBfsZVtHBp+SlLrQ6JHSjgmyh/ooAStxwEGIJD9xUAsAwpftqP9sqKVwHNfj X-Received: by 2002:a17:902:a989:b0:188:d4bf:dbfe with SMTP id bh9-20020a170902a98900b00188d4bfdbfemr44541764plb.31.1669855979487; Wed, 30 Nov 2022 16:52:59 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1669855979; cv=none; d=google.com; s=arc-20160816; b=HFwjr4wINMw3l7nA2gMD+RIR3/HEXUcWpEciOv1B4L/JLFKQFB3jvfv9cOaIO4inWb 5sZSxiAKX076jd+Y5AbtlmeJ4fDDjYYLs1F+8KgXGmOz9SRUxZu436bXXHtK4dv2HJ1G VbXWzyRDP+SFMvpHgpnNkT6WoG4PxlbzosGMINhb0lqc1ROf3jlHzzqXkVN8tkkvernn 1GuoJTC2wC5PI/K1f4IJKHfmJdsT0vEX2kI3PXqCDVZRHFnQjx1KN9ocnBZlBZWMmx+s KibPWYYfn//4a9sADU5ZDSUZ29MoxtKgOHmwYZtsh+ieajbWkjg6oJZHijnROyAuoS3Z IOwA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:content-language :mime-version:accept-language:in-reply-to:references:message-id:date :thread-index:thread-topic:subject:cc:to:from; bh=ke8ZM46opjwmx5PnpItlVEtGtHXe6sgPL6DK4J/22dI=; b=ZZFOmaejnUBR4jZJdwKqH+yOxyIGenwRvhZr+RN9ba72vYRs30LZSGVDi5zq/kB31h aCjRfSXLs/RrXgFY8g9Cbno1uWHkvuOT6HqU6cnyUanf+phN8t3wBXMMjPVqs2XJO6pm tNdkXLIFT6awzRG8RY79xyUOxGs22DroT2jStmSEhlm0J0RLK5TLkZ9oy8MXPo9Bx+XA utdMjkjPfHhkzglYt3VhTnoIbDy7mjQIWPb1kPxKDVuzN6q9YqB1gQB8IRvO5J3z3V5+ v9WxIxlUXA204jsbTy+ja0Mgl1JGisloYvvHI3FxnLcj+hkCm5Q9PnBLpzO9F3HJzur8 smPw== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=aculab.com Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id m5-20020a170902bb8500b00180556e1b6bsi2523654pls.609.2022.11.30.16.52.21; Wed, 30 Nov 2022 16:52:59 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=aculab.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230022AbiK3XUS convert rfc822-to-8bit (ORCPT + 83 others); Wed, 30 Nov 2022 18:20:18 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:59634 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230254AbiK3XTu (ORCPT ); Wed, 30 Nov 2022 18:19:50 -0500 Received: from eu-smtp-delivery-151.mimecast.com (eu-smtp-delivery-151.mimecast.com [185.58.85.151]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 0EA62D81 for ; Wed, 30 Nov 2022 15:12:54 -0800 (PST) Received: from AcuMS.aculab.com (156.67.243.121 [156.67.243.121]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384) id uk-mta-20-Ail7hLe4PC6pK14wdk6TGw-1; Wed, 30 Nov 2022 23:12:16 +0000 X-MC-Unique: Ail7hLe4PC6pK14wdk6TGw-1 Received: from AcuMS.Aculab.com (10.202.163.4) by AcuMS.aculab.com (10.202.163.4) with Microsoft SMTP Server (TLS) id 15.0.1497.42; Wed, 30 Nov 2022 23:12:15 +0000 Received: from AcuMS.Aculab.com ([::1]) by AcuMS.aculab.com ([::1]) with mapi id 15.00.1497.044; Wed, 30 Nov 2022 23:12:15 +0000 From: David Laight To: 'Andrew Lunn' , Brian Masney CC: "irusskikh@marvell.com" , "davem@davemloft.net" , "edumazet@google.com" , "kuba@kernel.org" , "pabeni@redhat.com" , "netdev@vger.kernel.org" , "linux-kernel@vger.kernel.org" , "cth451@gmail.com" Subject: RE: [PATCH] net: atlantic: fix check for invalid ethernet addresses Thread-Topic: [PATCH] net: atlantic: fix check for invalid ethernet addresses Thread-Index: AQHZBQNlZXesXr8LFkOm0c8QYDoVsa5YF2cw Date: Wed, 30 Nov 2022 23:12:15 +0000 Message-ID: <3adb7dc622a3429782ca89e83c8e020d@AcuMS.aculab.com> References: <20221130174259.1591567-1-bmasney@redhat.com> In-Reply-To: Accept-Language: en-GB, en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-ms-exchange-transport-fromentityheader: Hosted x-originating-ip: [10.202.205.107] MIME-Version: 1.0 X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: aculab.com Content-Language: en-US Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8BIT X-Spam-Status: No, score=-1.7 required=5.0 tests=BAYES_00,PDS_BAD_THREAD_QP_64, RCVD_IN_DNSWL_NONE,SPF_HELO_NONE,SPF_PASS autolearn=no autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org From: Andrew Lunn > Sent: 30 November 2022 21:33 ... > > That won't work for this board since that function only checks that the > > MAC "is not 00:00:00:00:00:00, is not a multicast address, and is not > > FF:FF:FF:FF:FF:FF." The MAC address that we get on all of our boards is > > 00:17:b6:00:00:00. > > Which is a valid MAC address. So i don't see why the kernel should > reject it and use a random one. It isn't very valid... The first three bytes are the mulicast, local and company bits. So the last three bytes being zero indicate you have the very first address the company allocated. Pretty much zero chance of that board ever working well enough to be in a system. David - Registered Address Lakeside, Bramley Road, Mount Farm, Milton Keynes, MK1 1PT, UK Registration No: 1397386 (Wales)