Received-SPF: pass (google.com: domain of linux-ext4-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
From:   Alex Sierra <alex.sierra@amd.com>
To:     akpm@linux-foundation.org, Felix.Kuehling@amd.com,
        linux-mm@kvack.org, rcampbell@nvidia.com,
        linux-ext4@vger.kernel.org, linux-xfs@vger.kernel.org
Cc:     amd-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org,
        hch@lst.de, jgg@nvidia.com, jglisse@redhat.com
Subject: [PATCH v2 07/12] drm/amdkfd: public type as sys mem on migration to ram
Date:   Mon, 13 Sep 2021 11:15:59 -0500
Message-Id: <20210913161604.31981-8-alex.sierra@amd.com>
In-Reply-To: <20210913161604.31981-1-alex.sierra@amd.com>
References: <20210913161604.31981-1-alex.sierra@amd.com>
Content-Transfer-Encoding: 8bit
Content-Type: text/plain
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?DHPQ05IZ2gkNLpdT4htMZ7TCRhYuwQUprk9eNKIsUel3LdsERHuES6sDOcHr?=
 =?us-ascii?Q?lMqe1QT4vey3DnU+dvL+37+bnmjfKc1AJ5y/AXyVhpLE7Mp04TpOfK3kX2La?=
 =?us-ascii?Q?8hzrVC3ynmffmrFy5++YbDVH3RbCSF679hhaBM+lMTvD/OpKOiEDsU98X0mu?=
 =?us-ascii?Q?deBjVdF2cYRMbvAGvVraqvJuwW5imPWjODhs9hsqVDShvNxND0VRhclCO/lc?=
 =?us-ascii?Q?OEl2l12OYhjQck0qLc4YfK/MdZwL0XmBWyZnw/FrqGfm3TykUmY2yHdeGAeo?=
 =?us-ascii?Q?vfviA4KKoDsvyT1TuAcpTMTkUb0uqPhMfhT6zaPMb5G9Ocp3jIW/BLhF3twU?=
 =?us-ascii?Q?WSWH4qicg1wzA5DXyOx9IIjP+KAOMrETy8P2vL/gGi0Ppnn1X9Nbp95e6DrF?=
 =?us-ascii?Q?spmBgn9Q+EFUhtz+Mt4ZoKMsTFHrgvU6MLtQog/ebLqlniHY/oZHClc/oLqt?=
 =?us-ascii?Q?KOdZlmN5cd3x/Xtv2Ny5sapiLQFZ0O6k6AY5hRgQToY+YmzsQJlzlgQ9n7ZY?=
 =?us-ascii?Q?RmrU3svEKRuHJqmKoAHAi1NWK2SaygaObk8iDZFnDl1OX5074snbsyk8o0RN?=
 =?us-ascii?Q?5diz5qXFUmhO3AqQWRpxnuablt8wh44K+A3gKmbRRdzBz1187mC8K+Y7/id8?=
 =?us-ascii?Q?rkG6HY/tuJMXkaz/lakBO+3TUAcnjtzjFvQUeJotTpoQPuup8Ek2Mhm/YYnU?=
 =?us-ascii?Q?l6siZrOsYS4V6XNuh13th/cyjWOZB3xor0CcoEzKSFie7pxSjt9L9BGV7/+g?=
 =?us-ascii?Q?m6kYeYZzzkGNzGpzKQJ6Y6Kb/MWe7Akcxs4FbwjcCGLvTid9EYaV4/5Cb1xn?=
 =?us-ascii?Q?Q6m7X8a6LGx51shpIwKGaM8ET6zChXw110MBr8KU/C7imF0T6XBkKijoVGJ1?=
 =?us-ascii?Q?3lxZuwj32QeRrmuiKGZNSeecJ5LtRnmwrL22K2CdH3hZDPwfh7rrPossgTQK?=
 =?us-ascii?Q?/aWJkKxhkuXIyum1PDBxjEWtfqY8EOFq46p0L0skDZILyli2z4VHvGBPSRr8?=
 =?us-ascii?Q?MdC246c1UhWEnLfVeU/KPJ1wrJz2VtLfRUSvOPhdwTfk2mQ5BegM7OuatSVe?=
 =?us-ascii?Q?G2D5n7XvX6sB31wK9ZMWm4Zcu5RWqyIgMHuvt1Ux/0k7Ukw5XxCdCgCltgm8?=
 =?us-ascii?Q?f6/cf4ilMGbuj+T0uyShP0uuLXOP+ozITCcpbZ4LMo24mW/B84IuRKoYPZDq?=
 =?us-ascii?Q?C49Mb8LbYsPI2MJvHsaUGCafo5AilIL6yfX9kWfayjBPqgaNh7AsUNe+IUsN?=
 =?us-ascii?Q?PKcemUjdqrJDs/jVX3LbSZidRUezBzgLwLovkSeKDjM72s7jRxzcv7ZQHM/B?=
 =?us-ascii?Q?2N1T4+9N/jXkBnYTjCt/pu1T?=
X-OriginatorOrg: amd.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 3d6b7dc9-51e7-423a-1531-08d976d1d395
X-MS-Exchange-CrossTenant-AuthSource: SA0PR12MB4430.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 13 Sep 2021 16:16:22.1791
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: xQxJM1uG0Bsm8AAb1uTeJSafFqAk5psuJpaUpKi3gH3tF6hBKnR981eLfCIQs8zCtVhkKSEg1T7oNTaZvM5V3Q==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SA0PR12MB4349
Precedence: bulk
List-ID: <linux-ext4.vger.kernel.org>
X-Mailing-List: linux-ext4@vger.kernel.org

Public device type memory on VRAM to RAM migration, has similar access
as System RAM from the CPU. This flag sets the source from the sender.
Which in Public type case, should be set as
MIGRATE_VMA_SELECT_DEVICE_PUBLIC.

Signed-off-by: Alex Sierra <alex.sierra@amd.com>
Reviewed-by: Felix Kuehling <Felix.Kuehling@amd.com>
---
 drivers/gpu/drm/amd/amdkfd/kfd_migrate.c | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_migrate.c b/drivers/gpu/drm/amd/amdkfd/kfd_migrate.c
index d0e04f79a06e..b5b9ae4e2e27 100644
--- a/drivers/gpu/drm/amd/amdkfd/kfd_migrate.c
+++ b/drivers/gpu/drm/amd/amdkfd/kfd_migrate.c
@@ -617,9 +617,12 @@ svm_migrate_vma_to_ram(struct amdgpu_device *adev, struct svm_range *prange,
 	migrate.vma = vma;
 	migrate.start = start;
 	migrate.end = end;
-	migrate.flags = MIGRATE_VMA_SELECT_DEVICE_PRIVATE;
 	migrate.pgmap_owner = SVM_ADEV_PGMAP_OWNER(adev);
 
+	if (adev->gmc.xgmi.connected_to_cpu)
+		migrate.flags = MIGRATE_VMA_SELECT_DEVICE_PUBLIC;
+	else
+		migrate.flags = MIGRATE_VMA_SELECT_DEVICE_PRIVATE;
 	size = 2 * sizeof(*migrate.src) + sizeof(uint64_t) + sizeof(dma_addr_t);
 	size *= npages;
 	buf = kvmalloc(size, GFP_KERNEL | __GFP_ZERO);
-- 
2.32.0