<div dir="ltr"><br><div class="gmail_extra"><div class="gmail_quote">On Fri, Apr 15, 2016 at 11:25 AM, Ilia Mirkin <span dir="ltr"><<a href="mailto:imirkin@alum.mit.edu" target="_blank">imirkin@alum.mit.edu</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="">On Fri, Apr 15, 2016 at 11:22 AM, Pierre Moreau <<a href="mailto:pierre.morrow@free.fr">pierre.morrow@free.fr</a>> wrote:<br>
> On 11:06 AM - Apr 15 2016, Ilia Mirkin wrote:<br>
>> On Fri, Apr 15, 2016 at 10:57 AM, Pierre Moreau <<a href="mailto:pierre.morrow@free.fr">pierre.morrow@free.fr</a>> wrote:<br>
>> > Currently, every backlight interface created by Nouveau uses the same name,<br>
>> > nv_backlight. This leads to a sysfs warning as it tries to create an already<br>
>> > existing folder. This patch adds a incremented number to the name, but keeps<br>
>> > the initial name as nv_backlight, to avoid possibly breaking userspace; the<br>
>> > second interface will be named nv_backlight1, and so on.<br>
>> ><br>
>> > Fixes: fdo#86539<br></span></blockquote><div> I believe Bugzilla: <a href="https://bugs.freedesktop.org/show_bug.cgi?id=86539">https://bugs.freedesktop.org/show_bug.cgi?id=86539</a> is the preferred format. I think this is picked up by the mesa release scripts or some such.<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="">
>> > Signed-off-by: Pierre Moreau <<a href="mailto:pierre.morrow@free.fr">pierre.morrow@free.fr</a>><br>
>> > ---<br>
>> >  drm/nouveau/nouveau_backlight.c | 35 +++++++++++++++++++++++++++++++++--<br>
>> >  1 file changed, 33 insertions(+), 2 deletions(-)<br>
>> ><br>
>> > diff --git a/drm/nouveau/nouveau_backlight.c b/drm/nouveau/nouveau_backlight.c<br>
>> > index 89eb460..914e2cb 100644<br>
>> > --- a/drm/nouveau/nouveau_backlight.c<br>
>> > +++ b/drm/nouveau/nouveau_backlight.c<br>
>> > @@ -36,6 +36,10 @@<br>
>> >  #include "nouveau_reg.h"<br>
>> >  #include "nouveau_encoder.h"<br>
>> ><br>
>> > +static atomic_t bl_interfaces_nb = { 0 };<br>
>><br>
>> static data is initialized to 0, this should be unnecessary.<br>
><br>
> I didn’t know that. But on the other hand, I like having it explicit, and it<br>
> should not add any overhead.<br>
<br>
</span>It increases the size of the object file. I believe it's kernel policy<br>
to avoid static initializations to 0. (Note that this doesn't hold in<br>
regular user applications, just the kernel.)<br>
<span class=""><font color="#888888"><br>
  -ilia<br>
</font></span><div class=""><div class="h5">_______________________________________________<br>
Nouveau mailing list<br>
<a href="mailto:Nouveau@lists.freedesktop.org">Nouveau@lists.freedesktop.org</a><br>
<a href="https://lists.freedesktop.org/mailman/listinfo/nouveau" rel="noreferrer" target="_blank">https://lists.freedesktop.org/mailman/listinfo/nouveau</a><br>
</div></div></blockquote></div><br></div></div>