lkml.org 
[lkml]   [2022]   [Jul]   [20]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] mm/cma_debug.c: align the name buffer length as struct cma
On Wed, 20 Jul 2022 10:45:33 +0000 "Kassey Li (QUIC)" <quic_yingangl@quicinc.com> wrote:

> Hi, Andrew:
>
> Is it the case that the output is never >16 chars anyway? If so, we'll be OK.
> >> Yes. I verified on my device by giving cma.name length l< 16 and >16 both. Result is OK. I mean > 16 showed correctly with this patch.
> >> without this patch, cma.name > 16, only showed 16 characters.
>

Well that's my point. With this patch the output is changed and this
might break userspace code which expects the truncation.

That should be OK as there are no guarantees with debugfs interfaces.
I added this as the patch's changelog:


: Avoids truncating the debugfs output to 16 chars. Potentially alters
: the userspace output, but this is a debugfs interface and there are no
: stability guarantees.


\
 
 \ /
  Last update: 2022-07-20 19:04    [W:0.659 / U:0.004 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site