On Tue, 20 Nov 2018 08:49:24 -0800, Christoph Hellwig wrote: > This could use a little more explanation, the code doesn't just > add a little if but also changes the existing case. Also where > can't it be null currently? I'll add an explanation in the next round. This patch shouldn't cause any change in behaviour. The vendor string is currently always NULL terminated, but won't be once patch 3/4 is applied, hence the need to add the maximum string width specifier. Thanks for the feedback.