lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <8aff7086-5cf4-4212-b97f-cf0bffd79440@acm.org>
Date: Thu, 27 Mar 2025 07:40:14 -0400
From: Bart Van Assche <bvanassche@....org>
To: Neil Armstrong <neil.armstrong@...aro.org>,
 Alim Akhtar <alim.akhtar@...sung.com>, Avri Altman <avri.altman@....com>,
 "James E.J. Bottomley" <James.Bottomley@...senPartnership.com>,
 "Martin K. Petersen" <martin.petersen@...cle.com>
Cc: Manivannan Sadhasivam <manivannan.sadhasivam@...aro.org>,
 linux-arm-msm@...r.kernel.org, linux-scsi@...r.kernel.org,
 linux-kernel@...r.kernel.org
Subject: Re: [PATCH RFC v2 1/2] ufs: core: drop last_intr_status/ts stats

On 3/26/25 4:36 AM, Neil Armstrong wrote:
> Drop last_intr_status & last_intr_ts drop the ufs_stats struct,
> and the associated debug code.

Patch descriptions should not only explain what has been changed but
also why a change is being made. In this case, this change prepares for
making an interrupt handler threaded. If this patch series has to be 
resent, please add this information to the patch description. Anyway,
since the patch itself looks good to me:

Reviewed-by: Bart Van Assche <bvanassche@....org>


Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ