<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Jul 29, 2019 at 9:58 AM Eugene Syromiatnikov <<a href="mailto:esyr@redhat.com">esyr@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><br>
> +# include "xlat/drm_stat_type.h"<br>
> +<br>
> +static int<br>
> +drm_get_stats(struct tcb *const tcp, const kernel_ulong_t arg)<br>
> +{<br>
> +     struct_drm_stats stats;<br>
> +<br>
> +     if (exiting(tcp)) {<br>
> +             tprints(", ");<br>
> +             if (umove_or_printaddr(tcp, arg, &stats))<br>
> +                     return RVAL_IOCTL_DECODED;<br>
> +             PRINT_FIELD_U("{", stats, count);<br>
> +             tprints(", data=[");<br>
> +<br>
<br>
> +             for (unsigned int i = 0; i < 15; i++) {<br>
Why is "15" hard-coded?<br>
<br>
Also, it seems that it should go up to "count".<br></blockquote><div><br></div><div>I just found that in kernel commit v3.12-rc1~61^2~98, GET_STATS</div><div>was no-oped out and there is a explanation about it. Do we have to</div><div>support decoding of this one? (Some other ioctls may come across</div><div>similar situations, how do we choose which is the oldest version to</div><div>support?)</div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><br>
</blockquote></div></div>