From nobody Mon Dec 1 22:35:04 2025 Received: from mail-ot1-f49.google.com (mail-ot1-f49.google.com [209.85.210.49]) (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 0293D22F74D for ; Fri, 28 Nov 2025 14:20:57 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.210.49 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1764339660; cv=none; b=lfoa1G0MRWOmZQcXJte4MaHPWDTI+OePGPIiNRliGKFTyNZc0pN9PIvzSGTfO8iQ525iZGbdjF8nqbFu4dq/OxxXJZINrmTILaaWUbI10DQzEniM4+jMTn+4IWTtyO8Y6T3Co5/Wl0fggDniYUyhDzI1QDQPteNlZYyx37jFAFA= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1764339660; c=relaxed/simple; bh=8AeeNuKarsu5qA34c7lNeF8QhulWBTrnEQ0CggOIbSI=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=tPZngXk0tIuSV81iC1AXwevvfQPo+D5o09oRMbjdE992kDoXzJtr5WoLEZA6DWH1zoCYb+i1BBy5sBOvA95u69X99SB8yAORYPoW0h+MWXnlAUhO/giQ0aYBEjMBz3+5z3bMYpYzCVEE5dHfnmKC8rJKc9Au1Gv9a4ObLymJRtM= 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.49 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-f49.google.com with SMTP id 46e09a7af769-7c6e815310aso1127857a34.0 for ; Fri, 28 Nov 2025 06:20:57 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1764339657; x=1764944457; 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=IO8fpf+Qe7KA1hAphkMPr2BDEzo3Q3M23aOQSgVd6Z8=; b=b6f/P7l/mZ0YxKXCD9D00NeLW94Hqh0Lk/sbMztFUr4P2SOL9D/NBMr//QWjH9ZF0h ug5amb1/Lqa3IYwc2By/fCCxR4MeXs+ZeEjN0VeBPnIzY4P8Ye0GG6efZqIYEkVUfDI4 kZRGHa3eHAUzIpkyI4VDauv9C/Kww7/Tlpblkc/nVxxpxftzk1KzbhkN0WAe6uTpmUsr DvJT15TtUQzyUiHaLFZczmUM464n4WBUU7RFwboEwRySssMLf8jKhzcZLigmcGhjomMy vo0J34ion4EBX3rSily4yVtHp4RxdikZrn5xQteIzit8QOuoPawg/7xT7fCsCbMrCYC2 /XHQ== X-Forwarded-Encrypted: i=1; AJvYcCXlXXNm2jAgYlBG34SFfShfA/6YGFgj3KioF0Y+C/6cjnLPPO8/tcDl7Z0r3G8fIzoYjgximkKxSDr1GPM=@vger.kernel.org X-Gm-Message-State: AOJu0YyYgAamPDzoG2hT7yh4UEOdAuF2ZG+WngivJCLpKHwQUUHYM1qK Z/cElTRTUPW7bF5fUYOw+Hg773jYgDOjoDcfFu/SS7RRFad46FcfVXR/ X-Gm-Gg: ASbGnctiBhbvze/+PtZ9AaXv45DP3IgTIqvtDHb2XJEVvCQhArJ8hBUAose9gBDN3PH n2toLJnWQOfsiBNQoJny1xKyvO4yXOAgP6I3HUC3QOGEenOmrOMSe6oQaDWEZgqdrP3ZbyxRM/N MUoa1KUuEvhKCiia6N+D3MF0qDOEOTo2IO2F4tG5id6VQqiQZEl4uPtYsD6LiH1R2ZSXTDYJ+hW v4ld0v5Og2B1z6wz2p8m/3Fu7MeoAHOb0tSXMy+pb6lo8KkIVlgdxlcFkbSSDLSDLi0mJH1oZlQ c7Q9JzTTyF0xPcn1EiLnf9Oz81j3SbR/bl/gVLdibyYg15LjWgyJMSsdjqqVWucIYjjNaH4xNHT YGZUMSL4gYDl9spw3ykrzIBme26iJwvx8KbPUNq3kdhKKzzmAbdsYR/DesSb7R/AzxDeuHFEJ6D a0dvTlESfpMA== X-Google-Smtp-Source: AGHT+IE8Csv0x3Otw25HTHmycCm8OmZBQcHgbovfIus9/bAoq/UixullE88axucO5VUxe/H7t0bIYA== X-Received: by 2002:a9d:6143:0:b0:7bc:f443:fa3c with SMTP id 46e09a7af769-7c798de468amr11684101a34.25.1764339657047; Fri, 28 Nov 2025 06:20:57 -0800 (PST) Received: from localhost ([2a03:2880:10ff::]) by smtp.gmail.com with ESMTPSA id 46e09a7af769-7c90fe0b238sm1586166a34.20.2025.11.28.06.20.56 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 28 Nov 2025 06:20:56 -0800 (PST) From: Breno Leitao Date: Fri, 28 Nov 2025 06:20:46 -0800 Subject: [PATCH net-next 1/4] 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: <20251128-netconsole_send_msg-v1-1-8cca4bbce9bc@debian.org> References: <20251128-netconsole_send_msg-v1-0-8cca4bbce9bc@debian.org> In-Reply-To: <20251128-netconsole_send_msg-v1-0-8cca4bbce9bc@debian.org> To: Breno Leitao , Andrew Lunn , "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Shuah Khan , Simon Horman , Jonathan Corbet Cc: netdev@vger.kernel.org, linux-kernel@vger.kernel.org, linux-kselftest@vger.kernel.org, linux-doc@vger.kernel.org, gustavold@gmail.com, asantostc@gmail.com, calvin@wbinvd.org, kernel-team@meta.com, Petr Mladek X-Mailer: b4 0.15-dev-a6db3 X-Developer-Signature: v=1; a=openpgp-sha256; l=2049; i=leitao@debian.org; h=from:subject:message-id; bh=8AeeNuKarsu5qA34c7lNeF8QhulWBTrnEQ0CggOIbSI=; b=owEBbQKS/ZANAwAIATWjk5/8eHdtAcsmYgBpKa/FePe42auqDY/Qs7XpYdwbHd6fJqLNwggTf H3VVdrUTfSJAjMEAAEIAB0WIQSshTmm6PRnAspKQ5s1o5Of/Hh3bQUCaSmvxQAKCRA1o5Of/Hh3 bWEJD/45t6CqWMn+sI2J6+MyW697ZsxhgZ539JQ7X8cHcLAwGA2l4VRPb+vljF+qZ6sV6BJCeZ+ Tsxb08lgFu+44cvvEKSO59GXsfsKgGPRVe3obqlCcy99XLONvtOjT1WZRgN9L9h85HG/YzXOoeQ j/17CmNiqgeRNwjiZDCcgnJqbsUlWumE/8WoI+8lL+AibpkKcKkTW2Bdil74n6zYF1k2tWDB4Cs 9trke+OdIrF4b47c3a91ZBxzdZi0918GKat/sKevUD7eDoQBrLOeQmiP+ZIv09pCBavFFquA86k ZPZg9i0CuRkCGQpwew0n94sHV78imAeHK6n+H8wKzKuBTgQ0hh+aCyU0X5+CIXOOMMz20jd6Qft d0H9qaOjkgbkP/t/YV9Up71zlF/DOlQDwEp1wDXtlH04PszFsRUhJ1VqDZksJLoAMl/owzcDQ50 onheqsBZqT6IwvdImVa88fwcS+02optkhu9kSZQXt1u+TWJ87RcJ9VcRJ+KJIxbudo6oS8Is0Iy 5cKccRhcZFN5qjxN0JfVJfVRMUfXRT5o5yGUYqw8TqiV/UO9KTDSy/tr+HuynBy0skfbitfVx8t bL70JidMIQe8IOrhsngRbjrw/PEnjEoPbx/JRtpotnia8TJrrLh+K4S0PfdLjcyxncZ6yUtRwQX Vg52jnE6T2U5QcA== 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. Signed-off-by: Breno Leitao Reviewed-by: Petr Mladek Reviewed-by: Simon Horman --- 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