Received-SPF: pass (google.com: domain of linux-kernel+bounces-126660-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) client-ip=2604:1380:4601:e00::3;
Message-ID: <1e94363a-b449-4efb-b2fe-c1dd710b57c9@amd.com>
Date: Mon, 1 Apr 2024 14:45:40 +0200
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH] dma-buf: Do not build debugfs related code when
 !CONFIG_DEBUG_FS
To: Tvrtko Ursulin <tursulin@ursulin.net>, "T.J. Mercier"
 <tjmercier@google.com>, Tvrtko Ursulin <tursulin@igalia.com>
Cc: dri-devel@lists.freedesktop.org, Sumit Semwal <sumit.semwal@linaro.org>,
 linux-media@vger.kernel.org, linaro-mm-sig@lists.linaro.org,
 linux-kernel@vger.kernel.org, kernel-dev@igalia.com
References: <20240328145323.68872-1-tursulin@igalia.com>
 <CABdmKX3V3HGA4mNQvqHqhcLqyr-A5kJK8v9vmuDybRvV-KsiOg@mail.gmail.com>
 <9a063c39-6d2f-43c3-98b3-e4f8c3c6e9c4@ursulin.net>
Content-Language: en-US
From: =?UTF-8?Q?Christian_K=C3=B6nig?= <christian.koenig@amd.com>
In-Reply-To: <9a063c39-6d2f-43c3-98b3-e4f8c3c6e9c4@ursulin.net>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 8bit
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0:
	=?utf-8?B?MzFPMnMvM1YyM0ZPdHlsc0J4TE5IRDY1R3Q2K1c2b0RvT0loOGU1RUJmRzZa?=
 =?utf-8?B?bVdpSUdyUzBFSlZ0KzE4YndFb0pUcW5xZld1Rkt2aXZZUjV0eFZ1bDRiVENO?=
 =?utf-8?B?V3NFNExoSjFRdXpWVlpicmg5alpLNTRXWmE5clJva1lDRVpDdlliMmVHM0tN?=
 =?utf-8?B?dlNDRm0wZVBTek5XaXViUmU3M3FIYWpVN3haSTJKVSsyTU1VWUtocENqd0FK?=
 =?utf-8?B?U2pJbjhXMTZZYmhhandQQ2J4Y0loY0dqOEQrWk1admlSY2tQMkcxMUJ0ZDg3?=
 =?utf-8?B?aXFHWHRrU0J4WUwwWkd2dkgrU3BDZW5ra1dpeWNJUCt5eXVyZ2MzbnJRbU1v?=
 =?utf-8?B?d1lIaHcvL2ZHMC9LdHhzNkUwK2FKdWZucmw4MklrRVl1aTV0QzRMOFN6elZ5?=
 =?utf-8?B?eTFDVE9YZkVKM0V0M0F0bHk1UW1TaDN3UFBxTngzVVgxNGlzNGRnbkpkM0pG?=
 =?utf-8?B?WklaeXdnNEdjZ0hjWGgydUt2STJVdnRZT3hpaDFvSEN6SE5veTJYOFhSckQw?=
 =?utf-8?B?TWhHZUpTWGZJUTVTR1Jtd2h4RGpTZWZ4WUI4S3Z6UkVsaXFDbWpYQmJtMFRE?=
 =?utf-8?B?SmQvYWhvVTE3RmNyWHMrWDR4eEJ6eTNMZmNjN1htRWFEcDFsZHNwN3BJTmth?=
 =?utf-8?B?UWxzVFVpSU5oS1lUTjFwenFBUVNseFMvRUw2ZnM0cnRUZXQzQ213MkxMc3J4?=
 =?utf-8?B?dEMramRSMGR0ZXdXUVhGeWhEM3hHckQ4ejJNVVU2RDd2MXZOTE9IeUNhKzIz?=
 =?utf-8?B?clA0NlczaVVOZ0pEWjFmNHcvTTZOVG0xQWM1dUg3OS90d25XZWdGMCtsSDVE?=
 =?utf-8?B?c0FiUlhiUUN6c2NPSlJiZG80cy90YUkvN00wekJEMWZQd3g2YlF4cjNyUXlC?=
 =?utf-8?B?YndZSjcveFA3WFF1YTVSSy9Db1NGM1o2d1I2c3gzeTRzUE9reWp5VElHSXds?=
 =?utf-8?B?NlVjOXRjMWpmUkhzeHBYYTZkb281MUl0K0c4OEpjUysxU1hUSGRxRlpBT0Ev?=
 =?utf-8?B?TGVxRHF3aTV1RFJNQmlDOFdpUlRIcGxvam9va3ZBbnBDL1o3TXpwSURHNExF?=
 =?utf-8?B?NnZKaWlSRzlEZjdqZWZadlJTSys1d3hyYVpwN25TQitOTDlkTm8rVExIWUJl?=
 =?utf-8?B?bWJpaFdXZS9qVUlEZnlKckN3MURmYUZkaXk3RmdvY0Z6TFQ0NU5vTytxYkxL?=
 =?utf-8?B?UkVRc2p1L3Z2OFVwUVRPT2EzV3lib0dkd3U1VDZZeklaMDBDaERNQlB4S2RS?=
 =?utf-8?B?YWlZVmJlR1dMd1ZtU2RzWTBJcjRtT0d6aUxuY3k2THNsNVpTbWltZXJvcGVV?=
 =?utf-8?B?RmVpcmxYTXVxSUVSdFpBUkR3dXRMNWFWTE8xQ3N3Uk55WmFiUGdJZ1l0OU5l?=
 =?utf-8?B?MzF0Q1UyVWErSE1YNFRVSzgxTmhtUy83bGtwaFV0YlRRMnpPbElYQTBROTM3?=
 =?utf-8?B?WW1WRWwyTWJJaS9tMGFBMWtZdUVISTM0YWdFRGtaU3RTVy9YdmN6UG9CU0Qx?=
 =?utf-8?B?VkpkR3BXTFhSUTI2MEFsaWJPZkt2ZWlmbWREUUVLcHJWNkRUcUZ5dzFIek8w?=
 =?utf-8?B?T1FYbTRZcHpubjlZYTFYSUhXYjFSMU8rdnd2WVpHRVlRWWN6OVlGMk9DVW5F?=
 =?utf-8?B?bUlTSm50WWhPNUNjZ0hXTU51UG05dWZ6bHEzeExyWDZlNDZuVGVReVFaZ0ZH?=
 =?utf-8?B?N0srZUxxNWxiSW4xWHNuWnh6Wlk5cC9VZUpMUm03QjNKNUlFTmZBZmtOcTVq?=
 =?utf-8?B?UUFFQjQyZlNab0I2bEl3OC9Vckp3dVZBZkpXUW84MVZYaGxFd3cyU3d2L2Fx?=
 =?utf-8?B?Q3NmblJyc3JYcmFsN1ZNUHo1YTFHdFVBRkgrbFlELy95YzcwRTRsWVZKc2lz?=
 =?utf-8?B?ZC9kK2ErUEJJVnVSVmovZ2ZPSkxLcU5XV2NTKzYxdUszcExnQ2hWUHRDZXRo?=
 =?utf-8?B?aTI4UlF4ell1MVRjTWJ4VTRQbExkQTlTdUpIOHQxYXNjWW1meGxCaW44a0ZE?=
 =?utf-8?B?WDFwSVoyS1A1dTl0c0gwRW4zZWRKSW5zYnprSUFhYmlhMzluRzJqTnNkODZY?=
 =?utf-8?B?akJHbnFRSzRQbHg1V3RGOFl0Z0MrRmhGWVpZdStuUkxxZFMyckNPTnVqQXVD?=
 =?utf-8?Q?a94k=3D?=
X-OriginatorOrg: amd.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 1e27f7a9-aa2a-444c-0f9b-08dc5249a649
X-MS-Exchange-CrossTenant-AuthSource: PH7PR12MB5685.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 01 Apr 2024 12:45:45.8653
 (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: mX9oYvLIPTYYs9XTFyowP7bzZnjofNQ4IQJ6ReiM8EEIs2/yexnEDITi2Q60Zc+o
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM3PR12MB9349

Am 01.04.24 um 14:39 schrieb Tvrtko Ursulin:
>
> On 29/03/2024 00:00, T.J. Mercier wrote:
>> On Thu, Mar 28, 2024 at 7:53 AM Tvrtko Ursulin <tursulin@igalia.com> 
>> wrote:
>>>
>>> From: Tvrtko Ursulin <tursulin@ursulin.net>
>>>
>>> There is no point in compiling in the list and mutex operations 
>>> which are
>>> only used from the dma-buf debugfs code, if debugfs is not compiled in.
>>>
>>> Put the code in questions behind some kconfig guards and so save 
>>> some text
>>> and maybe even a pointer per object at runtime when not enabled.
>>>
>>> Signed-off-by: Tvrtko Ursulin <tursulin@ursulin.net>
>>
>> Reviewed-by: T.J. Mercier <tjmercier@google.com>
>
> Thanks!
>
> How would patches to dma-buf be typically landed? Via what tree I 
> mean? drm-misc-next?

That should go through drm-misc-next.

And feel free to add Reviewed-by: Christian König 
<christian.koenig@amd.com> as well.

Regards,
Christian.

>
> Regards,
>
> Tvrtko