From: Jakub Kicinski <kuba@kernel.org>
To: Paolo Abeni <pabeni@redhat.com>
Cc: Alexander Duyck <alexander.duyck@gmail.com>,
netdev@vger.kernel.org, linux@armlinux.org.uk,
hkallweit1@gmail.com, andrew+netdev@lunn.ch, davem@davemloft.net,
kernel-team@meta.com, edumazet@google.com
Subject: Re: [net-next PATCH v3 0/8] Add support for 25G, 50G, and 100G to fbnic
Date: Fri, 20 Jun 2025 07:36:02 -0700 [thread overview]
Message-ID: <20250620073602.5ea8ea6c@kernel.org> (raw)
In-Reply-To: <5ce8c769-6c36-4d0a-831d-e8edab830beb@redhat.com>
On Thu, 19 Jun 2025 10:44:41 +0200 Paolo Abeni wrote:
> Apparently this is causing TSO tests
I had a closer look, I think the TSO test case is always failing,
the test that didn't fail before but fails now is the test for
pause statistics. Since the driver didn't support pause config
before the test returned XFAIL, now there is pause but no stats
so it fails. I guess we may have been better off using SKIP rather
than XFAIL in the test case, so that the CI doesn't consider lack
of results a pass.
next prev parent reply other threads:[~2025-06-20 14:36 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-06-18 22:07 [net-next PATCH v3 0/8] Add support for 25G, 50G, and 100G to fbnic Alexander Duyck
2025-06-18 22:07 ` [net-next PATCH v3 1/8] net: phy: Add interface types for 50G and 100G Alexander Duyck
2025-06-18 22:07 ` [net-next PATCH v3 2/8] fbnic: Do not consider mailbox "initialized" until we have verified fw version Alexander Duyck
2025-06-18 22:07 ` [net-next PATCH v3 3/8] fbnic: Retire "AUTO" flags and cleanup handling of FW link settings Alexander Duyck
2025-06-18 22:07 ` [net-next PATCH v3 4/8] fbnic: Replace link_mode with AUI Alexander Duyck
2025-06-18 22:07 ` [net-next PATCH v3 5/8] fbnic: Update FW link mode values to represent actual link modes Alexander Duyck
2025-06-18 22:07 ` [net-next PATCH v3 6/8] fbnic: Set correct supported modes and speeds based on FW setting Alexander Duyck
2025-06-18 22:08 ` [net-next PATCH v3 7/8] fbnic: Add support for reporting link config Alexander Duyck
2025-06-18 22:08 ` [net-next PATCH v3 8/8] fbnic: Add support for setting/getting pause configuration Alexander Duyck
2025-06-19 8:44 ` [net-next PATCH v3 0/8] Add support for 25G, 50G, and 100G to fbnic Paolo Abeni
2025-06-19 15:17 ` Jakub Kicinski
2025-06-20 14:36 ` Jakub Kicinski [this message]
2025-06-24 7:28 ` Paolo Abeni
2025-06-24 7:50 ` patchwork-bot+netdevbpf
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20250620073602.5ea8ea6c@kernel.org \
--to=kuba@kernel.org \
--cc=alexander.duyck@gmail.com \
--cc=andrew+netdev@lunn.ch \
--cc=davem@davemloft.net \
--cc=edumazet@google.com \
--cc=hkallweit1@gmail.com \
--cc=kernel-team@meta.com \
--cc=linux@armlinux.org.uk \
--cc=netdev@vger.kernel.org \
--cc=pabeni@redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).