Re: [PATCH 10/10] fs: make aio plug
From: Jeff Moyer
Date: Mon Jan 24 2011 - 14:38:13 EST
Jens Axboe <jaxboe@xxxxxxxxxxxx> writes:
>>>> As for the iput, I don't think you will get the final iput here, since
>>>> you've just scheduled I/O against the file.
>>>
>>> Good point, so the original comment is moot - it wont make a difference.
>>> Still, may not be a bad idea to do the freeing first. I was sort-of
>>> hoping to be able to kill the batching in fs/aio.c completely, though...
>>
>> Yes, that's what I meant above when I said I agreed with the TODO item.
>> Go ahead and nuke it.
>
> Will do, if you promise to help test it :-)
Yeah, I'll sign up to do some testing.
Cheers,
Jeff
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/