Received: by 2002:a25:1506:0:0:0:0:0 with SMTP id 6csp501595ybv; Thu, 20 Feb 2020 02:24:32 -0800 (PST) X-Google-Smtp-Source: APXvYqwP3dP8zfikDym7DoWlncnaclxSmMEicPo7uOTz5d18Ykc3fn0mGuE3tnEqrg/LKHih023E X-Received: by 2002:a9d:1706:: with SMTP id i6mr24111233ota.151.1582194272044; Thu, 20 Feb 2020 02:24:32 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1582194272; cv=none; d=google.com; s=arc-20160816; b=t5iLLHWBg+fqnvOWbgjD5rRKxJOTTWGjweMSYMxKE5ZojVmGLNCUQlnGLv826znFXn TkRe8GxO+7bFmFC30K1yuSElJdGG5C94gMsj8KODq8dG+Php0Rp0NVNWzdkf/msngR94 R6LJG6XF4KfdNN4B4fyFYhwOjwcKVEe405zxBwu60pnG+pmw8MiG7cr66gTJCMXeNLu+ 9Gwt1HJNiw97xx5dxtmLZASGqw/Io6I72mubFy/H226Rb+ltGqdJHSbD6EqHexfd0MOt 6O9bI/ZUnATDFP9I0Z9M954LTmEnaqxqhga53kikCxPKS8rSBh+6WqQyR1IL8wQnN4EP qJiQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:to:references:message-id :content-transfer-encoding:cc:date:in-reply-to:from:subject :mime-version:dkim-signature; bh=5UKDu7l3UNxspcPIx9HSKdFAH6sgOkJpPpiv9ECBSfg=; b=QgEpX0wuoyDi73yA9StfMeWQEVznb0cC38ix8Aceb0D+wJCeq/hSESoV1pkTh0o8Vo gmdPYR5PiP1RPghe49IDulEPrmzdluCzlLa04XIdzZSZb0Q4PdQvNOdr1dG5ZXcaxR4G 8oXnszDCfpEfb/j2Uxp6auH8h8D1qcQNkgImOWx3BhDj+N79/nLM+qLuBttzyW92bQ+Y 8dm+N0ruLRcHIYZnLBpKIw8q2c2gbxG7q1+OyyX6KbOccOwed2RID2HmMVkpZ2oKy42G ErV3A4kabmlbvc+hSJIWw05HbPcobofsoAuPKKvY439twXVtXCyhRT9dkJRCHFNum5nS a6Mg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@oracle.com header.s=corp-2020-01-29 header.b=sI2eZY5r; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=oracle.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id y6si11321871oih.217.2020.02.20.02.24.19; Thu, 20 Feb 2020 02:24:32 -0800 (PST) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; dkim=pass header.i=@oracle.com header.s=corp-2020-01-29 header.b=sI2eZY5r; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=oracle.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727326AbgBTKWs (ORCPT + 99 others); Thu, 20 Feb 2020 05:22:48 -0500 Received: from userp2130.oracle.com ([156.151.31.86]:47146 "EHLO userp2130.oracle.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726825AbgBTKWr (ORCPT ); Thu, 20 Feb 2020 05:22:47 -0500 Received: from pps.filterd (userp2130.oracle.com [127.0.0.1]) by userp2130.oracle.com (8.16.0.42/8.16.0.42) with SMTP id 01KAJ0Ye095952; Thu, 20 Feb 2020 10:22:36 GMT DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=oracle.com; h=content-type : mime-version : subject : from : in-reply-to : date : cc : content-transfer-encoding : message-id : references : to; s=corp-2020-01-29; bh=5UKDu7l3UNxspcPIx9HSKdFAH6sgOkJpPpiv9ECBSfg=; b=sI2eZY5rGOBRoS15GFUXiIhyaDvJm40S+ZOt8fdaWYrChDMJ8+nTePMoUpi1e7oZ/kKT ttbnBuBc2Mon7XwE1p03VlOOLvBbuoqiYueM23vfdwnBJ4jFbIxZemcv0sCqXZpntDJi 9CISvBHII29w+asX5+JRzkXdVbYh/2lrt/LlYl/BvBNrnxCURlPIz84LTsrseO3iRLcT D8nBhOjucPao8h2H5Jx2C+e+XaCTIhXIM5GElbYI2gmR/lZvpfR6wWZL2IPlr+JnYuNA oUCjhHWaGLKaYcMd2ZdTvl4oJc6Nfz5a/KHckF9atFE7Ap9omGPzPjjNyhLziS5YwJHE /w== Received: from userp3020.oracle.com (userp3020.oracle.com [156.151.31.79]) by userp2130.oracle.com with ESMTP id 2y8udd8tjx-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK); Thu, 20 Feb 2020 10:22:36 +0000 Received: from pps.filterd (userp3020.oracle.com [127.0.0.1]) by userp3020.oracle.com (8.16.0.42/8.16.0.42) with SMTP id 01KAK8kS014174; Thu, 20 Feb 2020 10:22:36 GMT Received: from userv0121.oracle.com (userv0121.oracle.com [156.151.31.72]) by userp3020.oracle.com with ESMTP id 2y8ud3dr5p-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK); Thu, 20 Feb 2020 10:22:36 +0000 Received: from abhmp0007.oracle.com (abhmp0007.oracle.com [141.146.116.13]) by userv0121.oracle.com (8.14.4/8.13.8) with ESMTP id 01KAMXqd025211; Thu, 20 Feb 2020 10:22:33 GMT Received: from [192.168.0.110] (/73.243.10.6) by default (Oracle Beehive Gateway v4.0) with ESMTP ; Thu, 20 Feb 2020 02:22:33 -0800 Content-Type: text/plain; charset=us-ascii Mime-Version: 1.0 (Mac OS X Mail 13.4 \(3608.80.19\)) Subject: Re: [PATCH] mm: Fix possible PMD dirty bit lost in set_pmd_migration_entry() From: William Kucharski In-Reply-To: <20200220075220.2327056-1-ying.huang@intel.com> Date: Thu, 20 Feb 2020 03:22:29 -0700 Cc: Andrew Morton , linux-mm@kvack.org, linux-kernel@vger.kernel.org, Zi Yan , "Kirill A . Shutemov" , Andrea Arcangeli , Michal Hocko , Vlastimil Babka Content-Transfer-Encoding: quoted-printable Message-Id: References: <20200220075220.2327056-1-ying.huang@intel.com> To: "Huang, Ying" X-Mailer: Apple Mail (2.3608.80.19) X-Proofpoint-Virus-Version: vendor=nai engine=6000 definitions=9536 signatures=668685 X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 bulkscore=0 malwarescore=0 mlxlogscore=999 suspectscore=0 adultscore=0 spamscore=0 mlxscore=0 phishscore=0 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2001150001 definitions=main-2002200076 X-Proofpoint-Virus-Version: vendor=nai engine=6000 definitions=9536 signatures=668685 X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 phishscore=0 impostorscore=0 mlxlogscore=999 malwarescore=0 mlxscore=0 suspectscore=0 priorityscore=1501 bulkscore=0 adultscore=0 spamscore=0 lowpriorityscore=0 clxscore=1011 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2001150001 definitions=main-2002200076 Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org > On Feb 20, 2020, at 12:52 AM, Huang, Ying = wrote: >=20 > Signed-off-by: "Huang, Ying" Looks good to me. Reviewed-by: William Kucharski > Cc: Zi Yan > Cc: Kirill A. Shutemov > Cc: Andrea Arcangeli > Cc: Michal Hocko > Cc: Vlastimil Babka > --- > mm/huge_memory.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) >=20 > diff --git a/mm/huge_memory.c b/mm/huge_memory.c > index 580098e115bd..b1e069e68189 100644 > --- a/mm/huge_memory.c > +++ b/mm/huge_memory.c > @@ -3060,8 +3060,7 @@ void set_pmd_migration_entry(struct = page_vma_mapped_walk *pvmw, > return; >=20 > flush_cache_range(vma, address, address + HPAGE_PMD_SIZE); > - pmdval =3D *pvmw->pmd; > - pmdp_invalidate(vma, address, pvmw->pmd); > + pmdval =3D pmdp_invalidate(vma, address, pvmw->pmd); > if (pmd_dirty(pmdval)) > set_page_dirty(page); > entry =3D make_migration_entry(page, pmd_write(pmdval)); > --=20 > 2.25.0 >=20 >=20