Re: [PATCH i-g-t 2/2] tests/core_hotunplug: Fix inline comment on missing device nodes

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Janusz,
On 2024-07-26 at 10:34:57 +0200, Janusz Krzysztofik wrote:
> The comment incorrectly blames udev for being responsible for delays in
> re-creation of device nodes after driver rebind.  While that could be
> true many years ago, nowadays that's the role of kernel devtmpfs.

Thank you for clarification,

Reviewed-by: Kamil Konieczny <kamil.konieczny@xxxxxxxxxxxxxxx>

> 
> Signed-off-by: Janusz Krzysztofik <janusz.krzysztofik@xxxxxxxxxxxxxxx>
> ---
>  tests/core_hotunplug.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/tests/core_hotunplug.c b/tests/core_hotunplug.c
> index db7a5c5e84..145fe141ed 100644
> --- a/tests/core_hotunplug.c
> +++ b/tests/core_hotunplug.c
> @@ -480,7 +480,7 @@ static void node_healthcheck(struct hotunplug *priv, unsigned flags)
>  
>  static bool healthcheck(struct hotunplug *priv, bool recover)
>  {
> -	/* give udev some time to recreate device nodes before we continue */
> +	/* give devtmpfs some time to recreate device nodes before we continue */
>  	sleep(1);
>  
>  	/* device name may have changed, rebuild IGT device list */
> -- 
> 2.45.2
> 



[Index of Archives]     [AMD Graphics]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux