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=-23.3 required=3.0 tests=BAYES_00,DKIMWL_WL_MED, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS, INCLUDES_CR_TRAILER,INCLUDES_PATCH,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS, USER_IN_DEF_DKIM_WL autolearn=ham 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 ACAE7C12002 for ; Fri, 16 Jul 2021 20:50:15 +0000 (UTC) Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by mail.kernel.org (Postfix) with ESMTP id 25D3F6128B for ; Fri, 16 Jul 2021 20:50:15 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 25D3F6128B Authentication-Results: mail.kernel.org; dmarc=fail (p=reject dis=none) header.from=google.com Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=owner-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix) id 76A0F8D00F4; Fri, 16 Jul 2021 16:50:15 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 6F2B08D00EC; Fri, 16 Jul 2021 16:50:15 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 5458F8D00F4; Fri, 16 Jul 2021 16:50:15 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from forelay.hostedemail.com (smtprelay0187.hostedemail.com [216.40.44.187]) by kanga.kvack.org (Postfix) with ESMTP id 29AEF8D00EC for ; Fri, 16 Jul 2021 16:50:15 -0400 (EDT) Received: from smtpin13.hostedemail.com (10.5.19.251.rfc1918.com [10.5.19.251]) by forelay02.hostedemail.com (Postfix) with ESMTP id CDFD62BA73 for ; Fri, 16 Jul 2021 20:50:13 +0000 (UTC) X-FDA: 78369643506.13.D1EF513 Received: from mail-qv1-f50.google.com (mail-qv1-f50.google.com [209.85.219.50]) by imf16.hostedemail.com (Postfix) with ESMTP id 90479F00008C for ; Fri, 16 Jul 2021 20:50:13 +0000 (UTC) Received: by mail-qv1-f50.google.com with SMTP id p10so5239735qvk.7 for ; Fri, 16 Jul 2021 13:50:13 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20161025; h=date:from:to:cc:subject:in-reply-to:message-id:references :mime-version; bh=eP+vdjS8P2oSI7HEDS+oh3O0Y+BLb63u8Xg1Z86gM8g=; b=OyZEvDSzk2JzXO1WlOoiW9zJM0/OC3IPYXIxFo7A1ItTKDB7ZkYWs6Z0LNEDzV0wP7 an2p0lafJ7R9/5GOEoUxx5Wv0KlatzZ9CYV3rmS+L/a/zpw1jlBEggj25yOGpnVt0+oh EvepArkPWyoYKLYU+SlIEbaEji7vbAdaUXBCLxfQyr0487znAEYX9qEmGTmYQz/UqSJr LOAgxeypTy0xCyTrlf6kvHmbzA4hMxzYGl6WmAlFO0Oeiwq98cSF6VBK3Un4xq0mq6M1 LCY57MDqkIqbLVf1IieX34spfe6AD4tfqsIO3oafpmk3Uq/qh78j7mSFT5yKpyORO8Mi u02A== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:in-reply-to:message-id :references:mime-version; bh=eP+vdjS8P2oSI7HEDS+oh3O0Y+BLb63u8Xg1Z86gM8g=; b=KhY6b6lxHDstXWr0JV/ijB0KP31APnd0A2+sBC0b5PhY8o+/9qDt6+EKIvBhXNUx7F KiUJFSLxWMyDZFk54CmWFMi5L9KdRzSisY+yu6HkACkQ5XAb1uByK5QKtz6piEcDa3rR EoVjWiMGSN/GnDL0DLQx2hZkuz70qxor+Zea40YAVzYVjtKOip3J3p4sNP9dtjxxIGGC kZ9qZBsY7hNjL7Nmyb/c4a2Fxh9pqccZtJDc9QCPdjpHAGovl4tnoQ+0QP6Bny6lUAyO GcWtmtQpeIIG6GRf9cKpe6ONtaB7s3NFOS/pcVSh72cu5UtWdS2dRW0RAtl9+poLMC/n 5ysA== X-Gm-Message-State: AOAM533f85ldRKhVuWpduNbvfLokb1zWt9vXr725HZ4Kq4iASt1h+Or7 a3bWjweAd1wYCLJXF5OOrRMryA== X-Google-Smtp-Source: ABdhPJxrik1zpDcx7wkwM7ODO9toNtzWr4VKncewI3Idid0KpMjorJgVg1Yumldacapk6mpiT/b5vw== X-Received: by 2002:a0c:d845:: with SMTP id i5mr12130743qvj.32.1626468612726; Fri, 16 Jul 2021 13:50:12 -0700 (PDT) Received: from ripple.attlocal.net (172-10-233-147.lightspeed.sntcca.sbcglobal.net. [172.10.233.147]) by smtp.gmail.com with ESMTPSA id c16sm3591087qtv.32.2021.07.16.13.50.11 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 16 Jul 2021 13:50:12 -0700 (PDT) Date: Fri, 16 Jul 2021 13:50:10 -0700 (PDT) From: Hugh Dickins X-X-Sender: hugh@ripple.anvils To: Zhansaya Bagdauletkyzy cc: akpm@linux-foundation.org, tyhicks@linux.microsoft.com, pasha.tatashin@soleen.com, willy@infradead.org, linux-mm@kvack.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v2] mm: KSM: fix data types In-Reply-To: <20210716055800.GA255851@asus> Message-ID: <181bd7fd-dcb6-3dea-305c-5764603d7d45@google.com> References: <20210716055800.GA255851@asus> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Authentication-Results: imf16.hostedemail.com; dkim=pass header.d=google.com header.s=20161025 header.b=OyZEvDSz; spf=pass (imf16.hostedemail.com: domain of hughd@google.com designates 209.85.219.50 as permitted sender) smtp.mailfrom=hughd@google.com; dmarc=pass (policy=reject) header.from=google.com X-Stat-Signature: qjsj8acx1b1pin8tiu7upmxdiqzncpbs X-Rspamd-Queue-Id: 90479F00008C X-Rspamd-Server: rspam01 X-HE-Tag: 1626468613-568980 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: On Fri, 16 Jul 2021, Zhansaya Bagdauletkyzy wrote: > Change data types of ksm_run and ksm_stable_node_chains_prune_millisecs > as there were a few discrepancies between the types and actual values > that were stored: > 1) ksm_run is declared as unsigned long but in run_store(), it is > converted to unsigned int. Change its type to unsigned int. NAK to that part. Compiling with CONFIG_MEMORY_HOTREMOVE=y would show why. Search for where ksm_run is used: in one place there's wait_on_bit(&ksm_run, ilog2(KSM_RUN_OFFLINE), TASK_UNINTERRUPTIBLE); and wait_on_bit() operates on unsigned long *word. Before that wait while offlining, yes, ksm_run used to be an unsigned int. But there's nothing to be fixed by changing it to unsigned int now. Save four bytes? Better look elsewhere if you want to save four bytes. > 2) ksm_stable_node_chains_prune_millisecs is declared as int, but in > stable__node_chains_prune_millisecs_store(), it can store values up to > UINT_MAX. Change its type to unsigned int. Ack to that part. > > Signed-off-by: Zhansaya Bagdauletkyzy > --- > v1 -> v2: > - merge two patches into one Sorry, that didn't work out so well. Hugh > > mm/ksm.c | 12 ++++++------ > 1 file changed, 6 insertions(+), 6 deletions(-) > > diff --git a/mm/ksm.c b/mm/ksm.c > index 3fa9bc8a67cf..2e4bd7662e52 100644 > --- a/mm/ksm.c > +++ b/mm/ksm.c > @@ -259,7 +259,7 @@ static unsigned long ksm_stable_node_chains; > static unsigned long ksm_stable_node_dups; > > /* Delay in pruning stale stable_node_dups in the stable_node_chains */ > -static int ksm_stable_node_chains_prune_millisecs = 2000; > +static unsigned int ksm_stable_node_chains_prune_millisecs = 2000; > > /* Maximum number of page slots sharing a stable node */ > static int ksm_max_page_sharing = 256; > @@ -289,7 +289,7 @@ static int ksm_nr_node_ids = 1; > #define KSM_RUN_MERGE 1 > #define KSM_RUN_UNMERGE 2 > #define KSM_RUN_OFFLINE 4 > -static unsigned long ksm_run = KSM_RUN_STOP; > +static unsigned int ksm_run = KSM_RUN_STOP; > static void wait_while_offlining(void); > > static DECLARE_WAIT_QUEUE_HEAD(ksm_thread_wait); > @@ -2874,7 +2874,7 @@ KSM_ATTR(pages_to_scan); > static ssize_t run_show(struct kobject *kobj, struct kobj_attribute *attr, > char *buf) > { > - return sysfs_emit(buf, "%lu\n", ksm_run); > + return sysfs_emit(buf, "%u\n", ksm_run); > } > > static ssize_t run_store(struct kobject *kobj, struct kobj_attribute *attr, > @@ -3105,11 +3105,11 @@ stable_node_chains_prune_millisecs_store(struct kobject *kobj, > struct kobj_attribute *attr, > const char *buf, size_t count) > { > - unsigned long msecs; > + unsigned int msecs; > int err; > > - err = kstrtoul(buf, 10, &msecs); > - if (err || msecs > UINT_MAX) > + err = kstrtouint(buf, 10, &msecs); > + if (err) > return -EINVAL; > > ksm_stable_node_chains_prune_millisecs = msecs; > -- > 2.25.1