On Mon, Jan 29, 2007 at 11:33:03AM +0100, Nick Piggin wrote:
> +             } else {
> +                     char *src, *dst;
> +                     src = kmap(src_page);
> +                     dst = kmap(page);
> +                     memcpy(dst + offset,
> +                             src + ((unsigned long)buf & ~PAGE_CACHE_MASK),
> +                             bytes);
> +                     kunmap(page);
> +                     kunmap(src_page);
> +                     copied = bytes;
> +             }
>               flush_dcache_page(page);

Hmm, I guess these should use kmap_atomic with KM_USER[01]?

The kmap is from an earlier iteration that wanted to sleep
with the page mapped into kernel.
-
To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to