Re: [PATCH] iomap: get/put the page in iomap_page_create/release()

[Piotr Jaroszynski <pjaroszynski@xxxxxxxxxx>]

On 11/15/18 1:30 AM, Christoph Hellwig wrote:
>> Fixes: 82cb14175e7d ("xfs: add support for sub-pagesize writeback
>>                       without buffer_heads")
> 
> I've never seen line breaks in Fixes tags, is this really a valid format?

Probably not, fixed in v2.

> 
>> +	/*
>> +	 * At least migrate_page_move_mapping() assumes that pages with private
>> +	 * data have their count elevated by 1.
>> +	 */
> 
> I'd drop the "At least".

Fixed in v2.

> 
> Otherwise this looks fine to me:
> 
> Reviewed-by: Christoph Hellwig <hch@xxxxxx>
> 

Thanks!

Thanks,
Piotr