From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-6.7 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS, URIBL_BLOCKED autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 7F78FC54E4B for ; Tue, 12 May 2020 07:14:53 +0000 (UTC) Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by mail.kernel.org (Postfix) with ESMTP id 4B31720675 for ; Tue, 12 May 2020 07:14:53 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 4B31720675 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=intel.com Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=owner-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix) id DC7E690009A; Tue, 12 May 2020 03:14:52 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id D792B900036; Tue, 12 May 2020 03:14:52 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id C8F8490009A; Tue, 12 May 2020 03:14:52 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from forelay.hostedemail.com (smtprelay0040.hostedemail.com [216.40.44.40]) by kanga.kvack.org (Postfix) with ESMTP id AE02F900036 for ; Tue, 12 May 2020 03:14:52 -0400 (EDT) Received: from smtpin14.hostedemail.com (10.5.19.251.rfc1918.com [10.5.19.251]) by forelay05.hostedemail.com (Postfix) with ESMTP id 6D216181AEF1A for ; Tue, 12 May 2020 07:14:52 +0000 (UTC) X-FDA: 76807204824.14.ice62_64810a3716244 X-HE-Tag: ice62_64810a3716244 X-Filterd-Recvd-Size: 3301 Received: from mga12.intel.com (mga12.intel.com [192.55.52.136]) by imf23.hostedemail.com (Postfix) with ESMTP for ; Tue, 12 May 2020 07:14:50 +0000 (UTC) IronPort-SDR: u8bWKn7WpeZdWaKf9r8gEuLPLhgYDJPU6EeczDGa5Bj2Ppm/YI3JYv57FMnHzsp4UNIZairVYo Fnyu+pcGGqQA== X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga004.jf.intel.com ([10.7.209.38]) by fmsmga106.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 12 May 2020 00:14:48 -0700 IronPort-SDR: V1z1KjEy6HMCzIPAuWxnhTOX5UkYMr5RHKngLgpjWwKFS+gEstjCNqBLBBz+84D82120aO5YFZ 13X3gAzlINRw== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.73,382,1583222400"; d="scan'208";a="409198422" Received: from yhuang-dev.sh.intel.com (HELO yhuang-dev) ([10.239.159.23]) by orsmga004.jf.intel.com with ESMTP; 12 May 2020 00:14:46 -0700 From: "Huang\, Ying" To: Michal Hocko Cc: Andrew Morton , , , Minchan Kim , Tim Chen , Hugh Dickins Subject: Re: [PATCH] mm, swap: Use prandom_u32_max() References: <20200512064147.514493-1-ying.huang@intel.com> <20200512065049.GN29153@dhcp22.suse.cz> Date: Tue, 12 May 2020 15:14:46 +0800 In-Reply-To: <20200512065049.GN29153@dhcp22.suse.cz> (Michal Hocko's message of "Tue, 12 May 2020 08:50:49 +0200") Message-ID: <875zd1pqu1.fsf@yhuang-dev.intel.com> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/26.1 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain; charset=ascii X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: Michal Hocko writes: > On Tue 12-05-20 14:41:46, Huang Ying wrote: >> To improve the code readability and get random number with higher >> quality. > > I understand the readability argument but why should prandom_u32_max > (which I was not aware of) provide a higher quality randomness? I am not expert on random number generator. I have heard about that the randomness of the low order bits of some random number generator isn't good enough. Anyway, by using the common implementation, the real random number generator expert can fix the possible issue once for all users. >> Signed-off-by: "Huang, Ying" >> Cc: Michal Hocko >> Cc: Minchan Kim >> Cc: Tim Chen >> Cc: Hugh Dickins > > To the change itself > Acked-by: Michal Hocko Thanks! Best Regards, Huang, Ying >> --- >> mm/swapfile.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/mm/swapfile.c b/mm/swapfile.c >> index a0a123e59ce6..2ec8b21201d6 100644 >> --- a/mm/swapfile.c >> +++ b/mm/swapfile.c >> @@ -3220,7 +3220,7 @@ SYSCALL_DEFINE2(swapon, const char __user *, specialfile, int, swap_flags) >> * select a random position to start with to help wear leveling >> * SSD >> */ >> - p->cluster_next = 1 + (prandom_u32() % p->highest_bit); >> + p->cluster_next = 1 + prandom_u32_max(p->highest_bit); >> nr_cluster = DIV_ROUND_UP(maxpages, SWAPFILE_CLUSTER); >> >> cluster_info = kvcalloc(nr_cluster, sizeof(*cluster_info), >> -- >> 2.26.2