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

Dave Airlie airlied at gmail.com
Sun Jun 16 01:15:48 PDT 2013


On Sun, Jun 16, 2013 at 5:07 AM, Daniel Vetter <daniel at ffwll.ch> 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 at chromium.org>
>
> 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.


More information about the dri-devel mailing list