Received-SPF: pass (google.com: domain of linux-kernel+bounces-24984-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:40f1:3f00::1 as permitted sender) client-ip=2604:1380:40f1:3f00::1;
Precedence: bulk
MIME-Version: 1.0
References: <20240112121844.17580-1-2045gemini@gmail.com> <3bbabf34-1eba-8983-439e-f23e811e80a8@linux.intel.com>
In-Reply-To: <3bbabf34-1eba-8983-439e-f23e811e80a8@linux.intel.com>
From: Gui-Dong Han <2045gemini@gmail.com>
Date: Sat, 13 Jan 2024 02:52:42 +0800
Message-ID: <CAOPYjvZYAxo=-H2eeJ8mHjx4af=TetbtZ_54-iX2cokzEHFeaA@mail.gmail.com>
Subject: Re: [PATCH] serial: core: Fix double fetch in uart_throttle/uart_unthrottle
To: =?UTF-8?Q?Ilpo_J=C3=A4rvinen?= <ilpo.jarvinen@linux.intel.com>
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>, Jiri Slaby <jirislaby@kernel.org>, 
	Tony Lindgren <tony@atomide.com>, l.sanfilippo@kunbus.com, john.ogness@linutronix.de, 
	tglx@linutronix.de, Andy Shevchenko <andriy.shevchenko@linux.intel.com>, 
	LKML <linux-kernel@vger.kernel.org>, linux-serial <linux-serial@vger.kernel.org>, 
	baijiaju1990@outlook.com, stable@vger.kernel.org
Content-Type: text/plain; charset="UTF-8"

Hi

I apologize for any issues and lack of clarity in my previous patch.
In patch v2, I've revised the fix to use local variables instead of
locks and improved the description to clearly explain the harm and
potential for concurrency.

The patch was developed and tested on linux-next, not Linux 5.17. My
reference to 5.17 was due to a project I'm working on, which involves
kernel static analysis and a comparison with earlier studies that
support up to Linux 5.17. Therefore, I initially ran my tool on 5.17
to filter potential bugs that are still unaddressed in the upstream.
Then I worked on linux-next to develop and test the patch. I
understand this might have caused misunderstandings, and I'll aim for
clearer communication in future submissions.

Thanks,
Han