From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-yw1-f178.google.com (mail-yw1-f178.google.com [209.85.128.178]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id A987D2ED872 for ; Thu, 9 Jul 2026 01:24:37 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.178 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1783560279; cv=none; b=OPNwR6vNMwQ7JGXDwsPzgtZYYKfy9Ef67fdjdDXcoA+GJbAYkP1SGNDFd7a3eZ5zCkdK5mL+zAzo9wY/Fgc2pua8eau3Wx3wnsQNeB7j/V8HesmAVNirJicmd72ihFNvClxqTD4cY/c1651Dt4QgVfx3ajNtX2xSaJrhysxufi0= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1783560279; c=relaxed/simple; bh=l7eHw/raYhzCKyoQvZO5b5gtYbr1anwl7xluE4HEstQ=; h=Date:From:To:Cc:Message-ID:In-Reply-To:References:Subject: Mime-Version:Content-Type; b=d3k5eLCuHTrzv+zx0uW7XBn0Q22CInEkL0FdQqwx/+G1YCQLG8YlRS7Yu7Mbq+rUkIqAgbU3vsQQQh8n/XSphyzqHYc7gcq2nQ8PmnNWkGb0SRwva8C9p5+rirs7ezRbvB62A7XaJjH/tY1tj3X9CbQUBs1pLsZohbrfHzhs2iQ= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=QggIFKPx; arc=none smtp.client-ip=209.85.128.178 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="QggIFKPx" Received: by mail-yw1-f178.google.com with SMTP id 00721157ae682-80bb578d58bso13987667b3.0 for ; Wed, 08 Jul 2026 18:24:37 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1783560277; x=1784165077; darn=vger.kernel.org; h=content-transfer-encoding:content-type:mime-version:subject :references:in-reply-to:message-id:cc:to:from:date:from:to:cc :subject:date:message-id:reply-to:content-type; bh=l7eHw/raYhzCKyoQvZO5b5gtYbr1anwl7xluE4HEstQ=; b=QggIFKPxJXqU+hgCDM8CbHi6qty1kxvO+hDKATtwDl0ALTj0vuyF3wyaS2piKuxSz3 KpkBHSIPtmpA8MGK7+G/wu+KjXcey9BeSNJ+zixcDTNeoK9CKNFZFCCtkw010krVemtP ZzLjyUeqBTbOIQPQlmE7e8/rxp54IPeEi1dFJro9oyoARm2QoMAezUrNeacurygnd0nw KDP21kf/GEW+w0WCS+XDi8mwqAGNNAbTIqNzGhMEC8lr53bFwwaLGWYGCAHfaZx6SBng O25F6CjKlI+y/V5Deic0wMCn1/l7AvaROyw5aCa9uXZTPLRk/ZH/TE4c/Wauz35vRcHQ j7jA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1783560277; x=1784165077; h=content-transfer-encoding:content-type:mime-version:subject :references:in-reply-to:message-id:cc:to:from:date:x-gm-gg :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to :content-type; bh=l7eHw/raYhzCKyoQvZO5b5gtYbr1anwl7xluE4HEstQ=; b=Hmv4lCVHkJxfBN2yqzod2SoWq+6Pw1vV+7e87gqQRBSHabBFdLt8xsw/BRso6+jBVj R0CADIonRkQ+iCXUifk5E6T26OLquV/D3NLhK9iNX7SRMmfY0n3KUsCMHWwbSPZLJTCy Hb/dzxYci2U5uBLp9sfQjYlaU2IQarL0JV96avoCTMhZhknQhLrliQqs98FXbFVMMoJ/ o0MOSt9yUfmUqCCI3pg5pxVcBLRfqX3sK37ZYrZjDgqrYlGQ9J7Sa3G5tWZsoKuePfcT AejxjYcyALMql+GfoS6+0/OPkdqkhXbK+rxH4C2TyRyMcZnOn0Q3q0br8fDMhttY/IWa gfHA== X-Forwarded-Encrypted: i=1; AHgh+RozXNdwpKC61+KHhK0/09Zoy3ZvCcvKC3mOJePIwFnDKkxzTnnqZ8dzzEcyuuhlncjBczafuCZerSsdXck=@vger.kernel.org X-Gm-Message-State: AOJu0Yzb61HomrSgguY7iiWdZ3jeAh7THcFdJUH8KALmREjJsNmQyhHi wwpefF43blsrYRUcQeaj9xlgUGziLfIzb5Igs5Rk8DWik2H2VvBnutZU X-Gm-Gg: AfdE7cll12Gu2WGbVU9Zj6jJbMkVY+O3/nj6MpVRHOwr7Ye/xLAhilACiskTnHE+9cI CI5DiM6I1SkPNPiAJhKIkwkrb9Nyi6SiNgZHeniGPFHZEBqdzt5SzZ/OY9ipJjkXxXPM/io/7/n acUd7q6Mg4uzWCMXLkyliGHzXm+PpC1iyn37asrpsmn61fQkgRXaix3v+eqzmD5i+lTETAw35sQ Wf85RUdqJNmHLccqFXLknb83BUrx2D02U8awXjM8MRYhjOiJQ9AV1eNup0nkB8eu14X8lMaOSzw TbjJNvzsxhrd1qypXcU+Ip0ZxymvW2IOBRsJufvQjVfSVASW8pyTb6paNq4fSN7iqz+GQV/CaOk B9uBd9Ub92k3LvEUnkpsjkWwMLoiMjvoG6QDBDZ5VGqco82zyE0SnjkJRtK5/hJz99nvcDrfzLZ njHqwnPYalAS/lBSsLSZByH0E0RFJgxkI5OKIuE6873V3BsSsXY6y26JSe7P1pITYttg== X-Received: by 2002:a05:690c:12:b0:81d:4f4b:94be with SMTP id 00721157ae682-81dbd41c1a7mr37863157b3.32.1783560276655; Wed, 08 Jul 2026 18:24:36 -0700 (PDT) Received: from gmail.com (172.235.85.34.bc.googleusercontent.com. [34.85.235.172]) by smtp.gmail.com with ESMTPSA id 00721157ae682-81e6be99124sm6315537b3.4.2026.07.08.18.24.36 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 08 Jul 2026 18:24:36 -0700 (PDT) Date: Wed, 08 Jul 2026 21:24:35 -0400 From: Willem de Bruijn To: =?UTF-8?B?U2hpbWluZyBDaGVuZyAo5oiQ6K+X5piOKQ==?= , "linux-kernel@vger.kernel.org" , "dsahern@kernel.org" , "imv4bel@gmail.com" , "linux-mediatek@lists.infradead.org" , "alice@isovalent.com" , "daniel.zahka@gmail.com" , "eilaimemedsnaimel@gmail.com" , "nbd@nbd.name" , "horms@kernel.org" , "kuba@kernel.org" , "pabeni@redhat.com" , "edumazet@google.com" , "willemdebruijn.kernel@gmail.com" , "willemb@google.com" , "netdev@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , "matthias.bgg@gmail.com" , "davem@davemloft.net" , AngeloGioacchino Del Regno , "sd@queasysnail.net" Cc: "steffen.klassert@secunet.com" , "stable@vger.kernel.org" , =?UTF-8?B?TGVuYSBXYW5nICjnjovlqJwp?= Message-ID: In-Reply-To: <2d71af40897d73dbd9e243ce5e25bbd3f99acc5d.camel@mediatek.com> References: <20260707021425.483-1-shiming.cheng@mediatek.com> <2d71af40897d73dbd9e243ce5e25bbd3f99acc5d.camel@mediatek.com> Subject: Re: [PATCH v6] net: gro: fix double aggregation of flush-marked skbs Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Shiming Cheng (=E6=88=90=E8=AF=97=E6=98=8E) wrote: > On Tue, 2026-07-07 at 11:16 -0400, Willem de Bruijn wrote: > > External email : Please do not click links or open attachments until > > you have verified the sender or the content. > > = > > = > > Shiming Cheng wrote: > > > The skb_gro_receive_list() function is missing a critical safety > > > check > > > that exists in the skb_gro_receive() implementation. Specifically, > > > it > > > does not validate NAPI_GRO_CB(skb)->flush before allowing packet > > > aggregation, as of commit 0ab03f353d36 ("net-gro: Fix GRO flush > > > when receiving a GSO packet."). > > = > > It does not check .. as of commit .. ? > > = > > No, skb_gro_receive checkos NAP_GRO_CB(skb)->flush as of that commit.= > > = > = > Is this wording okay? > = > Commit 0ab03f353d36 ("net-gro: Fix GRO flush when receiving a GSO > packet.") added a flush check to skb_gro_receive(), but > skb_gro_receive_list() lacks the same validation. > = > As a result, packets marked with NAPI_GRO_CB(skb)->flush may still be > re-aggregated. That sounds good to me, thanks. =