[PATCH] nvme: module parameter to disable pi checks
David Wei
dw at davidwei.uk
Wed Oct 23 14:21:46 PDT 2024
On 2024-10-23 13:21, Keith Busch wrote:
> On Wed, Oct 23, 2024 at 08:01:43PM +0000, Chaitanya Kulkarni wrote:
>> @@ -1774,9 +1773,9 @@ static bool nvme_init_integrity(struct
>> nvme_ns_head *head,
>> return nvme_ns_has_pi(head);
>>
>> if (disable_pi)
>> - pi_type = 0;
>> + goto out;
>
> I'm not against 'goto' in principle, but I try to restrict it to
> unwinding an error, and this isn't an error. I know, personal preference
> and all...
Agreed. This is more unreadable and a modern compiler will most likely
do the right thing anyway.
More information about the Linux-nvme
mailing list