From nobody Mon Feb 9 04:13:56 2026 Received: from mail-ot1-f51.google.com (mail-ot1-f51.google.com [209.85.210.51]) (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 B05072D0616 for ; Mon, 22 Dec 2025 14:52:29 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.210.51 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1766415153; cv=none; b=VqamC/KjXV+o7CFmKhbsl5ZCMYa+Zon79jH7fyLPBsDZhYCj7DbTTrB65vxnc7ndvbai/wbzne9xRDyXzmS+fCAcZEBUuG8keyNPaE9M3bDPxU8jQ7l8P1OzaDDmSUnaYBAMVcFFuJaySA6fHLAZfWx593SvxMg8jOlNGDZwvf4= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1766415153; c=relaxed/simple; bh=JKKJn8euRQ6QnFpf+G3BcPHFGDaTAmLiEhgvLCRk5xA=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=cLeBvZLpcx4+wWjRQIQRi/6Qcvt2k7TGi7/n7UxvKa8Qt0bACPGwzeDY9rIf/E2IL5UNjgaVSu7AfsorNRh8lRwrctM+nJaax6C3xfclmHkHEmkbUIm+Y5bkGkvpWQ50zxZ6Gv1lqlVH9FJGlMIenkCmjzEBJWXEldjtcyy3fLQ= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=debian.org; spf=pass smtp.mailfrom=gmail.com; arc=none smtp.client-ip=209.85.210.51 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=debian.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Received: by mail-ot1-f51.google.com with SMTP id 46e09a7af769-7c75387bb27so1667300a34.1 for ; Mon, 22 Dec 2025 06:52:29 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1766415148; x=1767019948; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :mime-version:subject:date:from:x-gm-gg:x-gm-message-state:from:to :cc:subject:date:message-id:reply-to; bh=6ERsjML2moqdleOPpyHCHm9fmTJ8lit1vijzoNSe2v0=; b=n+kukzzF+FAzC3Uh04LsylZFNbkKM33aUOdrxHax40sKvzrq4PPdVsuIzWjAID40Fs 6pO06ziGLKkDI1EWiTjJhniLftlUrDkeiFW5tYbrzCqoBkOFBPOKb+wD3qA2GKUGK4Yl 3S+qrRtLx6sgSwiKTtxwbdRC1AqCPt+VSce6ThtQLVTbZhKW2+go1dLDetfCeC3WMmwR v//+g1OuBjlLOZgdCCrlelqi1VuCi/hjVNQ2oUwaxwmGk6ZeUhzBaBc0LCM3aX/MAwkf V88OwfBAFGSRpdV0suQAIpEk0DouE8sYvd7dxAXsh25OFlgkHITPgluCk3i4x7zyvjXv 44HA== X-Forwarded-Encrypted: i=1; AJvYcCUqJclmnWYSaJ0cXcFXPQz/1FWW34GkBRXO+jM2Gie73yG3b3rI/kll0gMsrJuE/HggqxJ++qVvOU5NC3E=@vger.kernel.org X-Gm-Message-State: AOJu0YziDM2datTfLS0I1gFDYAZ10GXvV1kv7YqZiDiQZ1kDvafqIqNT 2s4jqY/tPlzQqMAH0uHRIilcx4Hj//hE8ybEiiTmIOaICdoCfl3M4qla X-Gm-Gg: AY/fxX7SU0SGMWD5KmM57POuKapFAbUyuTWSMTdLgrvTQldHqxzbBL8n+Fy810TpY+7 ElEvxgn9vFICEwRqFIiFVGRcEBkEHayCNtKPNieuRoggTThtAsHuHKogSDqmoiKrcYM+bQtPU72 yuLxMC0ymVSGE+fIYTHeL5/ubpYjvU3stxiPX4gV7ElW0OMPXYAGb0+cdvDa6JaAm7Fx6auH1z/ IACW7jRqG5JNVbaLImlTf1HIeTnyIA+mUk/0nBGhJHGJWT+CCyuSWWMiPhS5lGiWQw4EJaxwPyr NDUHkeLFrejPWKdX81kIUptw+XzdUOxcMGlGOy4+NhCMY1IFew+LGvGKYOUrHL9crSQgVS+6mXZ NBmX9nZz3t3Y/ZotipnV4BeCrt+z3VJx89oqosMIjRJI6RYwQaxyPDQa4Zit1nDGoORnS8DMqh/ ylNTesi2PXe4xESpfOS3lTRJek X-Google-Smtp-Source: AGHT+IEwiw5VfvorK/Ul+TfB+9hhGgpe8qUpgC9qG0t1x5N29CqWxupoUXwVJSomNaedw5ZVlmKPMQ== X-Received: by 2002:a05:6830:2649:b0:7c7:6bb4:1197 with SMTP id 46e09a7af769-7cc66a6f058mr4837844a34.24.1766415148286; Mon, 22 Dec 2025 06:52:28 -0800 (PST) Received: from localhost ([2a03:2880:10ff:5d::]) by smtp.gmail.com with ESMTPSA id 46e09a7af769-7cc667d572asm7459409a34.14.2025.12.22.06.52.27 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 22 Dec 2025 06:52:27 -0800 (PST) From: Breno Leitao Date: Mon, 22 Dec 2025 06:52:10 -0800 Subject: [PATCH net-next 1/2] netconsole: extract message fragmentation into send_msg_udp() 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 Message-Id: <20251222-nbcon-v1-1-65b43c098708@debian.org> References: <20251222-nbcon-v1-0-65b43c098708@debian.org> In-Reply-To: <20251222-nbcon-v1-0-65b43c098708@debian.org> To: Breno Leitao , Andrew Lunn , "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni Cc: netdev@vger.kernel.org, linux-kernel@vger.kernel.org, asantostc@gmail.com, efault@gmx.de, gustavold@gmail.com, calvin@wbinvd.org, jv@jvosburgh.net, kernel-team@meta.com, Petr Mladek , Simon Horman X-Mailer: b4 0.15-dev-47773 X-Developer-Signature: v=1; a=openpgp-sha256; l=2095; i=leitao@debian.org; h=from:subject:message-id; bh=JKKJn8euRQ6QnFpf+G3BcPHFGDaTAmLiEhgvLCRk5xA=; b=owEBbQKS/ZANAwAIATWjk5/8eHdtAcsmYgBpSVsqhCotq0bJbtdcEHfYfktYBhbVmvU/M9bAh FbRc91XE86JAjMEAAEIAB0WIQSshTmm6PRnAspKQ5s1o5Of/Hh3bQUCaUlbKgAKCRA1o5Of/Hh3 bWvJEACZRRAePZN5P3WDsYws8Xsy0kuoADHIaq9Zggtf9WOGRIqNVdMhZJ3ljvPMiS03IoDTv77 hCYQycItGtZ4u7S2JlW3WHHlgpwybsTbLplRCkqndDyhgHa1sMlIgWFB05+jMmBRzTnfix5cRtr m8+hyAU55rCSTfeThzXN4/XPcS0F+0+9uRCGzY/pQHcBKe62VdHNppGW4NcQuBPsl7VFZwVJgaU KRxC0IFX+6H64BPd0eYDwcjFa9lLYMGxDXRkBO2mkaio2N5oC2TItowm+ovvpOrabj7tDOCL6gj 9N9DmzIVWQNFrbqqa0rA5On7Qd/9qsRaEibBC8XG0LOXenDoyBbNbg3t5Ui7f29QW8wZGr8EdCC F6nci2zZKYpphzqmE5SQSPPDZocVvZ0nYfEuBwMEk/cwfBKE4ffIpPwbdVvcYRijqxWAOl8QPMA XrQhMk6Xm24cMcskxD1nPPYztOXekM36KeZMNBQ0U9vuJcqu4lhB13wgXEle+Y3ygA27JZ4MQ3M uS2DrylDpVn1+mkdjmXbqzRk0UMJxB2dEuPe0zsGDdtkxqQYt+JRUeIzz+HM9QQPwgL//JmGeqn Iyw6UUl7DNd875SfGv0hxzpGqXS0wKRERGZ1AdS3x3G75/NNzMATXXUMgXJzcnfXgz7SUXbj/Wh 24FzuQsimNzb8Cg== X-Developer-Key: i=leitao@debian.org; a=openpgp; fpr=AC8539A6E8F46702CA4A439B35A3939FFC78776D Extract the message fragmentation logic from write_msg() into a dedicated send_msg_udp() function. This improves code readability and prepares for future enhancements. The new send_msg_udp() function handles splitting messages that exceed MAX_PRINT_CHUNK into smaller fragments and sending them sequentially. This function is placed before send_ext_msg_udp() to maintain a logical ordering of related functions. No functional changes - this is purely a refactoring commit. Reviewed-by: Petr Mladek Reviewed-by: Simon Horman Signed-off-by: Breno Leitao --- drivers/net/netconsole.c | 24 +++++++++++++++--------- 1 file changed, 15 insertions(+), 9 deletions(-) diff --git a/drivers/net/netconsole.c b/drivers/net/netconsole.c index 9cb4dfc242f5..dc3bd7c9b049 100644 --- a/drivers/net/netconsole.c +++ b/drivers/net/netconsole.c @@ -1725,12 +1725,24 @@ static void write_ext_msg(struct console *con, cons= t char *msg, spin_unlock_irqrestore(&target_list_lock, flags); } =20 +static void send_msg_udp(struct netconsole_target *nt, const char *msg, + unsigned int len) +{ + const char *tmp =3D msg; + int frag, left =3D len; + + while (left > 0) { + frag =3D min(left, MAX_PRINT_CHUNK); + send_udp(nt, tmp, frag); + tmp +=3D frag; + left -=3D frag; + } +} + static void write_msg(struct console *con, const char *msg, unsigned int l= en) { - int frag, left; unsigned long flags; struct netconsole_target *nt; - const char *tmp; =20 if (oops_only && !oops_in_progress) return; @@ -1747,13 +1759,7 @@ static void write_msg(struct console *con, const cha= r *msg, unsigned int len) * at least one target if we die inside here, instead * of unnecessarily keeping all targets in lock-step. */ - tmp =3D msg; - for (left =3D len; left;) { - frag =3D min(left, MAX_PRINT_CHUNK); - send_udp(nt, tmp, frag); - tmp +=3D frag; - left -=3D frag; - } + send_msg_udp(nt, msg, len); } } spin_unlock_irqrestore(&target_list_lock, flags); --=20 2.47.3