On Tue, 14 Apr 2009, Jamie Lokier wrote: > Miklos Szeredi wrote: > > Remove the now unused generic_file_splice_write_nolock() function. > > It's conceptually broken anyway, because splice may need to wait for > > pipe events so holding locks across the whole operation is wrong. > > Did the conceptual brokenness affect userspace behaviour of > splice/sendfile at all? Splice: yes, sendfile: no. Sendfile uses an internal pipe and it always makes sure there's no blocking on that pipe. With splice it's up to the process feeding the pipe to make sure there are buffers for the splice to consume. If it the pipe is not fed properly then the splice call might hang, waiting for data, keeping any locks surrounding generic_file_splice_write_nolock() locked. Which in turn will block other operations from proceeding. Not what we want. Thanks, Miklos -- 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
