On Sat, Mar 16, 2013 at 2:24 PM, Dan Magenheimer <dan.magenhei...@oracle.com> wrote: >> From: Konrad Rzeszutek Wilk [mailto:kon...@darnok.org] >> Subject: Re: [PATCH v2 1/4] introduce zero filled pages handler >> >> > + >> > + for (pos = 0; pos < PAGE_SIZE / sizeof(*page); pos++) { >> > + if (page[pos]) >> > + return false; >> >> Perhaps allocate a static page filled with zeros and just do memcmp? > > That seems like a bad idea. Why compare two different > memory locations when comparing one memory location > to a register will do? >
Good point. I was hoping there was an fast memcmp that would do fancy SSE registers. But it is memory against memory instead of registers. Perhaps a cunning trick would be to check (as a shortcircuit) check against 'empty_zero_page' and if that check fails, then try to do the check for each byte in the code? > -- 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/
