Re: [PATCH 3/5] drm/debugfs: remove dev->debugfs_list and debugfs_mutex v2

2023-04-13 Thread Daniel Vetter
On Thu, Apr 13, 2023 at 11:34:45AM +0200, Christian König wrote: > Am 12.04.23 um 17:11 schrieb Daniel Vetter: > > On Wed, Apr 12, 2023 at 04:52:04PM +0200, Christian König wrote: > > > The mutex was completely pointless in the first place since any > > > parallel adding of files to this list would

Re: [PATCH 3/5] drm/debugfs: remove dev->debugfs_list and debugfs_mutex v2

2023-04-13 Thread Christian König
Am 12.04.23 um 17:11 schrieb Daniel Vetter: On Wed, Apr 12, 2023 at 04:52:04PM +0200, Christian König wrote: The mutex was completely pointless in the first place since any parallel adding of files to this list would result in random behavior since the list is filled and consumed multiple times.

Re: [PATCH 3/5] drm/debugfs: remove dev->debugfs_list and debugfs_mutex v2

2023-04-12 Thread Daniel Vetter
On Wed, Apr 12, 2023 at 04:52:04PM +0200, Christian König wrote: > The mutex was completely pointless in the first place since any > parallel adding of files to this list would result in random > behavior since the list is filled and consumed multiple times. > > Completely drop that approach and j

[PATCH 3/5] drm/debugfs: remove dev->debugfs_list and debugfs_mutex v2

2023-04-12 Thread Christian König
The mutex was completely pointless in the first place since any parallel adding of files to this list would result in random behavior since the list is filled and consumed multiple times. Completely drop that approach and just create the files directly but return -ENODEV while opening the file whe