On 07/04/2012 05:06 PM, Miklos Szeredi wrote:
> Pavel Emelyanov <xemul@xxxxxxxxxxxxx> writes:
>
>> The .writepages callback will issue writeback requests with more than one
>> page aboard. Make existing end/check code be aware of this.
>>
>> Signed-off-by: Pavel Emelyanov <xemul@xxxxxxxxxx>
>> ---
>> fs/fuse/file.c | 21 ++++++++++++++-------
>> 1 files changed, 14 insertions(+), 7 deletions(-)
>>
>> diff --git a/fs/fuse/file.c b/fs/fuse/file.c
>> index 6bf9723..47f0f2e 100644
>> --- a/fs/fuse/file.c
>> +++ b/fs/fuse/file.c
>> @@ -345,7 +345,7 @@ static bool fuse_page_is_writeback(struct inode *inode, pgoff_t index)
>>
>> BUG_ON(req->inode != inode);
>> curr_index = req->misc.write.in.offset >> PAGE_CACHE_SHIFT;
>> - if (curr_index == index) {
>> + if (curr_index == index && index < curr_index + req->num_pages) {
>
> This condition looks bogus.
Oops, indeed :( This should be
if (curr_index <= index && index < curr_index + req->num_pages)
> Thanks,
> Miklos
> .
>
Thanks,
Pavel
--
To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
