From mboxrd@z Thu Jan 1 00:00:00 1970 Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.remote-ip=209.85.214.199 ARC-Seal: i=1; d=subspace.kernel.org; s=arc20261015; a=rsa-sha256; cv=none; t=1793683774; b=uQW44vCWpohejbGTsfZt/jyPDpPnMuPp1/xb+8KTlpJXrC2RMVjPQvbQPTlq/KK1v+hz WJcE6+P77jxr03VYxVIkL8ODx4LyuV5VvkFVVRGSTxBIvtci+6S1uhjoSrGUjhEUsgnCh SxIeDJiCkbeIDC78zvj2FLLje5mRvih4qosxN+YpHUP0tFZdwdinv24pmDpq8DCBzhMVt P+Gx/dnj6aienBBXyP5bgrNARq2PfueDwEYyepS/OlukSgB8KE6F7ZFIpbNiKkudzVhPN g2O+lV6AH/EzzP2jvhtgQMqKF3I5gp1yss5ZTuV6oE9tWdFFtnnIZzogUagINnRSh7Q== ARC-Message-Signature: i=1; d=subspace.kernel.org; s=arc20261015; a=rsa-sha256; c=relaxed/relaxed; t=1793683774; h=DMARC-Filter:Received:DKIM-Signature:X-Google-DKIM-Signature: X-Forwarded-Encrypted:X-Gm-Message-State:X-Received:Date:In-Reply-To: Mime-Version:References:Message-ID:Subject:From:To:Cc:Content-Type; bh=ke9vhkENAXPfCm1yxHfr+SbOqiszin1sS4qIXy02ijk=; b=SewObLPuJW4pMkqM6Z393PQTM9tBF66ntCimkx/jaqcH8dd2CuazYBvQV+BsomUyYGrC 8aUX9AYASKivr/5/DEsUubxxWh4CuayQpz0j9R3NMq6RRMvd8GvVxBDIk6ZmunN2HZ5Qk ppt6uYE3W8R9PMaui0/OciNTCNVllm3SaucwjbU0gltsYfikN76+VQgkTknwBUieZ0UcW Wh2jWhai6vhsE1P0voNEJHqunDvVDFSvFTIPH/Nj3tyM2ZR1RlRa8veN160QkW8Qo4M/h F9UPVUGOpiqW5UC03FSxMpKMGfPO2qwBFjBfshH0GcUwVgYYMaJlz+UP4MtdciMBH6w== ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass header.from=google.com; spf=pass smtp.mailfrom=flex--kuniyu.bounces.google.com; dkim=pass header.d=google.com header.i=@google.com header.a=rsa-sha256 header.s=20260707 header.b=UmzAFCtx; arc=none smtp.remote-ip=209.85.214.199 Received: from mail-pl1-f199.google.com (mail-pl1-f199.google.com [209.85.214.199]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits) key-exchange X25519 server-signature ECDSA (prime256v1) server-digest SHA256) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id EFF9A606A33C for ; Tue, 3 Nov 2026 05:29:33 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.4.2 smtp.subspace.kernel.org EFF9A606A33C Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=google.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=flex--kuniyu.bounces.google.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key, unprotected) header.d=google.com header.i=@google.com header.a=rsa-sha256 header.s=20260707 header.b=UmzAFCtx Received: by mail-pl1-f199.google.com with SMTP id d9443c01a7336-2e9572e892dso26853565ad.1 for ; Mon, 02 Nov 2026 21:29:33 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20260707; t=1793683773; x=1794288573; darn=vger.kernel.org; h=content-type:cc:to:from:subject:message-id:references:mime-version :in-reply-to:date:from:to:cc:subject:date:message-id:reply-to :content-type; bh=ke9vhkENAXPfCm1yxHfr+SbOqiszin1sS4qIXy02ijk=; b=UmzAFCtxp1ZYyFABxl1CisByc7Hr2e969siZsRVUO5jvr9m5lyMi3dsG2NZaa6DbkS Q8filMC2M4hVX6GaGHy3WskJo5Ss81YXxBmHv8Gi3cYWgKeSPF/RHDi5+dirS5fixcse lmlGFAnXepRzUL004qTIZUB77qN5VCnadqWy0pYWfUhEFZZs52UGP/bbyIqea7T6Z9vn DhELoXdUTRArq8p4XtGXSWvwgHHdcbHZOXSSL+egm063O5C4EG5katn0QWTJQvViSFD3 fikNYAgztw0YQOL9Nb7azqmVcP8aky6HDdQz0uv/e0/YqjmnYm/mNXTh+9aJgyQVZzM1 YQeQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20260707; t=1793683773; x=1794288573; h=content-type:cc:to:from:subject:message-id:references:mime-version :in-reply-to:date:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to:content-type; bh=ke9vhkENAXPfCm1yxHfr+SbOqiszin1sS4qIXy02ijk=; b=lcr90TK6+rTnLVixnAM8eYk2LVMwLOtmdJcIVSRg2WxQj3+6OxL5AZwz7vZL2tu62a V623Kt5BI8hGC9ZX+jlQUn7UTWrVqeQ9zu8syKYGpyeNQfj0EyQYKKG2P9C6mcyydH6R 8+E2+4fXFJ25DFRCRb9TzGxx9rtxGiTZ3xDarixWZeyPcZdIdscRayn07x+xVuhchIgj 0D3zliJYVEFY3bjCo4L5Vj/GmUVtN+MLi3BDSbwSds/kXLxYo+9E/FAPiZAVBU4loxR/ x5Uzz8xCEWGNgaVuyo1e8vxZUXliHdaZNgmC89/79GM3xP+wJxeS0LQJE9b8OtyXhMpU 4NSg== X-Forwarded-Encrypted: i=1; AKwUvBxei3k4kW6nfbrRXm1AeizLSc0EwxS3dRZ2JP84kXaDTskUOG2Eg81ZAjir+GjF7iblzg3LXvkDe6/N7tQ=@vger.kernel.org X-Gm-Message-State: APuU4Z880znF1CiDoq912d0R1N+DD8k3nFECqjFwnABW1lCw7ow+Hh3Z x3oSoys8U2UcX3PYUSDhpNNvjkDhZO5GjMveBp75td4aSCwjfdVClqwUqK7X/0oSv4h9u8JdYGo XZK8Rpw== X-Received: from plbbb6.prod.google.com ([2002:a17:902:bc86:b0:2e8:18c6:a093]) (user=kuniyu job=prod-delivery.src-stubby-dispatcher) by 2002:a17:902:f712:b0:2e9:5829:9b40 with SMTP id d9443c01a7336-2eb3649be7emr111118405ad.9.1793683772335; Mon, 02 Nov 2026 21:29:32 -0800 (PST) Date: Tue, 3 Nov 2026 05:29:31 +0000 In-Reply-To: <20261101061331.286404-5-jiayuan.chen@linux.dev> X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Post: List-Help: List-Subscribe: List-Unsubscribe: List-Owner: Precedence: list Mime-Version: 1.0 References: <20261101061331.286404-1-jiayuan.chen@linux.dev> <20261101061331.286404-5-jiayuan.chen@linux.dev> Message-ID: Subject: Re: [PATCH net v3 4/5] ipv4: restrict igmp_qrv to 255 From: Kuniyuki Iwashima To: Jiayuan Chen Cc: netdev@vger.kernel.org, stable@vger.kernel.org, Fernando Fernandez Mancera , "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Simon Horman , Jonathan Corbet , Shuah Khan , Randy Dunlap , David Ahern , Ido Schimmel , Joel Granados , Thomas Gleixner , Petr Pavlu , Maoyi Xie , Jiri Wiesner , Oleg Nesterov , Gerd Rausch , Neal Cardwell , Chia-Yu Chang , Yung Chih Su , Wyatt Feng , Hannes Frederic Sowa , Flavio Leitner , "Paul E. McKenney" , Petr Mladek , linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org Content-Type: text/plain; charset="us-ascii" On Sun, 01 Nov 2026 14:12:02 +0800, Jiayuan Chen wrote: > net.ipv4.igmp_qrv only has a lower bound, but ip_mc_reset() stores it > into the u8 in_device::mr_qrv, and the IGMPv3 retransmit counters filled > from it (crcount, sf_crcount) are u8 as well. Anything above 255 > silently wraps. With 256, mr_qrv becomes 0, every > "in_dev->mr_qrv ?: sysctl_igmp_qrv" falls back to 256, which truncates > to 0 again in crcount, and igmpv3_send_cr() never puts the state change > records for the group into a report. The older version querier present > timeout (mr_qrv * mr_qi + mr_qri) also collapses to just mr_qri. > > Make the sysctl a u8 like mr_qrv and switch to proc_dou8vec_minmax(), > which rejects anything above 255, and document it. > > Fixes: a9fe8e29945d ("ipv4: implement igmp_qrv sysctl to tune igmp robustness variable") > Cc: stable@vger.kernel.org > Reviewed-by: Fernando Fernandez Mancera > Signed-off-by: Jiayuan Chen Reviewed-by: Kuniyuki Iwashima