[prev in list] [next in list] [prev in thread] [next in thread] 

List:       dri-devel
Subject:    Re: [PATCH 4/5] drm/debugfs: remove dev->debugfs_list and debugfs_mutex v2
From:       Andi Shyti <andi.shyti () linux ! intel ! com>
Date:       2023-08-31 21:56:06
Message-ID: ZPEMdoMvIjX/54Dd () ashyti-mobl2 ! lan
[Download RAW message or body]

Hi Christian,

On Tue, Aug 29, 2023 at 01:01:14PM +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 just create the files directly but
> return -ENODEV while opening the file when the minors are not
> registered yet.
> 
> v2: rebase on debugfs directory rework, limit access before minors are
>     registered.
> 
> Signed-off-by: Christian König <christian.koenig@amd.com>

Reviewed-by: Andi Shyti <andi.shyti@linux.intel.com> 

Andi
[prev in list] [next in list] [prev in thread] [next in thread]