On Fri, 2013-06-07 at 13:43 -0700, Andrew Morton wrote: > On Sat, 8 Jun 2013 00:23:18 +0400 Artem Savkov <artem.sav...@gmail.com> > wrote: [] > +++ a/mm/page_io.c [] > + /* > + * There is no guarantee that the page is in swap cache - the software > + * suspend code (at least) uses end_swap_bio_read() against a non- > + * swapcache page. So we must check PG_swapcache before proceeding with > + * this optimization. > + */ > + if (likely(PageSwapCache(page))) {
or if (unlikely(!PageSwapCache(page))) goto out; to save an indent level > +out: > unlock_page(page); > bio_put(bio); > } -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/