Re: [PATCH] mm: extend zero pages to same element pages for zram

From: zhouxianrong
Date: Sun Jan 22 2017 - 22:33:08 EST


hey Joonsoo:
i would test and give the same element type later.

On 2017/1/23 10:58, Joonsoo Kim wrote:
Hello,

On Sun, Jan 22, 2017 at 10:58:38AM +0800, zhouxianrong wrote:
1. memset is just set a int value but i want to set a long value.

Sorry for late review.

Do we really need to set a long value? I cannot believe that
long value is repeated in the page. Value repeatition is
usually done by value 0 or 1 and it's enough to use int. And, I heard
that value 0 or 1 is repeated in Android. Could you check the distribution
of the value in the same page?

Thanks.


.