Re: splice: move balance_dirty_pages_ratelimited() outside of splice actor

2007-06-12 Thread Jens Axboe
On Tue, Jun 12 2007, Andrew Morton wrote: > On Tue, 12 Jun 2007 20:15:41 +0200 > Jens Axboe <[EMAIL PROTECTED]> wrote: > > > On Tue, Jun 12 2007, Jens Axboe wrote: > > > On Tue, Jun 12 2007, Andrew Morton wrote: > > > > On Tue, 12 Jun 2007 14:44:50 +0200 Jens Axboe <[EMAIL PROTECTED]> wrote: > > >

Re: splice: move balance_dirty_pages_ratelimited() outside of splice actor

2007-06-12 Thread Andrew Morton
On Tue, 12 Jun 2007 20:15:41 +0200 Jens Axboe <[EMAIL PROTECTED]> wrote: > On Tue, Jun 12 2007, Jens Axboe wrote: > > On Tue, Jun 12 2007, Andrew Morton wrote: > > > On Tue, 12 Jun 2007 14:44:50 +0200 Jens Axboe <[EMAIL PROTECTED]> wrote: > > > > > > > splice > > > > > > btw, I'm staring in prof

Re: splice: move balance_dirty_pages_ratelimited() outside of splice actor

2007-06-12 Thread Jens Axboe
On Tue, Jun 12 2007, Jens Axboe wrote: > On Tue, Jun 12 2007, Andrew Morton wrote: > > On Tue, 12 Jun 2007 14:44:50 +0200 Jens Axboe <[EMAIL PROTECTED]> wrote: > > > > > splice > > > > btw, I'm staring in profound mystification at this: > > > > int generic_pipe_buf_steal(struct pipe_inode_info *

Re: splice: move balance_dirty_pages_ratelimited() outside of splice actor

2007-06-12 Thread Jens Axboe
On Tue, Jun 12 2007, Andrew Morton wrote: > On Tue, 12 Jun 2007 14:44:50 +0200 Jens Axboe <[EMAIL PROTECTED]> wrote: > > > splice > > btw, I'm staring in profound mystification at this: > > int generic_pipe_buf_steal(struct pipe_inode_info *pipe, > struct pipe_buffer *bu

Re: splice: move balance_dirty_pages_ratelimited() outside of splice actor

2007-06-12 Thread Andrew Morton
On Tue, 12 Jun 2007 14:44:50 +0200 Jens Axboe <[EMAIL PROTECTED]> wrote: > splice btw, I'm staring in profound mystification at this: int generic_pipe_buf_steal(struct pipe_inode_info *pipe, struct pipe_buffer *buf) { struct page *page = buf->page; if

Re: splice: move balance_dirty_pages_ratelimited() outside of splice actor

2007-06-12 Thread Andrew Morton
On Tue, 12 Jun 2007 14:44:50 +0200 Jens Axboe <[EMAIL PROTECTED]> wrote: > On Tue, Jun 12 2007, Peter Zijlstra wrote: > > On Tue, 2007-06-12 at 14:10 +0200, Jens Axboe wrote: > > > On Tue, Jun 12 2007, Peter Zijlstra wrote: > > > > On Tue, 2007-06-12 at 13:31 +0200, Jens Axboe wrote: > > > > > >

Re: splice: move balance_dirty_pages_ratelimited() outside of splice actor

2007-06-12 Thread Jens Axboe
On Tue, Jun 12 2007, Peter Zijlstra wrote: > On Tue, 2007-06-12 at 14:10 +0200, Jens Axboe wrote: > > On Tue, Jun 12 2007, Peter Zijlstra wrote: > > > On Tue, 2007-06-12 at 13:31 +0200, Jens Axboe wrote: > > > > > > > Would you prefer this change, then? I'd prefer keeping the current code, > > > >

Re: splice: move balance_dirty_pages_ratelimited() outside of splice actor

2007-06-12 Thread Peter Zijlstra
On Tue, 2007-06-12 at 14:10 +0200, Jens Axboe wrote: > On Tue, Jun 12 2007, Peter Zijlstra wrote: > > On Tue, 2007-06-12 at 13:31 +0200, Jens Axboe wrote: > > > > > Would you prefer this change, then? I'd prefer keeping the current code, > > > unless it's absolutely critical that we call > > > bal

Re: splice: move balance_dirty_pages_ratelimited() outside of splice actor

2007-06-12 Thread Jens Axboe
On Tue, Jun 12 2007, Peter Zijlstra wrote: > On Tue, 2007-06-12 at 13:31 +0200, Jens Axboe wrote: > > > Would you prefer this change, then? I'd prefer keeping the current code, > > unless it's absolutely critical that we call > > balance_dirty_pages_ratelimited() for each and every page instead of

Re: splice: move balance_dirty_pages_ratelimited() outside of splice actor

2007-06-12 Thread Peter Zijlstra
On Tue, 2007-06-12 at 13:31 +0200, Jens Axboe wrote: > Would you prefer this change, then? I'd prefer keeping the current code, > unless it's absolutely critical that we call > balance_dirty_pages_ratelimited() for each and every page instead of eg > every 16 pages here. For that we should call:

Re: splice: move balance_dirty_pages_ratelimited() outside of splice actor

2007-06-12 Thread Jens Axboe
On Tue, Jun 12 2007, Jens Axboe wrote: > On Tue, Jun 12 2007, Jens Axboe wrote: > > On Mon, Jun 11 2007, Andrew Morton wrote: > > > On Mon, 11 Jun 2007 21:59:15 GMT > > > Linux Kernel Mailing List wrote: > > > > > > > Gitweb: > > > > http://git.kernel.org/git/?p=linux/kernel/git/torvalds/lin

Re: splice: move balance_dirty_pages_ratelimited() outside of splice actor

2007-06-12 Thread Jens Axboe
On Tue, Jun 12 2007, Jens Axboe wrote: > On Mon, Jun 11 2007, Andrew Morton wrote: > > On Mon, 11 Jun 2007 21:59:15 GMT > > Linux Kernel Mailing List wrote: > > > > > Gitweb: > > > http://git.kernel.org/git/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=20d698db67059a63d217030dfd02872

Re: splice: move balance_dirty_pages_ratelimited() outside of splice actor

2007-06-11 Thread Jens Axboe
On Mon, Jun 11 2007, Andrew Morton wrote: > On Mon, 11 Jun 2007 21:59:15 GMT > Linux Kernel Mailing List wrote: > > > Gitweb: > > http://git.kernel.org/git/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=20d698db67059a63d217030dfd02872cb5f88dfb > > Commit: 20d698db67059a63d217030df

Re: splice: move balance_dirty_pages_ratelimited() outside of splice actor

2007-06-11 Thread Andrew Morton
On Mon, 11 Jun 2007 21:59:15 GMT Linux Kernel Mailing List wrote: > Gitweb: > http://git.kernel.org/git/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=20d698db67059a63d217030dfd02872cb5f88dfb > Commit: 20d698db67059a63d217030dfd02872cb5f88dfb > Parent: 17374ff1aa9ce2a0597416a1