From nobody Thu Aug 24 06:17:43 2023 X-Original-To: dev-commits-src-all@mlmmj.nyi.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mlmmj.nyi.freebsd.org (Postfix) with ESMTP id 4RWXv851Ybz4qgrF for ; Thu, 24 Aug 2023 06:17:56 +0000 (UTC) (envelope-from kevin.bowling@kev009.com) Received: from mail-pf1-x433.google.com (mail-pf1-x433.google.com [IPv6:2607:f8b0:4864:20::433]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256 client-signature RSA-PSS (2048 bits) client-digest SHA256) (Client CN "smtp.gmail.com", Issuer "GTS CA 1D4" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 4RWXv83ng1z4WKS for ; Thu, 24 Aug 2023 06:17:56 +0000 (UTC) (envelope-from kevin.bowling@kev009.com) Authentication-Results: mx1.freebsd.org; none Received: by mail-pf1-x433.google.com with SMTP id d2e1a72fcca58-68bed28cb22so411057b3a.2 for ; Wed, 23 Aug 2023 23:17:56 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=kev009.com; s=google; t=1692857875; x=1693462675; h=content-transfer-encoding:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:from:to:cc:subject:date :message-id:reply-to; bh=ohd+vLxALplnUfzMkIjBJRlvc3kUFdef5B6eak1qSY0=; b=gs8+hSn1gs1JMuCMOXxHa2tgdDRjUEbWhxoemCirU8H4GfIkLwcjos91x2RaVy9I5z w/p9U0yfZmOosh8zbg6uKC6aAtt/efQ9Pms/d/AEUp53yojshjfuzZIbI2+9Co77PUfF WMURjgLeUFsfG6Mi/9GTboGqkCqumjEztnRws= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1692857875; x=1693462675; h=content-transfer-encoding:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=ohd+vLxALplnUfzMkIjBJRlvc3kUFdef5B6eak1qSY0=; b=JvWhe4lcWQo4QydWTV/1RObLFYkGdto6Slr6WS84lJKYHP4RK0q9cYzsTrCQ8nXEMu ztYmzcE0QPAQyGVC8r9OFpUm/zxVhvgUBh5QC5xSP+SwrJjsVaH1v0zwysWLEq4UpIKd 2m3oUoTjga6s/5ssI3NDIaVrKOYVh3iA22p8z9hvKk50kXL0PgeoDs5qu1Jb5amkKY09 oZO2fIlnGXmHWyjfi9OVQybuIT/o8knz4UNlCRkJOag+814G7S8NkAYdDNRCfKgvQHDC NjmHCt7kk0RSPCQr6037WWb1HKPBHXop0G3lTxEyhVfJXy3utlVE1e5OJ9GxMZAGyq9N srng== X-Gm-Message-State: AOJu0Yz2y/OVyk5YYhHja1QDEPSYAlo2XFitMv+6VHK7Cn9Lba5WPbQx liSdgr40iVkPZV3XteazvuEsHuS3ayphxhwR6FfC4w== X-Google-Smtp-Source: AGHT+IEHxGXvduan6It0BdXqZxXGOPX/pqGe2vyvtMh04E8/NKBIVO/8xkgGiye7LSxLtiq0dbfz3phid3kdeac77yg= X-Received: by 2002:a05:6a00:22c1:b0:689:f13a:5eb5 with SMTP id f1-20020a056a0022c100b00689f13a5eb5mr14718147pfj.28.1692857874777; Wed, 23 Aug 2023 23:17:54 -0700 (PDT) List-Id: Commit messages for all branches of the src repository List-Archive: https://lists.freebsd.org/archives/dev-commits-src-all List-Help: List-Post: List-Subscribe: List-Unsubscribe: Sender: owner-dev-commits-src-all@freebsd.org X-BeenThere: dev-commits-src-all@freebsd.org MIME-Version: 1.0 References: <202211042255.2A4MtqVv032693@gitrepo.freebsd.org> <419530F4-DB8F-46EA-BF45-C130BCF8A3DE@FreeBSD.org> <05979953-C089-4D56-99E3-BBBF066FE277@FreeBSD.org> In-Reply-To: From: Kevin Bowling Date: Wed, 23 Aug 2023 23:17:43 -0700 Message-ID: Subject: Re: git: 91fcacc35597 - main - if_bnxt: Add support for VLAN on Thor To: Sumit Saxena Cc: Warner Losh , Kristof Provost , Warner Losh , "" , "" , src-committers , Chandrakanth Patil Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable X-Rspamd-Queue-Id: 4RWXv83ng1z4WKS X-Spamd-Bar: ---- X-Rspamd-Pre-Result: action=no action; module=replies; Message is reply to one we originated X-Spamd-Result: default: False [-4.00 / 15.00]; REPLY(-4.00)[]; ASN(0.00)[asn:15169, ipnet:2607:f8b0::/32, country:US] Kristof, Sumit, Chandrakanth, I have posted an update to https://reviews.freebsd.org/D41558 with more details about why this re-init was happening and changing the defaults to match the expected behavior when this driver was written. Note that this review does not address the initialization issues within bnxt (which MarkJ details in https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=3D269133). It will reduce the impact of the bug in some common usages and is a necessary improvement. On Wed, Aug 23, 2023 at 12:18=E2=80=AFAM Sumit Saxena wrote: > > +Chandrakanth Patil > > Chandrakanth is working on this bug. > > -Sumit > > On Wed, Aug 23, 2023 at 6:01=E2=80=AFAM Warner Losh wrot= e: > > > > > > > > On Tue, Aug 22, 2023, 6:26 PM Warner Losh wrote: > >> > >> > >> > >> On Tue, Aug 22, 2023, 5:59 PM Kevin Bowling = wrote: > >>> > >>> On Tue, Aug 22, 2023 at 3:39=E2=80=AFPM Kevin Bowling wrote: > >>> > > >>> > > >>> > > >>> > On Tue, Aug 22, 2023 at 2:07 PM Kristof Provost wr= ote: > >>> >> > >>> >> On 27 Jun 2023, at 11:17, Kristof Provost wrote: > >>> >> > >>> >> On 4 Nov 2022, at 23:55, Warner Losh wrote: > >>> >> > >>> >> The branch main has been updated by imp: > >>> >> > >>> >> URL: https://cgit.FreeBSD.org/src/commit/?id=3D91fcacc355971f74aa2= 6fc7861020dc3a2a2d717 > >>> >> > >>> >> commit 91fcacc355971f74aa26fc7861020dc3a2a2d717 > >>> >> Author: Sumit Saxena > >>> >> AuthorDate: 2022-11-04 22:24:32 +0000 > >>> >> Commit: Warner Losh > >>> >> CommitDate: 2022-11-04 22:24:32 +0000 > >>> >> > >>> >> if_bnxt: Add support for VLAN on Thor > >>> >> > >>> >> Reviewed by: imp > >>> >> Differential Revision: https://reviews.freebsd.org/D36443 > >>> >> --- > >>> >> sys/dev/bnxt/bnxt.h | 3 +- > >>> >> sys/dev/bnxt/bnxt_hwrm.c | 110 ++++++++++++++++++++++++++++-------= ------------ > >>> >> sys/dev/bnxt/bnxt_hwrm.h | 7 ++- > >>> >> sys/dev/bnxt/if_bnxt.c | 7 ++- > >>> >> 4 files changed, 74 insertions(+), 53 deletions(-) > >>> >> > >>> >> This commit appears to have broken vlan on these interfaces. > >>> >> My hardware is a Broadcom BCM57416 NetXtreme-E 10GBase-T Ethernet,= (in a Dell T640, if that helps). > >>> >> > >>> >> A simple vlan creation on top of one: > >>> >> ifconfig vlan create > >>> >> ifconfig vlan0 vlan 201 vlandev bnxt0 > >>> >> > >>> >> Results in a loss of connectivity. The kernel logs this, which I a= ssume is related: > >>> >> > >>> >> `bnxt0: HWRM_CFA_L2_FILTER_ALLOC command returned INVALID_PARAMS e= rror.` > >>> >> > >>> >> After reverting this (and 72e9dbb58cad5262190cf2eae47f764021072128= ) I can create vlan interfaces on top of bnxt0 without losing connectivity. > >>> >> > >>> >> > >>> >> Should we revert this before 14.0? > >>> >> > >>> >> There=E2=80=99s also a report of what I think is the same issue in= https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=3D269133 > >>> >> > >>> >> There=E2=80=99s a proposed patch there, but it did not work for me= . > >>> > > >>> > > >>> > In markj=E2=80=99s analysis he mentions the issue of the reinit. > >>> > > >>> > It sounds like there are multiple bugs in this driver but one quick= band aid may be to mark the driver as not needing re-init for VLAN modific= ations.. that sounds unlikely and was probably an accident when the if rese= t functionality was added. You can check e1000 for that. > >>> > >>> Here's what I mean as a patch https://reviews.freebsd.org/D41558 > >>> > >>> > Note I don=E2=80=99t have this hw and haven=E2=80=99t audited the c= ode so this could be noise but it=E2=80=99s an easy thing to check. > >> > >> > >> Is there a good bug we can post that link to for someone to test? > > > > > > Duh.. posted to bug above and > > https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=3D222680 > > > > Warner > > > >> > >> Warner > >>> > >>> > > >>> >> Best regards, > >>> >> Kristof