Hi. On Thu, Aug 29, 2019 at 02:05:28AM -0400, Kenny Ho <Kenny.Ho@xxxxxxx> wrote: > diff --git a/drivers/gpu/drm/ttm/ttm_bo.c b/drivers/gpu/drm/ttm/ttm_bo.c > @@ -1256,6 +1257,12 @@ int ttm_bo_validate(struct ttm_buffer_object *bo, > [...] > + move_delay /= 2000; /* check every half period in ms*/ > [...] > diff --git a/kernel/cgroup/drm.c b/kernel/cgroup/drm.c > [...] > @@ -382,6 +548,25 @@ static ssize_t drmcg_limit_write(struct kernfs_open_file *of, char *buf, > [...] > + if (rc || val < 2000) { This just caught my eye and it may be simply caused by RFC-ness of the series but I'd suggest turning this into a constant with descriptive name. My 2 cents, Michal
Attachment:
signature.asc
Description: Digital signature
_______________________________________________ amd-gfx mailing list amd-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/amd-gfx