Re: Review: Don't use kmap() in xfs_iozero().

[Christoph Hellwig <hch@xxxxxxxxxxxxx>]

On Fri, Feb 02, 2007 at 04:40:44PM +1100, David Chinner wrote:
> kmap is inefficient and does scale well. kmap_atomic() is a better
> choice. Use the generic wrapper function instead of open coding the
> kmap-memset-dcache flush-kumap stuff.  Suggested by Andrew Morton.
> 
> Comments?

Looks good.

> +             memclear_highpage_flush(page, (unsigned int)offset, bytes);

Do you need the cast here?  An unsigned long should be automatically
demoted to an unsigned int when passing it as an argument.