Re: [PATCH 1/3] block: Allow mapping of vmalloc-ed buffers

To: Chaitanya Kulkarni <Chaitanya.Kulkarni@xxxxxxx>, Damien Le Moal <Damien.LeMoal@xxxxxxx>, "linux-scsi@xxxxxxxxxxxxxxx" <linux-scsi@xxxxxxxxxxxxxxx>, "Martin K . Petersen" <martin.petersen@xxxxxxxxxx>, "linux-block@xxxxxxxxxxxxxxx" <linux-block@xxxxxxxxxxxxxxx>, Jens Axboe <axboe@xxxxxxxxx>
Subject: Re: [PATCH 1/3] block: Allow mapping of vmalloc-ed buffers
From: Bart Van Assche <bvanassche@xxxxxxx>
Date: Tue, 25 Jun 2019 08:59:08 -0700
Cc: Christoph Hellwig <hch@xxxxxx>
In-reply-to: <BYAPR04MB5749C9178CB54B4A488408A986E30@BYAPR04MB5749.namprd04.prod.outlook.com>
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.7.1

On 6/24/19 8:24 PM, Chaitanya Kulkarni wrote:

nit:- Can we use is_vmalloc_addr() call directly so that
"if (is_vmalloc)" ->  "if (is_vmalloc_addr(data))" and remove is_vmalloc
variable.

That would change a single call of is_vmalloc_addr() into multiple?

Bart.

Follow-Ups:
- Re: [PATCH 1/3] block: Allow mapping of vmalloc-ed buffers
  - From: Chaitanya Kulkarni

References:
- [PATCH 0/3] Fix zone revalidation memory allocation failures
  - From: Damien Le Moal
- [PATCH 1/3] block: Allow mapping of vmalloc-ed buffers
  - From: Damien Le Moal
- Re: [PATCH 1/3] block: Allow mapping of vmalloc-ed buffers
  - From: Chaitanya Kulkarni