lkml.org 
[lkml]   [2022]   [Feb]   [27]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
SubjectRe: [PATCH 3/6] block, fs: assert that key paths use iovecs, and nothing else
From
On 2/27/22 13:58, Jens Axboe wrote:
>> diff --git a/block/bio.c b/block/bio.c
>> index b15f5466ce08..4679d6539e2d 100644
>> --- a/block/bio.c
>> +++ b/block/bio.c
>> @@ -1167,6 +1167,8 @@ static int __bio_iov_iter_get_pages(struct bio *bio, struct iov_iter *iter)
>> BUILD_BUG_ON(PAGE_PTRS_PER_BVEC < 2);
>> pages += entries_left * (PAGE_PTRS_PER_BVEC - 1);
>>
>> + WARN_ON_ONCE(!iter_is_iovec(iter));
>> +
>
> If these make sense, why aren't they also returning an error?
>

Oops. Got caught up in the luxury of local testing and watching the
logs, but also returning errors is of course the right answer. Will fix.

thanks,
--
John Hubbard
NVIDIA

\
 
 \ /
  Last update: 2022-02-27 23:13    [W:0.044 / U:0.268 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site