Re: [PATCH v2 1/2] ufs: mcq: Fix missing argument 'hba' in MCQ_OPR_OFFSET_n

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

To: Minwoo Im <minwoo.im@xxxxxxxxxxx>, "James E . J . Bottomley" <James.Bottomley@xxxxxxxxxxxxxxxxxxxxx>, "Martin K . Petersen" <martin.petersen@xxxxxxxxxx>, Alim Akhtar <alim.akhtar@xxxxxxxxxxx>, Avri Altman <avri.altman@xxxxxxx>
Subject: Re: [PATCH v2 1/2] ufs: mcq: Fix missing argument 'hba' in MCQ_OPR_OFFSET_n
From: Bart Van Assche <bvanassche@xxxxxxx>
Date: Mon, 20 May 2024 11:17:01 -0700
Cc: linux-scsi@xxxxxxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx, Joel Granados <j.granados@xxxxxxxxxxx>, gost.dev@xxxxxxxxxxx, Asutosh Das <quic_asutoshd@xxxxxxxxxxx>
In-reply-to: <20240519221457.772346-2-minwoo.im@samsung.com>
User-agent: Mozilla Thunderbird

On 5/19/24 15:14, Minwoo Im wrote:

The MCQ_OPR_OFFSET_n macro has taken 'hba' on the caller context
without receiving 'hba' instance as an argument.  To prevent potential
bugs in future use cases, this patch added an argument 'hba'.


Reviewed-by: Bart Van Assche <bvanassche@xxxxxxx>

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

References:
- [PATCH v2 0/2] ufs: mcq: Fix and cleanup unsafe macros
  - From: Minwoo Im
- [PATCH v2 1/2] ufs: mcq: Fix missing argument 'hba' in MCQ_OPR_OFFSET_n
  - From: Minwoo Im

Prev by Date: Re: [PATCH 04/23] scsi: initialize scsi midlayer limits before allocating the queue
Next by Date: Re: [PATCH v2 2/2] ufs: mcq: Convert MCQ_CFG_n to a inline function
Previous by thread: [PATCH v2 1/2] ufs: mcq: Fix missing argument 'hba' in MCQ_OPR_OFFSET_n
Next by thread: [PATCH v2 2/2] ufs: mcq: Convert MCQ_CFG_n to a inline function
Index(es):
- Date
- Thread

[Index of Archives] [SCSI Target Devel] [Linux SCSI Target Infrastructure] [Kernel Newbies] [IDE] [Security] [Git] [Netfilter] [Bugtraq] [Yosemite News] [MIPS Linux] [ARM Linux] [Linux Security] [Linux RAID] [Linux ATA RAID] [Linux IIO] [Samba] [Device Mapper]