From nobody Thu Oct 01 07:25:41 2026 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 4hwNk35tjKz6v9vb for ; Thu, 01 Oct 2026 07:25:47 +0000 (UTC) (envelope-from git@FreeBSD.org) Received: from mxrelay.nyi.freebsd.org (mxrelay.nyi.freebsd.org [IPv6:2610:1c1:1:606c::19:3]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange x25519 server-signature RSA-PSS (4096 bits) server-digest SHA256 client-signature RSA-PSS (4096 bits) client-digest SHA256) (Client CN "mxrelay.nyi.freebsd.org", Issuer "YR2" (not verified)) by mx1.freebsd.org (Postfix) with ESMTPS id 4hwNk30SDTz4VNP for ; Thu, 01 Oct 2026 07:25:47 +0000 (UTC) (envelope-from git@FreeBSD.org) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=freebsd.org; s=dkim; t=1790839547; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding; bh=P51OcKhBXKK5ueoamtJLcL2VyvRbeDtx8UufjukJU54=; b=pYwqsnClJ8Ri7qAluwC+r8eOcUNGfFZ2uQRyni4qcOHg3Z0tRNbPH07cLMFa3DRUzJBTwS 0VOKtn3a52dRA37SOZ+Y+3TatLPULonP8yAJ3r/Ydkcj3yG//AFXVM5TxiYHX/8ndleG+P nnopq7DsE8Y0BujYN9OoEnZLSxKcvrKTu7cIdyE4dAj7spnRM/LTFrGMYo24evEYiUZv7u BY73jlVZUA3t0Eek73aVLuAIPI+7LF4mido5iRRgOUzASsS5zeaczYfphyuEOn1WUC16Eo zRySjkYITKLQeiYQ5x3O/q7/xP21TVhlWVHTx6AjedruCUqW97J2krhL+ZsBfA== ARC-Seal: i=1; a=rsa-sha256; d=freebsd.org; s=dkim; cv=none; t=1790839547; b=vPBErDfg68NY6XVbCPDCWaWE4EvAWkT4XeyVNrBV24nOsIyfl5C0JzzTDEyi9ly5roMcsD 9+WQTy6JelaV4pKnOdkdqEmG3h+MP90/ZuONJ1TSeplSL7ambqJCbaPIGxG9VnNDNLy+7d G1klDKFeYhai4TcxCzvNWyLP3g99oBg79iV1JWk10pm8u4nd6Qk9sqeHM+P6EeB0yg+gf6 7ILefR9AZpj/N11bN1AWk3E7sawptrYlknFfvzRjBAhrcIsHITWnF/9Jt2XBpD0HcXh1pN PFmx46eZqeM9zrwqUUfjOEoRo7LbH0aTJSbXSVFOFyj85c35h7i+9PdqlDdRIg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=freebsd.org; s=dkim; t=1790839547; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding; bh=P51OcKhBXKK5ueoamtJLcL2VyvRbeDtx8UufjukJU54=; b=ht2ng7IcSrvElrOljBg5JfC+xR7kDDVXlwT8Fk0i7HA5Ui4QW55fj6fmQqKcU0cuqkq+U3 nQiAYgVLuEMwy+qjkZVOyfjRrVUBBsYjz6aKVrbtdnkF8pRijsUwXfslqEgU3Fnpv2Ia1D RVHZrOpNlZrGGmvasYxb7yqGQFDVotav22ByB5edIObTrFvuPEaA7gXNjcPYt17bIQRa3b S264B18yvs855JOH27DMlKmePYQnK0CTIkDF7NxDBg5rc152s/g/+yRa2CZ0kfYmRtJdjY ciNnWZ8TIreLkoN5YH1GuVnMAjZvVEHkZClrM8u4PpuTKee9+OLllKuZJP67dw== ARC-Authentication-Results: i=1; mx1.freebsd.org; none Received: from gitrepo.freebsd.org (gitrepo.freebsd.org [IPv6:2610:1c1:1:6068::e6a:5]) by mxrelay.nyi.freebsd.org (Postfix) with ESMTP id 4hwNk26M6RzMB2 for ; Thu, 01 Oct 2026 07:25:46 +0000 (UTC) (envelope-from git@FreeBSD.org) Received: from git (uid 1279) (envelope-from git@FreeBSD.org) id 21112 by gitrepo.freebsd.org (DragonFly Mail Agent v0.13+ on gitrepo.freebsd.org); Thu, 01 Oct 2026 07:25:41 +0000 To: src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-branches@FreeBSD.org From: Piotr Kubaj Subject: git: a9103f8f07cb - stable/15 - llan: byte swap the receive queue entries 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: X-BeenThere: dev-commits-src-all@freebsd.org Sender: owner-dev-commits-src-all@FreeBSD.org List-Id: List-Post: List-Help: List-Subscribe: List-Unsubscribe: List-Owner: Precedence: list MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit X-Git-Committer: pkubaj X-Git-Repository: src X-Git-Refname: refs/heads/stable/15 X-Git-Reftype: branch X-Git-Commit: a9103f8f07cbdd5c2f69475b053fd41b31b1425d Auto-Submitted: auto-generated Date: Thu, 01 Oct 2026 07:25:41 +0000 Message-Id: <6abe0af5.21112.14ae6f45@gitrepo.freebsd.org> The branch stable/15 has been updated by pkubaj: URL: https://cgit.FreeBSD.org/src/commit/?id=a9103f8f07cbdd5c2f69475b053fd41b31b1425d commit a9103f8f07cbdd5c2f69475b053fd41b31b1425d Author: Piotr Kubaj AuthorDate: 2026-09-23 13:21:00 +0000 Commit: Piotr Kubaj CommitDate: 2026-10-01 07:25:35 +0000 llan: byte swap the receive queue entries The receive queue of a PAPR logical LAN is filled in by the hypervisor, so its fields are big endian, but llan_intr() read the offset and the length of each frame natively. On a little endian kernel the length of a 134 byte frame reads as 0x86000000, and ether_input() discards the mbuf because m_len is not even large enough for an ethernet header. No frame is ever received. Reproduced on a POWER9 pseries guest with a spapr-vlan interface. Before: llan0: discard frame w/o leading ethernet header (len -2046820352 pkt len -2046820352) llan0 1500 52:54:00:12:34:56 123 118 0 5838733312 7 0 that is 118 input errors out of 123 packets, dhclient(8) never completes and ping(8) loses every packet, although transmit works because the transmit path passes the lengths in hcall registers rather than through memory. Afterwards the interface gets a DHCP lease and ping reports no loss. MFC after: 1 week Differential Revision: https://reviews.freebsd.org/D59926 Approved by: jhibbits (cherry picked from commit 6429199a21136cad3c82b43618a6ca2958b90bbf) --- sys/powerpc/pseries/phyp_llan.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/sys/powerpc/pseries/phyp_llan.c b/sys/powerpc/pseries/phyp_llan.c index 763eebc15b36..86ad7dc96fb7 100644 --- a/sys/powerpc/pseries/phyp_llan.c +++ b/sys/powerpc/pseries/phyp_llan.c @@ -385,8 +385,8 @@ restart: while ((sc->rx_buf[sc->rx_dma_slot].control >> 7) == sc->rx_valid_val) { rx = (struct llan_xfer *)sc->rx_buf[sc->rx_dma_slot].handle; m = rx->rx_mbuf; - m_adj(m, sc->rx_buf[sc->rx_dma_slot].offset - 8); - m->m_len = sc->rx_buf[sc->rx_dma_slot].length; + m_adj(m, be16toh(sc->rx_buf[sc->rx_dma_slot].offset) - 8); + m->m_len = be32toh(sc->rx_buf[sc->rx_dma_slot].length); /* llan_add_rxbuf does DMA sync and unload as well as requeue */ if (llan_add_rxbuf(sc, rx) != 0) { @@ -395,8 +395,8 @@ restart: } if_inc_counter(sc->ifp, IFCOUNTER_IPACKETS, 1); - m_adj(m, sc->rx_buf[sc->rx_dma_slot].offset); - m->m_len = sc->rx_buf[sc->rx_dma_slot].length; + m_adj(m, be16toh(sc->rx_buf[sc->rx_dma_slot].offset)); + m->m_len = be32toh(sc->rx_buf[sc->rx_dma_slot].length); m->m_pkthdr.rcvif = sc->ifp; m->m_pkthdr.len = m->m_len; sc->rx_dma_slot++;