[<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
Confused about mailing lists and their use?
Read about mailing lists on Wikipedia
and check out these
guidelines on proper formatting of your messages.
