From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-oi1-f175.google.com (mail-oi1-f175.google.com [209.85.167.175]) (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 65D603D3CEC for ; Thu, 22 Jan 2026 11:32:34 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.167.175 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1769081556; cv=none; b=MKcleADIW8z53WRCnhOpf1NlbITCbc/qJ7i2x5gjS4rsrgSX4kwe6KNgc2edbl+bBwn1/xEB0gjA0vPWSbv4WL9PF0RxcOZpqJrWdfM7phsCkNH/VK6dHi3KBF5WP3b6y0WRApmAAge+3o/zo39CnqeFzXNYxJApl0DVzmy6Ftk= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1769081556; c=relaxed/simple; bh=Hz1gu3DrXXHFmq3h1ZTNOZftr7Xlzo3ths2B7ZsBtME=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=X0Yj8GzjOKzQgAsP3wY/0bkVHNbNNmwMHUSdww41PMegOTyjGWGSptN3sg2WzJhPUneJJn7cYSx80Nbfg9oUoP/vIjht910RvQp2WwX5B7od3iJYGVKsxzXqUW8RI+0B/OXtDr/nqhQjmj2kOjBKfVROL0xgP2AA5yyv43Y/s1s= 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.167.175 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-oi1-f175.google.com with SMTP id 5614622812f47-459fa8b6044so531298b6e.2 for ; Thu, 22 Jan 2026 03:32:34 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1769081553; x=1769686353; 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=mHAlKWdigu/dpowmiGEPP+hWTchneUXrrcu0qUva3LI=; b=fz0vJ/oZrcM35K3sDCLt33jbL3R8CM73aqQ+bY5ewl8KOs1Mk42sfgfHbGDmSoPt4f eRpqQVfc1CIOCq0Zz/qyjfBykP2O/PTUb3FlEVrEhVfOdHwWmalWus5vBO1jbjaONLs4 uJBVeexqfrg2wfDdZHP3bqPDGxOKbdtijDtz/spSX0vsZCH3VNk1IOVbI5WALhm2IBCp r7kCVR3JbFp6o9BAT2+Gy2fnbomBL2PxVQ6jZMBM6naDNHynyQufrQcPlbz9W+e/eUMh uo4lkAz4WOkV+/JOnfk5G7o6BUyIxtuu41xG6E86GnzbvH/T6jPVIdp9b4sopq8qRJls D4kw== X-Forwarded-Encrypted: i=1; AJvYcCWWm96NmDqP63vXOGqBhQGccVF6bEA1G4GOZe5H2kJFTblPZxV8zmk699PQzCLRogt78BeZgXVMfpA0Jao=@vger.kernel.org X-Gm-Message-State: AOJu0Yw2VN/gOJ5snMzpqbaA8KjNGZILusKO0xH/lvHL5T3gwo8vnXeh kDPxyO1BgqOeFm1QjpwT0I1ZUDWkIlgITYsXvUBDhRm7wgK6BDlhkdKB X-Gm-Gg: AZuq6aLdNuV2tavfc/11OuAyQ6IM1c30yZloWFB25WccgIwO2vfDQs1CetC9TPzVNpq PEVZnhpzM237up6Jh4lfd88JBbDYzHuneNUDhDmiBpD0yETfgjUA7VIXJ60CdpR3kB/QpYKz19h NHqf8HU64hpvvPsjyoD7GX78C+igZ1bEDAd4MvSm4ukIeMKk0i41LXk4jEUefdHTbV3CAQfdFgi F4Nv3SrfvLdlb5xSHpGuHUyPRfX6KbG92GmpfK9yYweYhSX7/KJdN74+pdMNwMPGsbp7VerAdJF g3Sh5s5LrhY9pXOXfE0T6GDJbz2eWS24LHzhi7zOnW2+6Wxqqp0Xvqp1T3NUq64o1xcPhdPbHay lzmrw8MFG7Bx4rI3paLoBNOmfrM9ZBz4eB0+6uK9PTqZYTlj87iam/rukFjGav+d+w7e8yXWkEt wlI8SnbZL7pUFc X-Received: by 2002:a05:6808:6901:b0:45c:881c:e0d1 with SMTP id 5614622812f47-45c9d74062emr10548102b6e.23.1769081553090; Thu, 22 Jan 2026 03:32:33 -0800 (PST) Received: from localhost ([2a03:2880:10ff:45::]) by smtp.gmail.com with ESMTPSA id 5614622812f47-45c9e0086a4sm10160669b6e.12.2026.01.22.03.32.32 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 22 Jan 2026 03:32:32 -0800 (PST) From: Breno Leitao Date: Thu, 22 Jan 2026 03:32:10 -0800 Subject: [PATCH net-next v3 2/5] 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: 7bit Message-Id: <20260122-nbcon-v3-2-a722f2f0dfa5@debian.org> References: <20260122-nbcon-v3-0-a722f2f0dfa5@debian.org> In-Reply-To: <20260122-nbcon-v3-0-a722f2f0dfa5@debian.org> To: Breno Leitao , Andrew Lunn , "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , pmladek@suse.com, john.ogness@linutronix.de Cc: Greg Kroah-Hartman , Steven Rostedt , Sergey Senozhatsky , Andrew Morton , netdev@vger.kernel.org, linux-kernel@vger.kernel.org, asantostc@gmail.com, efault@gmx.de, gustavold@gmail.com, calvin@wbinvd.org, jv@jvosburgh.net, mpdesouza@suse.com, kernel-team@meta.com, Simon Horman X-Mailer: b4 0.15-dev-47773 X-Developer-Signature: v=1; a=openpgp-sha256; l=2097; i=leitao@debian.org; h=from:subject:message-id; bh=Hz1gu3DrXXHFmq3h1ZTNOZftr7Xlzo3ths2B7ZsBtME=; b=owEBbQKS/ZANAwAIATWjk5/8eHdtAcsmYgBpcgrM65kr4q4DiCJRkPIWZcOuT5I5mkopp6WFR YokhN3LVR6JAjMEAAEIAB0WIQSshTmm6PRnAspKQ5s1o5Of/Hh3bQUCaXIKzAAKCRA1o5Of/Hh3 bevtEACNY0e+WVQ18HdS1G6qyTCgDBgS+8yw11ndlh6PPQ+j8HzBhWVIvnAYPpEXRNO7O0J5hzf xRvXvnBMHEU4aPe871JRT6FDLSdXOszc6qM6xp5TLJp5p9mkJLCKfQIALmA0ihUukOfonjd6o6k eJ9kfb+QehJ/pkKr9wvAJPSI9d4oXfZDMnZR5fsSTh5KS49YmHN3wNM99KZQEsgsqSvvhfDisNk OmHzH9Q8kthxY1+Ui5dBT82YOy5iFP7j3/rfuAoSghJJIhZr/2xHlBESV1h49v55gmWzKVg1P+o 8qnJLal+gECsuqIMu+TL/QOoJrI3jkS+0PXXs2pDy37TQe1CM7EHGdvDUjIvHdHGlm1zdwFG6xt V544D3FAVeFrexFPgRkycAQF6ZOjug6lZv3RP+ruSXnM+Fz4+I1CeCv2+jDhAkiDAAKJtmzggC3 hYIs8+aGtuxXqxOwY0ilw7X49KCLw4K/PlmokITi6L6LOO8XDVzmtd/FotnWjmRbQMGj8BvA4tR vBKmNh4KEcSp/T9EahEmFs66gu9QoEUs8rZ2r9R/zX6+KfSQmqO/XZGXjAwL7Ci3AVkYn5GVJWU 8XKAypyd5Q0Jj3Xq7/22FUw8x0deHBpg++aSC9TI+fecbRDTMkLtwdR+tUH9nfuoW6icgZQV3A1 tIksNHQVnVXvxmg== 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 9cb4dfc242f5f..dc3bd7c9b0498 100644 --- a/drivers/net/netconsole.c +++ b/drivers/net/netconsole.c @@ -1725,12 +1725,24 @@ static void write_ext_msg(struct console *con, const char *msg, spin_unlock_irqrestore(&target_list_lock, flags); } +static void send_msg_udp(struct netconsole_target *nt, const char *msg, + unsigned int len) +{ + const char *tmp = msg; + int frag, left = len; + + while (left > 0) { + frag = min(left, MAX_PRINT_CHUNK); + send_udp(nt, tmp, frag); + tmp += frag; + left -= frag; + } +} + static void write_msg(struct console *con, const char *msg, unsigned int len) { - int frag, left; unsigned long flags; struct netconsole_target *nt; - const char *tmp; if (oops_only && !oops_in_progress) return; @@ -1747,13 +1759,7 @@ static void write_msg(struct console *con, const char *msg, unsigned int len) * at least one target if we die inside here, instead * of unnecessarily keeping all targets in lock-step. */ - tmp = msg; - for (left = len; left;) { - frag = min(left, MAX_PRINT_CHUNK); - send_udp(nt, tmp, frag); - tmp += frag; - left -= frag; - } + send_msg_udp(nt, msg, len); } } spin_unlock_irqrestore(&target_list_lock, flags); -- 2.47.3