Re: [PATCH AUTOSEL for 4.15 142/189] firmware: dmi_scan: Fix handling of empty DMI strings
From: Parag Warudkar
Date: Mon Apr 09 2018 - 09:45:04 EST
On Mon, Apr 9, 2018 at 8:48 AM, Jean Delvare <jdelvare@xxxxxxx> wrote:
> As a conclusion, it's doable, but the benefit is very small and limited
> to a few broken systems, and it has the downside of not discouraging
> low-quality tables, so my position is that it's not worth it and not
> desirable.
Thanks for the explanation Jean, I just wanted to make sure we are not
going back to the original problem of allocating lots of memory for
lots of spaces. Given the limited benefit and low likelihood of
encountering lots of strings with lots of leading and trailing spaces,
I agree, it's best to stick to the spec. And it's also no longer a
static 2048 byte array we are allocating from anymore!
--Parag