Re: [PATCH v3 4/5] drm/framebuffer: Include linux/sched.h in header file

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

 



On Thu, Nov 29, 2018 at 4:14 PM Noralf Trønnes <noralf@xxxxxxxxxxx> wrote:
>
>
> Den 29.11.2018 10.35, skrev Daniel Vetter:
> > On Wed, Nov 28, 2018 at 10:27:12PM +0100, Noralf Trønnes wrote:
> >> A cleanup of the includes in drm_fb_cma_helper resulted in:
> >>
> >>    CC [M]  drivers/gpu/drm/drm_fb_cma_helper.o
> >> In file included from drivers/gpu/drm/drm_fb_cma_helper.c:21:0:
> >> include/drm/drm_framebuffer.h:128:12: error: ‘TASK_COMM_LEN’ undeclared here (not in a function)
> >>    char comm[TASK_COMM_LEN];
> > Huh, I'm not seeing this. And we have 3 defconfigs for compile-testing,
> > they still seem to work. Which patch broke this? Should be quick to bisect
> > since fully automateable ... And would be good to have the Fixes: tag for
> > this.
> >
> > Anyway, Acked-by: Daniel Vetter <daniel.vetter@xxxxxxxx>
>
>
> My wording isn't optimal I see. It's the cleanup in the next patch that
> uncovered this.
>
> Maybe this is better:
>
> drm_framebuffer.h references TASK_COMM_LEN but doesn't include the header
> that defines this macro. Include linux/sched.h so the user doesn't have
> to make sure it's present.

Yeah just include this, and we're good to go I think.
-Daniel

>
>
> Noralf.
>
> >> Fix it by including linux/sched.h.
> >>
> >> Signed-off-by: Noralf Trønnes <noralf@xxxxxxxxxxx>
> >> ---
> >>   include/drm/drm_framebuffer.h | 1 +
> >>   1 file changed, 1 insertion(+)
> >>
> >> diff --git a/include/drm/drm_framebuffer.h b/include/drm/drm_framebuffer.h
> >> index c94acedfb08e..d4ce58568fa6 100644
> >> --- a/include/drm/drm_framebuffer.h
> >> +++ b/include/drm/drm_framebuffer.h
> >> @@ -25,6 +25,7 @@
> >>
> >>   #include <linux/list.h>
> >>   #include <linux/ctype.h>
> >> +#include <linux/sched.h>
> >>   #include <drm/drm_mode_object.h>
> >>
> >>   struct drm_framebuffer;
> >> --
> >> 2.15.1
> >>
> >> _______________________________________________
> >> dri-devel mailing list
> >> dri-devel@xxxxxxxxxxxxxxxxxxxxx
> >> https://lists.freedesktop.org/mailman/listinfo/dri-devel



-- 
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
https://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