Re: [PATCH] drm: Initialize list in event->base.link

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sun, Jun 16, 2013 at 5:07 AM, Daniel Vetter <daniel@xxxxxxxx> wrote:
> On Fri, Jun 14, 2013 at 01:39:05PM -0700, Stéphane Marchesin wrote:
>> The structure was kzalloced, so prev == next == NULL by default which
>> is invalid.
>>
>> Signed-off-by: Stéphane Marchesin <marcheu@xxxxxxxxxxxx>
>
> We do a list_add_tail which doesn't seem to care about unitizalized list
> items, and when removing it we also use list_del and not list_del_init. So
> I don't think we actually need this, neither for correctness nor for
> consitency. Or am I missing something?

I was going to say the same thing.

Dave.
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/dri-devel





[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux