[PATCH v3] drm/amdgpu: update documentation for amdgpu_drv.c

Sonny.Jiang@xxxxxxx (Jiang, Sonny) · Thu, 5 Jul 2018 14:47:07 +0000

Change it to,

Value 0 is invalidated, will be adjusted to 10000.  Negative values mean 'infinite timeout' (MAX_JIFFY_OFFSET).

Thanks,

Sonny

________________________________
From: Christian KÃ¶nig <ckoenig.leichtzumerken@xxxxxxxxx>
Sent: Thursday, July 5, 2018 3:30:28 AM
To: Jiang, Sonny; amd-gfx at lists.freedesktop.org
Subject: Re: [PATCH v3] drm/amdgpu: update documentation for amdgpu_drv.c

Am 04.07.2018 um 20:20 schrieb Sonny Jiang:
> [SNIP]
>
> +/**
> + * DOC: lockup_timeout (int)
> + * Set GPU scheduler timeout value in ms. It must be > 0.  The default is 10000.
> + */
>   MODULE_PARM_DESC(lockup_timeout, "GPU lockup timeout in ms > 0 (default 10000)");
>   module_param_named(lockup_timeout, amdgpu_lockup_timeout, int, 0444);

Actually the description here is incorrect, a value of 0 is handled as
infinite timeout IIRC. Please fix while at it.

Apart from that the patch is Acked-by: Christian KÃ¶nig
<christian.koenig at amd.com>.

Christian.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/amd-gfx/attachments/20180705/60cddaa9/attachment.html>