Re: [patch 5/5] Optimize page_mkclean_one

2007-07-02 Thread Martin Schwidefsky
On Sun, 2007-07-01 at 15:27 +0200, Peter Zijlstra wrote: > > But I could easily be overlooking something: Peter will recall. > > /me tries to get his brain up to speed after the OLS closing party :-) Oh-oh, the Black Thorn party :-) > I did both pte_dirty and pte_write because I was extra carefu

Re: [patch 5/5] Optimize page_mkclean_one

2007-07-01 Thread Martin Schwidefsky
On Sun, 2007-07-01 at 09:54 +0100, Hugh Dickins wrote: > On Sun, 1 Jul 2007, Martin Schwidefsky wrote: > > > > > > Expect you're right, but I _really_ don't want to comment, when I don't > > > understand that "|| pte_write" in the first place, and don't know the > > > consequence of pte_dirty && !

Re: [patch 5/5] Optimize page_mkclean_one

2007-07-01 Thread Peter Zijlstra
On Sun, 2007-07-01 at 09:54 +0100, Hugh Dickins wrote: > But I could easily be overlooking something: Peter will recall. /me tries to get his brain up to speed after the OLS closing party :-) I did both pte_dirty and pte_write because I was extra careful. One _should_ imply the other, but since

Re: [patch 5/5] Optimize page_mkclean_one

2007-07-01 Thread Miklos Szeredi
> page_mkclean_one is used to clear the dirty bit and to set the write > protect bit of a pte. In additions it returns true if the pte either > has been dirty or if it has been writable. As far as I can see the > function should return true only if the pte has been dirty, or page > writeback will n

Re: [patch 5/5] Optimize page_mkclean_one

2007-07-01 Thread Hugh Dickins
On Sun, 1 Jul 2007, Martin Schwidefsky wrote: > > > > Expect you're right, but I _really_ don't want to comment, when I don't > > understand that "|| pte_write" in the first place, and don't know the > > consequence of pte_dirty && !pte_write or !pte_dirty && pte_write there. > > The pte_write()

Re: [patch 5/5] Optimize page_mkclean_one

2007-07-01 Thread Martin Schwidefsky
On Sat, 2007-06-30 at 15:04 +0100, Hugh Dickins wrote: > > Oh yes, the dirty handling is tricky. I had to fix a really nasty bug > > with it lately. As for page_mkclean_one the difference is that it > > doesn't claim a page is dirty if only the write protect bit has not been > > set. If we manage t

Re: [patch 5/5] Optimize page_mkclean_one

2007-06-30 Thread Hugh Dickins
On Fri, 29 Jun 2007, Martin Schwidefsky wrote: > On Fri, 2007-06-29 at 19:56 +0100, Hugh Dickins wrote: > > I don't dare comment on your page_mkclean_one patch (5/5), > > that dirty page business has grown too subtle for me. > > Oh yes, the dirty handling is tricky. I had to fix a really nasty bug

[patch 5/5] Optimize page_mkclean_one

2007-06-29 Thread Martin Schwidefsky
From: Martin Schwidefsky <[EMAIL PROTECTED]> page_mkclean_one is used to clear the dirty bit and to set the write protect bit of a pte. In additions it returns true if the pte either has been dirty or if it has been writable. As far as I can see the function should return true only if the pte has