Message-ID: <c0635ff3-027f-bcd7-afbc-46f4e62d3651@amd.com>
Date:   Mon, 6 Feb 2023 17:14:54 +0100
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101
 Thunderbird/102.4.2
Subject: Re: [Nouveau] [PATCH drm-next 05/14] drm/nouveau: new VM_BIND uapi
 interfaces
Content-Language: en-US
To:     Danilo Krummrich <dakr@redhat.com>, Dave Airlie <airlied@gmail.com>
Cc:     Matthew Brost <matthew.brost@intel.com>, daniel@ffwll.ch,
        corbet@lwn.net, dri-devel@lists.freedesktop.org,
        linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org,
        mripard@kernel.org, bskeggs@redhat.com, jason@jlekstrand.net,
        nouveau@lists.freedesktop.org, airlied@redhat.com
References: <20230118061256.2689-1-dakr@redhat.com>
 <20230118061256.2689-6-dakr@redhat.com>
 <Y9MjSeMcsd18r9vM@DUT025-TGLU.fm.intel.com>
 <7c046ff9-728d-7634-9d77-8536308c7481@redhat.com>
 <c2256c7d-e768-ae3f-d465-b9f8080d111b@amd.com>
 <2427a918-5348-d1ef-ccae-a29c1ff33c83@redhat.com>
 <a214b28b-043c-a8bb-69da-b4d8216fce56@amd.com>
 <3a76bfa9-8ee5-a7d9-b9fb-a98181baec0b@redhat.com>
 <49ac3f95-6eda-9009-4b28-0167213301b2@amd.com>
 <bc523c5c-efe6-1a7f-b49a-e0867dc1413d@redhat.com>
 <15fb0179-c7c5-8a64-ed08-841189919f5e@redhat.com>
 <1840e9fb-fd1b-79b7-4238-54ae97333d0b@amd.com>
 <CAPM=9txON8VCb3H7vDY_DOgtUg2Ad3mBvYVxgSMyZ1noOu-rBQ@mail.gmail.com>
 <a1c526e0-0df7-12cb-c5a1-06e9cd0d876b@amd.com>
 <3f935a7e-fede-2bad-c029-4a3af850c9b5@redhat.com>
 <95d0631b-545c-ea4d-7439-75422e9a9120@amd.com>
 <67958920-c5bb-a0f5-2306-e3ae4fdbaeb3@redhat.com>
From:   =?UTF-8?Q?Christian_K=c3=b6nig?= <christian.koenig@amd.com>
In-Reply-To: <67958920-c5bb-a0f5-2306-e3ae4fdbaeb3@redhat.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?RFg0YzhwVSsvelErNXcwcmJ3L2xiTVZxeGlaTDN2a1huNHBwa0NHeXk2NWxk?=
 =?utf-8?B?bGIzaTNLb3BTaFpxeFBtTzFlbUpFMWRDZFJIbWtsTHF4bU1kSnhoSmxGK2NI?=
 =?utf-8?B?NnkreHFuVGdoeWQxUEhtaStiV1UrTmlQcndJOHU5SHNBb0t1TzZLQ0UxaC8z?=
 =?utf-8?B?bDU1NlpMTS9TakJlWWE4TCtFaXQ1a1NqZkdRWFd4R21SQ1NYdTFEc3UwR082?=
 =?utf-8?B?cU9seExYSHo1cE53RlFSa3k1L2NQRUJYdU9VcDNUKzB2SStwNDBQSWJKd0lP?=
 =?utf-8?B?Q09TemF0d3ZuSjZvYTNoRFJMZ0JGSHJrOVpPNEkvNk1YTDYwSi9UY3E5TXF3?=
 =?utf-8?B?T3ZuTWt0eDRIUWZLVmJaTEZsY0s0OGF1U1ZKbStybGdlR2tvZFREZkpKL3Rt?=
 =?utf-8?B?WGFweHAvTVM4MFJtUkYrMm0yU0pCNUVCRTF3UnhkLzJERlZvTFc2UDhBNCtZ?=
 =?utf-8?B?S2xrVEhPSHhaRHA0TVRQenhzbFdoS2JpTDhIakFBdUx4dzJra3dLYkJ5TGFL?=
 =?utf-8?B?Q0lPRHlzNHR5SW1ZN1RHWHFuSWlWc3BSWkpMQkJaR05SZnprVnVRakV1TXpD?=
 =?utf-8?B?VjN1YS9DdmpRaWh4TFUzeHdCOHA1cE5MODRFWTJqbXFCZmxFdksvNXZtSkRy?=
 =?utf-8?B?eEsvby9iaWVQVU9DSUJaeVZDV2pSRlg5RTRpZ3c0M0RGQ3BEaCsrQzgwcks1?=
 =?utf-8?B?bFhUNkhGekl0SUtSaFZMK0xXc3ZBLzkyMi9BbnQraHV4V2RZNG5OMjAyRDY3?=
 =?utf-8?B?amxiQS8rbTVYblYxcDBQQk1ucHJlcnIrYXUvYmM4V2w2eWhOVXZrcllrckFk?=
 =?utf-8?B?dW9aM08vRzdMSVo5K0loYnhxb2E3VXJhWHdDMGRML01MSmJaaHdPYXJuS0Ey?=
 =?utf-8?B?RnhreDhwdUV4cEZXM3NLU2J4dHhMS0l0VHdHREpONmdrMEx4ckRxeFl1c2Rl?=
 =?utf-8?B?T0x0NU1mNnVYZHpCdUZtbTFEbmhaam81dUh6OEMvdk1Yd1lPYWx6YS9JMXhk?=
 =?utf-8?B?NEVTY01tM1lIajNpWm9MOHlQWG9kRmJxSDcwMFNtZloyOGhkL2diOS9FTEc0?=
 =?utf-8?B?NHJ3eldBMU56eGE2Ujc0NzJsZGtLQkNQK3ArdS9HMzV3WHBFYngrcW1mRTZj?=
 =?utf-8?B?amJCMXRzSFhIYVAvVnZqdTB5ajJjZTVuY3JNVjY5T3kzU2NXRkp0THExamFx?=
 =?utf-8?B?eERuQWVrTjlBRWFmdWV1bnBJd0ZGbThCdGVKSTFqcnUrc1JoNUo4QVp5cmZ1?=
 =?utf-8?B?QkpINmdmRnVpNzd1eU01UlI0WlFnS0VHOUo0OStVVm9aQ3VRdUYzUlRQMU01?=
 =?utf-8?B?RXQzNFNyRTY1OW5kUjYvazhkUDVqTmJPaXNuRnNGTkRkZ0ZGbm5zUWFZeGNk?=
 =?utf-8?B?K3lkOHRiOU5zcjNBakVYUFVOMHpQWkJzTkx3OWVWMVlMRzhpRDdDNStqN0gr?=
 =?utf-8?B?TURSV0JWQTJ1MGxPdGdwMlgxWWM2bjFYUHYrWjFrZGpKZmIxR21xN3VGL3pl?=
 =?utf-8?B?QWp0RmdLVXdQQkxYN0JUSFovS2tBVXkydHVTcFpCejNLdGsvSHM1WDZvZEdi?=
 =?utf-8?B?bms4OGIxRWRxdnFyYmwxUGpVYVVDWWt3dnB0aEJ5UFp1UTdPaS9vTzFQa3BT?=
 =?utf-8?B?M1AwbGJ1UU9mQ1JnTmRWWjJ4SWdWNjU4blF1cDdYdms0RXY2WnFYYzBVczFE?=
 =?utf-8?B?dnYvZFhKQmFFUWlETzNCb3FjZng3Nk1vbmRGbmlVb1Vka3Z4Y1FyL213YUJi?=
 =?utf-8?B?ZnZDVzd3akZFbmU5WEtpRGRJcldhdGxPZTl5Q1JKTytPZUY2T0VpZ1dBdE9W?=
 =?utf-8?B?SWN3a1doSkp4TUZpY0lQNGhpTVdzcEh2alYxMTFFWEJQQ0FPSFByUVFCMXl1?=
 =?utf-8?B?dkRjZ0RoaFdLNXRha3M4Y1ZWemZPejB6eEtjWGxjbWJVLzJFYVM3WHBjdGd1?=
 =?utf-8?B?M2NvcHRMdnV2b2tiMm9abi9GY3ZQSm5tZzVGZzFIY1l2RHY4TkpCbnR5dWcx?=
 =?utf-8?B?U01WTmVkeERoNSsyVitoOHRSSytaN2M0WXgvbEM4Y2JJdzNhYXQvT25mS1pB?=
 =?utf-8?B?T0hyTndnSXFnU09nYmtJVk8vM2kreUlmZVh1dHJ3ajR5cEQ1dlBVbFRRaGkv?=
 =?utf-8?B?NUJORmVsNXVIVGp2TFJWMnFwU3JyQVBrN2NSTnB5dncrMXdFbk1XRDEwdnFH?=
 =?utf-8?Q?bsYD+PPXmVE8TSZHBSDzA0lPKG3U9hn117cjOQweEgtt?=
X-OriginatorOrg: amd.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 545af347-bbb0-4c61-8112-08db085d4c8a
X-MS-Exchange-CrossTenant-AuthSource: BN8PR12MB3587.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 06 Feb 2023 16:15:01.5464
 (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: ny9pO6KBM6hIhN8Oizxqr9RFBcVpgfCL6PbOuG/tYj2OUdvU0Uygdf10N4UZaRzZ
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SJ0PR12MB5408
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Concentrating this discussion on a very big misunderstanding first.

Am 06.02.23 um 14:27 schrieb Danilo Krummrich:
> [SNIP]
> My understanding is that userspace is fully responsible on the parts 
> of the GPU VA space it owns. This means that userspace needs to take 
> care to *not* ask the kernel to modify mappings that are in use currently.

This is a completely wrong assumption! Take a look at what games like 
Forza Horizzon are doing.

Basically that game allocates a very big sparse area and fills it with 
pages from BOs while shaders are accessing it. And yes, as far as I know 
this is completely valid behavior.

So you need to be able to handle this case anyway and the approach with 
the regions won't help you at all preventing that.

Regards,
Christian.