From: Andrew Morton on
On Mon, 26 Jul 2010 16:25:42 +0800
wzt.wzt(a)gmail.com wrote:

> mapping->a_ops->direct_IO() is not checked, if it's a NULL pointer,
> that will casue an oops. pagecache_write_begin/end is exported to
> other functions, so they need to check null pointer before use them.
>

The patch checks a lot more things than ->directIO!

It would be best to not add this overhead if possible. Did you
actually observe an oops? If so, please fully describe it.

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo(a)vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/