Received: by 2002:a05:7412:ba23:b0:fa:4c10:6cad with SMTP id jp35csp796673rdb; Thu, 18 Jan 2024 21:47:28 -0800 (PST) X-Google-Smtp-Source: AGHT+IHKL0oWLwLzVQRCkoahKb2tPveR/qnw/e/oVPD678FBbBg3XP02yp3/HpNZ6KEauIPIwr4j X-Received: by 2002:a05:620a:1790:b0:783:758d:6d63 with SMTP id ay16-20020a05620a179000b00783758d6d63mr1357002qkb.26.1705643247969; Thu, 18 Jan 2024 21:47:27 -0800 (PST) ARC-Seal: i=2; a=rsa-sha256; t=1705643247; cv=pass; d=google.com; s=arc-20160816; b=XgkI9KbO4lHut8aw2OWUQxo7l0a+W3WTU7Q0KNV52MNvu1dc+xxP1Jqg6GrBRncmKs 36/eOzegC/DK+TMXIEipGXOhHKicOvV/F8O5XyVFqGa5O6LxV94FajhWgxU04HADVoFq LNEdoLl9MIqKmv2EbOQSkKfHa1hqXBEETF92/IlXCfUXxULcz78MF1UtV3pyyONfLdOl zFNvXqbdQVLqz1lEjRD+1RzJXj+r3pzaJXO9yaAh5turZ2PLyZYnaQ4+MZ7Qf7f9Rgsh WUC+fD5Zbefkmxqbgs5380c4jbG4ovs7gqzhIt7PXxecMNrwcaOVTbQAqYdWqvmm933d wQmA== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=in-reply-to:content-disposition:mime-version:list-unsubscribe :list-subscribe:list-id:precedence:references:message-id:subject:cc :to:from:date:dkim-signature; bh=7wWsGShU/Vha8cm6cBxkBJALMxhsZojtXeKjQQ0DoZI=; fh=5rkvPF38uHV8b/jOvlWyoBd5+jLKngeA0EwB3OIgKd4=; b=oa7ipu9Q+SDYSldAbepCYYA3r4TxCetN3d1KzkYJzjJ81ZlLl03ZDqSLC6ePF7DJyz q9sA8FiWv+ZmOqJfXgtJJZFvJNaajpX1rflVf7lPQtCZ2wdvZa4n3vih0npu8KdG26Uj HQ0W4b721Qm+DlQa9Pj5EgYtatRg6nAS8tPKluyTtz7r63+9wWWMI0nbiYRzZb042c/w OaIg1+0Q0J67AJMtOWmFeJmBD+sLM7dnISfP90AzEbt1Ynmx0EgxCFsosae6N2Z2Rqdd E3ScUM7KmAXZUQU2Nlk9mUBxP7hwwJ0gUMlM3EaT4tcqpWlcGEwsjqm8Eg1jU5WJ11nx y2Wg== ARC-Authentication-Results: i=2; mx.google.com; dkim=pass header.i=@linuxfoundation.org header.s=korg header.b=D10owxLI; arc=pass (i=1 dkim=pass dkdomain=linuxfoundation.org); spf=pass (google.com: domain of linux-kernel+bounces-30784-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) smtp.mailfrom="linux-kernel+bounces-30784-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linuxfoundation.org Return-Path: Received: from ny.mirrors.kernel.org (ny.mirrors.kernel.org. [147.75.199.223]) by mx.google.com with ESMTPS id i4-20020a05620a248400b00781bac03862si16558881qkn.96.2024.01.18.21.47.27 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 18 Jan 2024 21:47:27 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel+bounces-30784-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) client-ip=147.75.199.223; Authentication-Results: mx.google.com; dkim=pass header.i=@linuxfoundation.org header.s=korg header.b=D10owxLI; arc=pass (i=1 dkim=pass dkdomain=linuxfoundation.org); spf=pass (google.com: domain of linux-kernel+bounces-30784-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) smtp.mailfrom="linux-kernel+bounces-30784-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linuxfoundation.org Received: from smtp.subspace.kernel.org (wormhole.subspace.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ny.mirrors.kernel.org (Postfix) with ESMTPS id B5E641C22DEE for ; Fri, 19 Jan 2024 05:47:27 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 8D1934A3C; Fri, 19 Jan 2024 05:47:17 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=linuxfoundation.org header.i=@linuxfoundation.org header.b="D10owxLI" Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id AB24B6FB1; Fri, 19 Jan 2024 05:47:16 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1705643236; cv=none; b=JIjEEaUNlgZXTByMMMjxmHcJdK2cYRrI56+MSbzh3hsFvmaeXXINTHfwuVXLPoDoRC2fyv8Tqkj1OCyZ5oriAPX8E3y8gsgoSUQxJ9I8kFYx9mef4JkRqueTgbbtxuHIryZUtWqxwfmglllscMtUNTCtpBuK5nW1cdGzplhDhNg= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1705643236; c=relaxed/simple; bh=MNDvuNwiiy4+YlotPmen7Gor1toPlR5dd2opwcIDllI=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=GCw4I+NAoogBYKegmn/mY+s1cvdLsV1VCqD412emHEx4m9iHVEkHwrRMr9yHu9qovDZyxwiAhPRPi+e1g9xf12Se/QDBxWcqHMxsiuOwtvW0KoVGCEjPpXxBDIWtu848ATX2jQimZ69YGeY5kcmtSjRMGcBJa2asvqy9dcbQbWM= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=linuxfoundation.org header.i=@linuxfoundation.org header.b=D10owxLI; arc=none smtp.client-ip=10.30.226.201 Received: by smtp.kernel.org (Postfix) with ESMTPSA id 7FFC9C433C7; Fri, 19 Jan 2024 05:47:15 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=linuxfoundation.org; s=korg; t=1705643236; bh=MNDvuNwiiy4+YlotPmen7Gor1toPlR5dd2opwcIDllI=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=D10owxLIHodiEVHT3LPgLIOdoqbvjy5pjLddXK71vyCfPvYa/zohlwClepQHngu/e ZzJgAurcNjaFM1b2oQwjlXOQg1VioPBsluBsWOhCqCtvFPNmRml7unBY2KURQpYGZr QWCALOCSEfGjhmz4Z9ixFoAegq3yiOAqlJgKqaEU= Date: Fri, 19 Jan 2024 06:47:13 +0100 From: "gregkh@linuxfoundation.org" To: David Laight Cc: 'Sreenath Vijayan' , "john.ogness@linutronix.de" , "corbet@lwn.net" , "jirislaby@kernel.org" , "rdunlap@infradead.org" , "pmladek@suse.com" , "rostedt@goodmis.org" , "senozhatsky@chromium.org" , "linux-doc@vger.kernel.org" , "linux-kernel@vger.kernel.org" , "linux-serial@vger.kernel.org" , "taichi.shimoyashiki@sony.com" , "daniel.palmer@sony.com" , "anandakumar.balasubramaniam@sony.com" Subject: Re: [PATCH v3 2/2] tty/sysrq: Dump printk ring buffer messages via sysrq Message-ID: <2024011958-coerce-niece-6301@gregkh> References: <57daf43c5270f7532b269b9f0e90d126ca012354.1705331453.git.sreenath.vijayan@sony.com> <57f8c1f40ccc4a4f9d29e97f50a36b4f@AcuMS.aculab.com> 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=us-ascii Content-Disposition: inline In-Reply-To: <57f8c1f40ccc4a4f9d29e97f50a36b4f@AcuMS.aculab.com> On Thu, Jan 18, 2024 at 10:56:59PM +0000, David Laight wrote: > From: Sreenath Vijayan > > Sent: 17 January 2024 11:14 > .... > > /* Key Operations table and lock */ > > static DEFINE_SPINLOCK(sysrq_key_table_lock); > > > > @@ -505,7 +523,7 @@ static const struct sysrq_key_op *sysrq_key_table[62] = { > > NULL, /* A */ > > NULL, /* B */ > > NULL, /* C */ > > - NULL, /* D */ > > + &sysrq_dmesg_dump_op, /* D */ > > NULL, /* E */ > > NULL, /* F */ > > NULL, /* G */ > > That looks like it ought to use C99 initialisers: > ['D' - 'A'] = &sysrq_dmesg_dump_op, > > Possible with a #define to hide the offset. Maybe in the future, but for now, let's leave it as-is please. thanks, greg k-h