[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <CAJHCu1LburVFrSX3uUhpRwSnYp-=Hm+KW-Vg_NOZG_Z5y4KmqA@mail.gmail.com>
Date: Wed, 14 Mar 2018 13:43:52 +0100
From: Salvatore Mesoraca <s.mesoraca16@...il.com>
To: Joonas Lahtinen <joonas.lahtinen@...ux.intel.com>
Cc: linux-kernel@...r.kernel.org, dri-devel@...ts.freedesktop.org,
intel-gfx@...ts.freedesktop.org,
Kernel Hardening <kernel-hardening@...ts.openwall.com>,
David Airlie <airlied@...ux.ie>,
Jani Nikula <jani.nikula@...ux.intel.com>,
Kees Cook <keescook@...omium.org>,
Rodrigo Vivi <rodrigo.vivi@...el.com>
Subject: Re: [PATCH] drm/i915: drop various VLAs in i915_debugfs.c
2018-03-14 13:27 GMT+01:00 Joonas Lahtinen <joonas.lahtinen@...ux.intel.com>:
> CHV_SS_MAX should be good enough. Make these function scoped (so #define
> at the beginning and #undef at the end of function).
>
> Do use ARRAY_SIZE() instead of repeating.
Thank you very much for your suggestions.
Unfortunately, it seems that someone else already fixed this issue, so
I'll just drop this patch.
Best regards,
Salvatore
Powered by blists - more mailing lists