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.5 required=3.0 tests=DKIM_INVALID,DKIM_SIGNED, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI,SPF_HELO_NONE, SPF_PASS,USER_AGENT_GIT 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 0F325C4332D for ; Fri, 20 Mar 2020 16:50:27 +0000 (UTC) Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by mail.kernel.org (Postfix) with ESMTP id C6B0420739 for ; Fri, 20 Mar 2020 16:50:26 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=fail reason="signature verification failed" (2048-bit key) header.d=ziepe.ca header.i=@ziepe.ca header.b="NaCjjzld" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org C6B0420739 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=ziepe.ca Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=owner-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix) id 4DD2E6B0008; Fri, 20 Mar 2020 12:50:26 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 4656D6B000C; Fri, 20 Mar 2020 12:50:26 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 2D56B6B000A; Fri, 20 Mar 2020 12:50:26 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from forelay.hostedemail.com (smtprelay0234.hostedemail.com [216.40.44.234]) by kanga.kvack.org (Postfix) with ESMTP id 0F8C16B0007 for ; Fri, 20 Mar 2020 12:50:26 -0400 (EDT) Received: from smtpin28.hostedemail.com (10.5.19.251.rfc1918.com [10.5.19.251]) by forelay01.hostedemail.com (Postfix) with ESMTP id 8FE02180AD81A for ; Fri, 20 Mar 2020 16:50:25 +0000 (UTC) X-FDA: 76616328810.28.wing57_3ab60f26f701 X-HE-Tag: wing57_3ab60f26f701 X-Filterd-Recvd-Size: 5707 Received: from mail-qk1-f194.google.com (mail-qk1-f194.google.com [209.85.222.194]) by imf41.hostedemail.com (Postfix) with ESMTP for ; Fri, 20 Mar 2020 16:50:25 +0000 (UTC) Received: by mail-qk1-f194.google.com with SMTP id q188so2227500qke.8 for ; Fri, 20 Mar 2020 09:50:25 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ziepe.ca; s=google; h=from:to:cc:subject:date:message-id:mime-version :content-transfer-encoding; bh=87iKgwaPy8pXzXL9YA5WV3WZml27jYwBmFGI1UGSVUk=; b=NaCjjzldyTssF/3z0KZL0Fqab/MMlCzRVjrru2siMv9BUoCtsVnQTd1gi0e4g7Bv7C 7lWk8bdKCdsAa8UzZNISAM9HmDetrX83wCNt9BSFlOheErwFqT8tMGKNvHewjQr45veO HWg6Vy16BlP0KGklKKcCiS4VNcLzsIKutYoiODMmJPm1d2altsZ0q+sdSgGp2+g62JZb 3CyCsoqW0ZnZ0HCHgfI/B0TdbvC598XLc03bFzjXoW1P352HHxZ0KJ8lx/pW9bAVegn3 mG+bqtTK0HVO+f8zqeTr/natEqIDzoAw+jhHbk3xIlfLbjCkv7KSx/3qYUaodlY+D0bf kGQw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:mime-version :content-transfer-encoding; bh=87iKgwaPy8pXzXL9YA5WV3WZml27jYwBmFGI1UGSVUk=; b=U3YS5DWZnSwvYputY92NBxzs6PrRETvQ5bO6drIPca75jUQ68c5GPPb6kosaVESt02 KyuRAfEe2aytdFl+ONUTRgfCYHVtkd+2zm0hyFovrv8NSCsH5dpYmDliAnU6FeKtEUOk JcZWPFXGgTXDb4EyzttvbJWJsaYJcyEfiDJJP0o2tKV2fAuJxNFdc4oWAGCXGsv2ITzv hlOtE56oSNItKrFkl1+xktOWopIztlsawY3d7rTjI2xX7YPyb2oLHEYFxDhAH3zaKCx2 qzcZcqrQ9PWdhPwSSdHdvxnFFAHBHXEtHyadUQ1GAPPke758PllUNnts62ouYwBEeoeP ZoMA== X-Gm-Message-State: ANhLgQ1O66Weefm72QxiYrb5KvRFcUaK4TgK5pL3avoSoLyX9g2xieNi SeTymT2VL8Geh0hVINIcoaPqQw== X-Google-Smtp-Source: ADFU+vvveuclm6zQpq1BtpQVw2AZCya6AVI9ugZ2/VnoudKnWPkwAxuYh/+4WEp0jmf9H9GqVc8R6Q== X-Received: by 2002:a37:8581:: with SMTP id h123mr8465387qkd.158.1584723024381; Fri, 20 Mar 2020 09:50:24 -0700 (PDT) Received: from ziepe.ca (hlfxns017vw-142-68-57-212.dhcp-dynamic.fibreop.ns.bellaliant.net. [142.68.57.212]) by smtp.gmail.com with ESMTPSA id x74sm4382043qkb.40.2020.03.20.09.50.23 (version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256); Fri, 20 Mar 2020 09:50:23 -0700 (PDT) Received: from jgg by mlx.ziepe.ca with local (Exim 4.90_1) (envelope-from ) id 1jFKql-0005jg-5N; Fri, 20 Mar 2020 13:50:23 -0300 From: Jason Gunthorpe To: Jerome Glisse , Ralph Campbell , Felix.Kuehling@amd.com Cc: linux-mm@kvack.org, John Hubbard , dri-devel@lists.freedesktop.org, amd-gfx@lists.freedesktop.org, Christoph Hellwig , Philip Yang , Jason Gunthorpe Subject: [PATCH hmm 0/6] Small hmm_range_fault() cleanups Date: Fri, 20 Mar 2020 13:48:59 -0300 Message-Id: <20200320164905.21722-1-jgg@ziepe.ca> X-Mailer: git-send-email 2.25.1 MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable 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: From: Jason Gunthorpe I've had these in my work queue for a bit, nothing profound here, just so= me small edits for clarity. Ralph's hmm tester will need a small diff to work after this - which illustrates how setting default_flags =3D=3D 0 is the same as what was ca= lled SNAPSHOT: diff --git a/lib/test_hmm.c b/lib/test_hmm.c index 6ca953926dc13f..5f31f5b3e64cb9 100644 --- a/lib/test_hmm.c +++ b/lib/test_hmm.c @@ -300,7 +300,7 @@ static int dmirror_range_fault(struct dmirror *dmirro= r, =20 range->notifier_seq =3D mmu_interval_read_begin(range->notifier); down_read(&mm->mmap_sem); - count =3D hmm_range_fault(range, 0); + count =3D hmm_range_fault(range); up_read(&mm->mmap_sem); if (count <=3D 0) { if (count =3D=3D 0 || count =3D=3D -EBUSY) @@ -337,8 +337,7 @@ static int dmirror_fault(struct dmirror *dmirror, uns= igned long start, .flags =3D dmirror_hmm_flags, .values =3D dmirror_hmm_values, .pfn_shift =3D DPT_SHIFT, - .pfn_flags_mask =3D ~(dmirror_hmm_flags[HMM_PFN_VALID] | - dmirror_hmm_flags[HMM_PFN_WRITE]), + .pfn_flags_mask =3D 0, .default_flags =3D dmirror_hmm_flags[HMM_PFN_VALID] | (write ? dmirror_hmm_flags[HMM_PFN_WRITE] : 0), .dev_private_owner =3D dmirror->mdevice, @@ -872,7 +871,7 @@ static int dmirror_range_snapshot(struct dmirror *dmi= rror, range->notifier_seq =3D mmu_interval_read_begin(range->notifier); =20 down_read(&mm->mmap_sem); - count =3D hmm_range_fault(range, HMM_FAULT_SNAPSHOT); + count =3D hmm_range_fault(range); up_read(&mm->mmap_sem); if (count <=3D 0) { if (count =3D=3D 0 || count =3D=3D -EBUSY) @@ -916,7 +915,7 @@ static int dmirror_snapshot(struct dmirror *dmirror, .flags =3D dmirror_hmm_flags, .values =3D dmirror_hmm_values, .pfn_shift =3D DPT_SHIFT, - .pfn_flags_mask =3D ~0ULL, + .pfn_flags_mask =3D 0, .dev_private_owner =3D dmirror->mdevice, }; int ret =3D 0; Jason Gunthorpe (6): mm/hmm: remove pgmap checking for devmap pages mm/hmm: return the fault type from hmm_pte_need_fault() mm/hmm: remove unused code and tidy comments mm/hmm: remove HMM_FAULT_SNAPSHOT mm/hmm: remove the CONFIG_TRANSPARENT_HUGEPAGE #ifdef mm/hmm: use device_private_entry_to_pfn() Documentation/vm/hmm.rst | 12 +- drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c | 2 +- drivers/gpu/drm/nouveau/nouveau_svm.c | 2 +- include/linux/hmm.h | 55 +----- mm/hmm.c | 238 +++++++++--------------- 5 files changed, 98 insertions(+), 211 deletions(-) --=20 2.25.1