Re: [PATCH] fs/fuse/dev.c: use zero_user_page instead

2007-07-20 Thread Dave Young
On 7/20/07, rae l <[EMAIL PROTECTED]> wrote: On 7/20/07, Dave Young <[EMAIL PROTECTED]> wrote: > > - if (page && zeroing && count < PAGE_SIZE) { > > - void *mapaddr = kmap_atomic(page, KM_USER1); > > - memset(mapaddr, 0, PAGE_SIZE); > > - kunmap_atom

Re: [PATCH] fs/fuse/dev.c: use zero_user_page instead

2007-07-20 Thread rae l
On 7/20/07, Dave Young <[EMAIL PROTECTED]> wrote: > - if (page && zeroing && count < PAGE_SIZE) { > - void *mapaddr = kmap_atomic(page, KM_USER1); > - memset(mapaddr, 0, PAGE_SIZE); > - kunmap_atomic(mapaddr, KM_USER1); > - } > + if (pag

Re: [PATCH] fs/fuse/dev.c: use zero_user_page instead

2007-07-20 Thread Dave Young
On 7/20/07, Denis Cheng <[EMAIL PROTECTED]> wrote: Signed-off-by: Denis Cheng <[EMAIL PROTECTED]> --- I'm not very sure zero_user_page is correctly used here, so please feel free to give comments. and why here it uses KM_USER1 not KM_USER0, What are the differences? fs/fuse/dev.c |8 +++

[PATCH] fs/fuse/dev.c: use zero_user_page instead

2007-07-20 Thread Denis Cheng
Signed-off-by: Denis Cheng <[EMAIL PROTECTED]> --- I'm not very sure zero_user_page is correctly used here, so please feel free to give comments. and why here it uses KM_USER1 not KM_USER0, What are the differences? fs/fuse/dev.c |8 +++- 1 files changed, 3 insertions(+), 5 deletions(-)