Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Tue, 1 Aug 2023 11:48:09 -0700
From:   Ira Weiny <ira.weiny@intel.com>
To:     Alison Schofield <alison.schofield@intel.com>,
        Ira Weiny <ira.weiny@intel.com>
CC:     Davidlohr Bueso <dave@stgolabs.net>,
        Jonathan Cameron <jonathan.cameron@huawei.com>,
        Dave Jiang <dave.jiang@intel.com>,
        "Vishal Verma" <vishal.l.verma@intel.com>,
        Dan Williams <dan.j.williams@intel.com>,
        <linux-cxl@vger.kernel.org>, <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH] cxl/mbox: Fix debug message print
Message-ID: <64c953697f857_2616829424@iweiny-mobl.notmuch>
References: <20230731-cxl-fix-clear-event-debug-print-v1-1-42c068f500d1@intel.com>
 <ZMhesrtXf31iST/O@aschofie-mobl2>
Content-Type: text/plain; charset="us-ascii"
Content-Disposition: inline
In-Reply-To: <ZMhesrtXf31iST/O@aschofie-mobl2>
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?j2DIDFp6A4tTEtR9TkdVaxmJZm8c/9+KJh2viZD1xuTcmxgofmwYAZIrUYCa?=
 =?us-ascii?Q?c44HMOFR+NUqDZGSnTl/oRMMYfyryIex15zCTqsMF+zQYJgoaMB0GDcao3DG?=
 =?us-ascii?Q?x0FVxTHfuwRTl3N+PGRGxo2aZIxuLhvMDZT0UyMFu3u3aoR9wIpJdYxaCs8x?=
 =?us-ascii?Q?rO23Xjpddi1HwMd4EYoQCf6Txn2YKlp8riI1qDYmuuJ45Cbe8oNJ4UZClLsd?=
 =?us-ascii?Q?6UIe6neSOiX0xLR/OPzg+tpQrpn2XTGuGoLVfLof/SVxMBSud3frVU9g+lP9?=
 =?us-ascii?Q?5Ny5uH4C8UaHPrIZoZlpsiiC39ZgRsVxAfSwDOzN59iNLykaJJ8TTrUmYuBe?=
 =?us-ascii?Q?3Dj0/mSForIxFtoY6UpHoci9sLuaHJ6J++H8WrZtjCAA0DsMV9YB55Q+eHFe?=
 =?us-ascii?Q?J+J3W2xRZXtHY6ocK7h5KmBrUOGjA4DbnyAFe+UCDKGkogEfDeN7+1ANfWSM?=
 =?us-ascii?Q?rq9ZMSBmedG3TG5RXKiZ0TcuMmrrn+1JQY11IyLch1h1YSbSgl7BSW6mjgrr?=
 =?us-ascii?Q?WefRrMy5LcgO71K6LzSFAqCSEAfNQibbHsGeffwH6APRg9cNreCbm+Rw4VyX?=
 =?us-ascii?Q?K0gVRkzDM0Or2p01HKkfxeDJVgL7FyB0yBCk1w4iEvLhdl+8MbkngER5DY3C?=
 =?us-ascii?Q?1SCxtZHBve94bSGI0LIXGASImqB4sE/z9tH+AzGVKnzc5ARd//YmaArqwYRp?=
 =?us-ascii?Q?D8Ama7GIURZftWTmU1pnxb1t8+sGR1Gppe9BRxfF9wMUDUUEllxtaoPSw2/0?=
 =?us-ascii?Q?g2VU5RGXZzqg+heL/hVN2ewcalYKHOJCk+3h78IyXpv+rbMBgOeA3Aw8BAZk?=
 =?us-ascii?Q?N2AQPR2DQ27/k0SsQAQXuWSRjM4ZypjYhitakIJw9CsjZLNZZ/PN5iDyNGfz?=
 =?us-ascii?Q?WWsWEOIeQmanmzDVRBPcXQyA8fskxhODuFu2trbJwob3JhTzPozVSwftPzrX?=
 =?us-ascii?Q?TB420oiZFALAUFyr5lVn6SyEZkS3egT08++9g8EfGYbhc8LSTLNIEB4LXZ+K?=
 =?us-ascii?Q?v/S4u0AECxb0mrD63rQbWaHIbQvGAt551JvlM0nxjLVRpeInELOMGXUy6Lgz?=
 =?us-ascii?Q?MLoDDaKU07rporIJgj/2tZaM7MBY28fBfQuOmBemOyhPykR+3XDWwXvFxwNZ?=
 =?us-ascii?Q?IJ59/4wLZlUIQ9Omrjqpjr/yp2YdtIpQ5QWFO9/yaDA0VBUdq9sbfmw4UB4J?=
 =?us-ascii?Q?4d9YASHV5RsFnI9nMTI8SnvVI9HpoYuOBqweZmYbtK5KK3Vwy00kBB2mLL4C?=
 =?us-ascii?Q?VqDoXucwP5WrEIen5qhqXu1zoh6Spd1pGA0IUArP8tpyeD9L7Wtb9aHKNrXN?=
 =?us-ascii?Q?Q7hcVRqQqUU9OvGJ118vgaghusgzu2r7rOHKDzBndM5fIzcbrcmMhnf1cuUk?=
 =?us-ascii?Q?xcn0bPkbYm0D/xNnAaSe6uilIL8uT0fbEB8NgN2lrvlPSvZQxWbhU7TAhkii?=
 =?us-ascii?Q?x6ycjguPPYz8GAfQoXU9Jf4AJe7WnlodMHa3pbfUVe3jJjX3pZQWu0RePxzK?=
 =?us-ascii?Q?GuTkVFhwgTp21r3R8cj+sUlcjTckOnbhN3T9QBexnXfQrs1SOlMhpvIuEAoc?=
 =?us-ascii?Q?r3gigBnNyVCVm8Fsa35sxsa3FX9Scsll8yBhoveP?=
X-MS-Exchange-CrossTenant-Network-Message-Id: 194df60b-a264-46f6-1abe-08db92bfdbba
X-MS-Exchange-CrossTenant-AuthSource: SA1PR11MB6733.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 01 Aug 2023 18:48:12.7640
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: uPaFes7cd9cw4wJaQMxBmTG7zAN/LgurpYidzVBK4/UmFERJYs5/fDVESVKpCv2T29LcncfZOKSFmNQZPgWc9g==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: IA1PR11MB7823
X-OriginatorOrg: intel.com
X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,
        DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,
        RCVD_IN_DNSWL_BLOCKED,SPF_HELO_NONE,SPF_NONE,T_SCC_BODY_TEXT_LINE,
        URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Alison Schofield wrote:
> On Mon, Jul 31, 2023 at 01:52:27PM -0700, Ira Weiny wrote:
> 
> Looks like this is rolling for the distro list, so..
> 
> Please be more specific than this:
> [PATCH] cxl/mbox: Fix debug message print
> 
> Something like this is more useful to folks scanning the one-liners:
> [PATCH] cxl/mbox: Use correct handle in events debug message

That is better thanks!

> 
> > The handle value used to report an event being cleared by dev_dbg() is
> > incorrect due to a post increment of the payload handle index.
> > 
> > Delay the increment of the index until after the print.  Also add the
> > debugging for event processing which was useful in finding this bug.
> > 
> 
> "Also" always smells like something that should be a separate patch.

It could be but I did not see this as something which warranted its own
patch as it lead me to fix the bug.  Since Dan feels the fix should be
backported that changes my attitude.

> 
> I guess you could rewrite the commit message and keep it in one patch:
> [PATCH] cxl/mbox: Improve event dev_dbg() messages

Except that misses that this is a bug fix.

Ira