From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wm1-f51.google.com (mail-wm1-f51.google.com [209.85.128.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 241A536EA8E for ; Tue, 2 Jun 2026 08:54:08 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.51 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1780390449; cv=none; b=dpekEkOOto6u4P1i0SAOr4s/VwP8e6AGtDBbmT9IuKFKeB7FF1eBzm1vRnZkfN3dsvdgJ+3VUFXuyeA9Q+HX60xJjl3aU3pPh1LZ5EBFQKAOYMan7X1dOVP42VghSF3kbHIWV5L4m7PZCHLnf3MxutgRiiRzvcFCaZkr4dc6mdY= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1780390449; c=relaxed/simple; bh=tjBPP6vwYVT9PN7pxaSqYfQdRMoDmbHXU+zTX5/4YbI=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=pq0c3SfnhICUashjZ99kv5QipgfFmFzNeClZTwplIUcxgeo9uGQbrATDPZdfAvFdls699SNsoB8lFBDJEPkscGVZMQRDiWcgEjudrQ7q4thFtBKuhHpNsdmYXEyhWqqLTenvDJ9XMC7EmLyOS+nVIsfkm42hOMMlvjI3k40wOSw= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=suse.com; spf=pass smtp.mailfrom=suse.com; dkim=pass (2048-bit key) header.d=suse.com header.i=@suse.com header.b=fRVMGyhW; arc=none smtp.client-ip=209.85.128.51 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=suse.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=suse.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=suse.com header.i=@suse.com header.b="fRVMGyhW" Received: by mail-wm1-f51.google.com with SMTP id 5b1f17b1804b1-490388fd0dbso111618975e9.0 for ; Tue, 02 Jun 2026 01:54:07 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=suse.com; s=google; t=1780390446; x=1780995246; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=v1suNiINZUqCrLI/VwQHHQcRyvtfBY7Ngnxz2o1UORo=; b=fRVMGyhWvNZ0rLzTYSejtnf9fFpdLDUpD6/mNS0ZE+vHOrd430RT6qmkyX4SlzUGpY pK5C3myEo8idDdgzHjGrpTaW8T1RkvpbE/SuPOV6SOKBCCLEiV6C5re7Ae+EEBxBhwYT zKtFmUeNDCkesWbM44n9AI+D9ZRC1S+lmkyEzTzJvNM+gfIXbQaLvdmes7AHEOJVwjll p/vAsguFr9dO7NzHLfQ3P0LxlwKg7oaTuqwQA46XD8rLnhWD4JlLuo1WeFbNUWqY1hzh Vg6wT0FADReBAGAAcZQbg3lbJ3Js78jMQg21U+tV5KVK0I9IzTrhbge60pMHzHPhBQfd hrUw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1780390446; x=1780995246; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-gg:x-gm-message-state:from :to:cc:subject:date:message-id:reply-to; bh=v1suNiINZUqCrLI/VwQHHQcRyvtfBY7Ngnxz2o1UORo=; b=RcTgF1xwSeC+llsGw2nggRl2vlfCzd0vOuGa/hYfFwlmy1IZTTNK05fCxnkl81KzKE DZMbmiMW0htlYDWnlHMCBf/JOS+3XHJQSRT6CuyHTmKZCfG4lOF9WjZnKBZLcINk352Q y0498aa5sz5Xzo4Hqp/Myg/ywmiIt7kTLNhraw+ky4kcViTwDC4EAY8vguzLOss1Fmew 0uPG3oNTOM2ahn4nxHc2WrQ78dTrI3nNAnWJukx7BkRd1jGcXs34Xj38OzlGWZHI+hCp s2S5bV23JBVr45Fr1sbf06/vgBZOTfdYGhExRZWxqRqTWE3E1pKb4/1CX9vhznY/6OUY VSfQ== X-Forwarded-Encrypted: i=1; AFNElJ9zQ4T3C96e7iTKJkQ+pJ5QOILLgSxeAzHh4+lN2f9RiV5MU313/BnQ+gyaIbNYxUxRkHuMxEm5Fd0Idjg=@vger.kernel.org X-Gm-Message-State: AOJu0YwmvoAzaZyzMB0ld7kuaw0epj4z+dpwXXs2F4KPhJpaINr1KL7N danJzmSsxSHy1CDiQTyNunYtfjZtU6wpt9uftZ9YYrIlynFRW/Ar1IUTdq/H6g52ruk= X-Gm-Gg: Acq92OFpcv88Dv5jKRHi/7+U6O0toviOW85A9hcM+OCnzVDoz7zU9U+lqCoRDczVXei u87ufgd3/kjSs01QNVOt6XHBFF3R1+CoeNbkbrzSWI17uvSx/FtqIS47diVcnZ2x3vV13eLpU3J 2KP1K4qO/H+n1CVsLI5mWsY0lMkpQ/ydlc1a4k4CZwK4Cjmp2D+1iLjMZmJiRzfu/TCzJJobC/9 5eX/w9seaIfqVgURoAO7WpojIlWd0zgtrNR/Uy1jErfXlRCobQBLHHLm9pPKEOzFhJfNRukX4Ww HOWI0+BBCMDXe3uAbvSK1etSv3+XdaEVZ0WjoXTj2Kr4iY5gVYWGJtX8iBpgPmAe9qSzwNimyX6 TnspkVC1CTPslBXZF0R2Cpr1ARKipfeG/ZFaflHFxCIHSMM35FEd4rymh5D4qdSBP17ruq7mWoi R53HSp3KoI7leQhrShTOEpz3KMa2SRZfK8KuQa X-Received: by 2002:a05:600c:348f:b0:48f:e230:c3fb with SMTP id 5b1f17b1804b1-490a2988e6dmr292361175e9.33.1780390446371; Tue, 02 Jun 2026 01:54:06 -0700 (PDT) Received: from pathway.suse.cz ([176.114.240.130]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-490b0daefbbsm88488945e9.0.2026.06.02.01.54.05 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 02 Jun 2026 01:54:05 -0700 (PDT) From: Petr Mladek To: John Ogness Cc: Sergey Senozhatsky , Steven Rostedt , Marcos Paulo de Souza , Chris Down , linux-kernel@vger.kernel.org, Petr Mladek Subject: [PATCH v3 03/11] printk: Separate code for adding/updating preferred console metadata Date: Tue, 2 Jun 2026 10:53:04 +0200 Message-ID: <20260602085312.228251-4-pmladek@suse.com> X-Mailer: git-send-email 2.54.0 In-Reply-To: <20260602085312.228251-1-pmladek@suse.com> References: <20260602085312.228251-1-pmladek@suse.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit The logic for adding or updating a preferred console is currently duplicated within __add_preferred_console(), making the code difficult to follow and prone to consistency issues. Introduce update_preferred_console() to centralize the initialization and updating of struct preferred_console entries. This refactoring explicitly defines and enforces the following rules: 1. Console names and/or indexes are not set when a console is preferred via devname; these are resolved later during device matching. 2. Console names are only added alongside a valid index. 3. Only matching entries are updated. 4. Console and Braille options are never cleared. They are updated only via the command line. 5. The global 'preferred_dev_console' index and 'console_set_on_cmdline' flag are updated consistently. Additionally, rename braille_set_options() to braille_update_options() to better reflect its conditional behavior. Behavior change: The original code never updated the preferred console options when it was preferred more times, e.g. via the command line and/or some platform specific code, e.g. SPCR or device tree. The new code explicitly allows to update the console options when they are preferred over the command line. It mostly worked even before but only because the command line was processed early enough before handling SPCR, device tree, or other platform specific init code. The main behavior change is when the same console is preferred more times on the command line. Newly, the later or Braille variant wins. It is a more common and expected behavior. Signed-off-by: Petr Mladek --- kernel/printk/braille.h | 7 +- kernel/printk/printk.c | 153 +++++++++++++++++++++++++++++++--------- 2 files changed, 122 insertions(+), 38 deletions(-) diff --git a/kernel/printk/braille.h b/kernel/printk/braille.h index 55cd3178a17a..0bdac303f8b1 100644 --- a/kernel/printk/braille.h +++ b/kernel/printk/braille.h @@ -5,9 +5,10 @@ #ifdef CONFIG_A11Y_BRAILLE_CONSOLE static inline void -braille_set_options(struct preferred_console *pc, char *brl_options) +braille_update_options(struct preferred_console *pc, char *brl_options) { - pc->brl_options = brl_options; + if (brl_options) + pc->brl_options = brl_options; } /* @@ -29,7 +30,7 @@ _braille_unregister_console(struct console *console); #else static inline void -braille_set_options(struct preferred_console *pc, char *brl_options) +braille_update_options(struct preferred_console *pc, char *brl_options) { } diff --git a/kernel/printk/printk.c b/kernel/printk/printk.c index 5938e9df7345..4e4aadc51f6b 100644 --- a/kernel/printk/printk.c +++ b/kernel/printk/printk.c @@ -2544,18 +2544,123 @@ asmlinkage __visible void early_printk(const char *fmt, ...) } #endif -static void set_user_specified(struct preferred_console *pc, bool user_specified) +/** add_or_update_preferred_console - Add or update a given entry in + * the preferred_consoles[] table. + * @i: index of the entry in @preferred_consoles table which should get updated. + * @name: The name of the preferred console driver. + * @idx: Preferred console index, e.g. port number. + * @devname: The name of the preferred physical device. + * @options: Options used when setting up the console driver. + * @brl_options: Options used when setting up the console driver + * as a braille console. + * @user_specified: True if preferred via the kernel command line. + * + * The function ensures that the given values are consistent. Also + * it updates some global variables which are used to make the right + * decisions in register_console(). + * + * Rules: + * + * 1. Either @name and valid @idx OR @devname and @idx=-1 are allowed. + * Note that a valid @name and @idx will get assigned later when + * @devname matches during the device initialization. + * 2. Specify @brl_options if the console should be enabled as + * a Braille console [*] + * 3. Only matching entries can be updated. + * 4. @options passed via the command line are used when the same + * console is preferred also by some platform-specific code. + * + * [*] Braille console is using the mechanism for registering consoles + * but it is very special. It is primarily used for user interaction + * with the system. It neither gets printk() messages nor is associated + * with /dev/console. + */ +static int add_or_update_preferred_console(unsigned int i, + const char *name, const short idx, + const char *devname, char *options, + char *brl_options, bool user_specified) { - if (!user_specified) - return; + struct preferred_console *pc; + + if (i >= MAX_PREFERRED_CONSOLES) { + pr_err_once("Reached maximal number of preferred consoles.\n"); + return -E2BIG; + } + + pc = &preferred_consoles[i]; + + if (WARN_ON(!name && !devname)) + return -EINVAL; + + if (devname) { + /* + * A valid console name and index will get assigned when + * a matching device gets registered. + */ + if (name) { + pr_err("Adding a preferred console devname with a hard-coded console name: %s, %s\n", + devname, name); + return -EINVAL; + } + if (pc->name[0]) { + pr_err("Updating a preferred console entry with an already assigned console name via devname: %s, %s\n", + devname, pc->name); + return -EINVAL; + } + if (idx != -1) { + pr_err("Adding a preferred console devname with a hard-coded index: %s, %d\n", + devname, idx); + return -EINVAL; + } + + if (!pc->devname[0]) { + /* This is a new entry. */ + strscpy(pc->devname, devname); + pc->index = idx; + } else if (strcmp(pc->devname, devname) != 0) { + pr_err("Updating a preferred console with an invalid devname: %s vs. %s\n", + pc->devname, devname); + return -EINVAL; + } + } + + if (name) { + /* A console name must be defined with a valid index. */ + if (idx < 0) { + pr_err("Adding a preferred console with an invalid index: %s, %d\n", + name, idx); + return -EINVAL; + } + + if (!pc->name[0]) { + /* This is a new entry. */ + strscpy(pc->name, name); + pc->index = idx; + } else if (strcmp(pc->name, name) != 0 || pc->index != idx) { + pr_err("Updating a preferred console with an invalid name or index: %s%d vs. %s%d\n", + pc->name, pc->index, name, idx); + return -EINVAL; + } + } + + if (!pc->options || (user_specified && options)) + pc->options = options; + + braille_update_options(pc, brl_options); + + if (!brl_options) + preferred_dev_console = i; /* * @pc console was defined by the user on the command line. * Do not clear when added twice also by SPCR or the device tree. */ - pc->user_specified = true; - /* At least one console defined by the user on the command line. */ - console_set_on_cmdline = 1; + if (user_specified) { + pc->user_specified = true; + console_set_on_cmdline = 1; + } + + return 0; } static int __add_preferred_console(const char *name, const short idx, @@ -2563,19 +2668,11 @@ static int __add_preferred_console(const char *name, const short idx, char *brl_options, bool user_specified) { struct preferred_console *pc; - int i; + unsigned int i; if (!name && !devname) return -EINVAL; - /* - * We use a signed short index for struct console for device drivers to - * indicate a not yet assigned index or port. However, a negative index - * value is not valid when the console name and index are defined on - * the command line. - */ - if (name && idx < 0) - return -EINVAL; /* * See if this tty is not yet registered, and @@ -2584,28 +2681,14 @@ static int __add_preferred_console(const char *name, const short idx, for (i = 0, pc = preferred_consoles; i < MAX_PREFERRED_CONSOLES && (pc->name[0] || pc->devname[0]); i++, pc++) { - if ((name && strcmp(pc->name, name) == 0 && pc->index == idx) || - (devname && strcmp(pc->devname, devname) == 0)) { - if (!brl_options) - preferred_dev_console = i; - set_user_specified(pc, user_specified); - return 0; - } + if (name && strcmp(pc->name, name) == 0 && pc->index == idx) + break; + if (devname && strcmp(pc->devname, devname) == 0) + break; } - if (i == MAX_PREFERRED_CONSOLES) - return -E2BIG; - if (!brl_options) - preferred_dev_console = i; - if (name) - strscpy(pc->name, name); - if (devname) - strscpy(pc->devname, devname); - pc->options = options; - set_user_specified(pc, user_specified); - braille_set_options(pc, brl_options); - pc->index = idx; - return 0; + return add_or_update_preferred_console(i, name, idx, devname, options, + brl_options, user_specified); } static int __init console_msg_format_setup(char *str) -- 2.54.0