[PATCH] drm/mgag200: fix build on alpha arch

[PATCH] drm/mgag200: fix build on alpha arch

[Sam Ravnborg]

When building imgag200 for the alpha architecture it fails like this:
mgag200_drv.c:233:9: error: implicit declaration of function ‘vmalloc’
  233 |  bios = vmalloc(size);
      |         ^~~~~~~
      |         kmalloc

When building for other architectures vmalloc.h is pulled in via some
other header file - for example asm-generic/io.h.
Use an explicit include of vmalloc.h to fix the build.

Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
Fixes: e20dfd27f7aa ("drm/mgag200: Add support for G200 desktop cards")
Cc: Thomas Zimmermann <tzimmermann@suse.de>
Cc: Egbert Eich <eich@suse.com>
Cc: Takashi Iwai <tiwai@suse.de>
Cc: Lyude Paul <lyude@redhat.com>
---
I have lost track if the offending commit is on the way to upstream or
it is just in drm-misc-next. But I think it is the latter so we can
apply this to drm-misc-next.

	Sam

 drivers/gpu/drm/mgag200/mgag200_drv.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/gpu/drm/mgag200/mgag200_drv.c b/drivers/gpu/drm/mgag200/mgag200_drv.c
index 09170d46aa53..b282b0e42c2d 100644
--- a/drivers/gpu/drm/mgag200/mgag200_drv.c
+++ b/drivers/gpu/drm/mgag200/mgag200_drv.c
@@ -9,6 +9,7 @@
 #include <linux/console.h>
 #include <linux/module.h>
 #include <linux/pci.h>
+#include <linux/vmalloc.h>
 
 #include <drm/drm_drv.h>
 #include <drm/drm_file.h>
-- 
2.25.1

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

Re: [PATCH] drm/mgag200: fix build on alpha arch

[Daniel Vetter]

On Fri, Aug 7, 2020 at 8:05 PM Sam Ravnborg <sam@ravnborg.org> wrote:
>
> When building imgag200 for the alpha architecture it fails like this:
> mgag200_drv.c:233:9: error: implicit declaration of function ‘vmalloc’
>   233 |  bios = vmalloc(size);
>       |         ^~~~~~~
>       |         kmalloc
>
> When building for other architectures vmalloc.h is pulled in via some
> other header file - for example asm-generic/io.h.
> Use an explicit include of vmalloc.h to fix the build.
>
> Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
> Fixes: e20dfd27f7aa ("drm/mgag200: Add support for G200 desktop cards")
> Cc: Thomas Zimmermann <tzimmermann@suse.de>
> Cc: Egbert Eich <eich@suse.com>
> Cc: Takashi Iwai <tiwai@suse.de>
> Cc: Lyude Paul <lyude@redhat.com>
> ---
> I have lost track if the offending commit is on the way to upstream or
> it is just in drm-misc-next. But I think it is the latter so we can
> apply this to drm-misc-next.

$ dim status # fetches all the trees
$ dim tag-contains e20dfd27f7aa
drm-misc/drm-misc-next

You guess correctly :-)

Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch>
>
>         Sam
>
>  drivers/gpu/drm/mgag200/mgag200_drv.c | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/drivers/gpu/drm/mgag200/mgag200_drv.c b/drivers/gpu/drm/mgag200/mgag200_drv.c
> index 09170d46aa53..b282b0e42c2d 100644
> --- a/drivers/gpu/drm/mgag200/mgag200_drv.c
> +++ b/drivers/gpu/drm/mgag200/mgag200_drv.c
> @@ -9,6 +9,7 @@
>  #include <linux/console.h>
>  #include <linux/module.h>
>  #include <linux/pci.h>
> +#include <linux/vmalloc.h>
>
>  #include <drm/drm_drv.h>
>  #include <drm/drm_file.h>
> --
> 2.25.1
>
> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel



-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

Re: [PATCH] drm/mgag200: fix build on alpha arch

[Sam Ravnborg]

Hi Daniel.

On Fri, Aug 07, 2020 at 10:36:34PM +0200, Daniel Vetter wrote:
> On Fri, Aug 7, 2020 at 8:05 PM Sam Ravnborg <sam@ravnborg.org> wrote:
> >
> > When building imgag200 for the alpha architecture it fails like this:
> > mgag200_drv.c:233:9: error: implicit declaration of function ‘vmalloc’
> >   233 |  bios = vmalloc(size);
> >       |         ^~~~~~~
> >       |         kmalloc
> >
> > When building for other architectures vmalloc.h is pulled in via some
> > other header file - for example asm-generic/io.h.
> > Use an explicit include of vmalloc.h to fix the build.
> >
> > Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
> > Fixes: e20dfd27f7aa ("drm/mgag200: Add support for G200 desktop cards")
> > Cc: Thomas Zimmermann <tzimmermann@suse.de>
> > Cc: Egbert Eich <eich@suse.com>
> > Cc: Takashi Iwai <tiwai@suse.de>
> > Cc: Lyude Paul <lyude@redhat.com>
> > ---
> > I have lost track if the offending commit is on the way to upstream or
> > it is just in drm-misc-next. But I think it is the latter so we can
> > apply this to drm-misc-next.
> 
> $ dim status # fetches all the trees
Hmm, except that it did not update drm-misc-next?!?

I have read "Commiters that want to check the status of their
current branch should use normal git status commands."

So that, I guess, may explain it.

> $ dim tag-contains e20dfd27f7aa
> drm-misc/drm-misc-next
Nice little command - but no "tag-contains" in my dim version.
There is "dim tc" but no-one have any chance guessing what tc
is an abbrevation for - sigh.

> 
> You guess correctly :-)
> 
> Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch>

Thanks, pushed to drm-misc-next.

	Sam


> >
> >         Sam
> >
> >  drivers/gpu/drm/mgag200/mgag200_drv.c | 1 +
> >  1 file changed, 1 insertion(+)
> >
> > diff --git a/drivers/gpu/drm/mgag200/mgag200_drv.c b/drivers/gpu/drm/mgag200/mgag200_drv.c
> > index 09170d46aa53..b282b0e42c2d 100644
> > --- a/drivers/gpu/drm/mgag200/mgag200_drv.c
> > +++ b/drivers/gpu/drm/mgag200/mgag200_drv.c
> > @@ -9,6 +9,7 @@
> >  #include <linux/console.h>
> >  #include <linux/module.h>
> >  #include <linux/pci.h>
> > +#include <linux/vmalloc.h>
> >
> >  #include <drm/drm_drv.h>
> >  #include <drm/drm_file.h>
> > --
> > 2.25.1
> >
> > _______________________________________________
> > dri-devel mailing list
> > dri-devel@lists.freedesktop.org
> > https://lists.freedesktop.org/mailman/listinfo/dri-devel
> 
> 
> 
> -- 
> Daniel Vetter
> Software Engineer, Intel Corporation
> http://blog.ffwll.ch
> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

Re: [PATCH] drm/mgag200: fix build on alpha arch

[Thomas Zimmermann]

[-- Attachment #1.1.1: Type: text/plain, Size: 1929 bytes --]

Hi Sam

thanks for taken care of this issue. Alpha is a rare architecture these
days. How do you build and test for it?

Am 07.08.20 um 20:05 schrieb Sam Ravnborg:
> When building imgag200 for the alpha architecture it fails like this:
> mgag200_drv.c:233:9: error: implicit declaration of function ‘vmalloc’
>   233 |  bios = vmalloc(size);
>       |         ^~~~~~~
>       |         kmalloc
> 
> When building for other architectures vmalloc.h is pulled in via some
> other header file - for example asm-generic/io.h.
> Use an explicit include of vmalloc.h to fix the build.
> 
> Signed-off-by: Sam Ravnborg <sam@ravnborg.org>

Sorry for being (too) late, but still:

Reviewed-by: Thomas Zimmermann <tzimmermann@suse.de>

Best regards
Thomas

> Fixes: e20dfd27f7aa ("drm/mgag200: Add support for G200 desktop cards")
> Cc: Thomas Zimmermann <tzimmermann@suse.de>
> Cc: Egbert Eich <eich@suse.com>
> Cc: Takashi Iwai <tiwai@suse.de>
> Cc: Lyude Paul <lyude@redhat.com>
> ---
> I have lost track if the offending commit is on the way to upstream or
> it is just in drm-misc-next. But I think it is the latter so we can
> apply this to drm-misc-next.
> 
> 	Sam
> 
>  drivers/gpu/drm/mgag200/mgag200_drv.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/drivers/gpu/drm/mgag200/mgag200_drv.c b/drivers/gpu/drm/mgag200/mgag200_drv.c
> index 09170d46aa53..b282b0e42c2d 100644
> --- a/drivers/gpu/drm/mgag200/mgag200_drv.c
> +++ b/drivers/gpu/drm/mgag200/mgag200_drv.c
> @@ -9,6 +9,7 @@
>  #include <linux/console.h>
>  #include <linux/module.h>
>  #include <linux/pci.h>
> +#include <linux/vmalloc.h>
>  
>  #include <drm/drm_drv.h>
>  #include <drm/drm_file.h>
> 

-- 
Thomas Zimmermann
Graphics Driver Developer
SUSE Software Solutions Germany GmbH
Maxfeldstr. 5, 90409 Nürnberg, Germany
(HRB 36809, AG Nürnberg)
Geschäftsführer: Felix Imendörffer


[-- Attachment #1.2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 516 bytes --]

[-- Attachment #2: Type: text/plain, Size: 160 bytes --]

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

Re: [PATCH] drm/mgag200: fix build on alpha arch

[Sam Ravnborg]

Hi Thomas.

On Tue, Aug 11, 2020 at 08:59:13AM +0200, Thomas Zimmermann wrote:
> Hi Sam
> 
> thanks for taken care of this issue. Alpha is a rare architecture these
> days. How do you build and test for it?

I am on ubuntu here so I have installed:
apt install gcc-alpha-linux-gnu

And then alpha is one of the architectures I build for.
My full list:
archs="alpha arm arm64 sparc64 i386 x86 powerpc s390 riscv sh"

I build these configs:
configs="allmodconfig allyesconfig allnoconfig defconfig"

It takes a while so usually I build only the relevant sub-set such as
drivers/gpu/drm/bridge/

I have, unfortunately, no testing. Would love something qemu based but
have never invested time into this.

	Sam

> 
> Am 07.08.20 um 20:05 schrieb Sam Ravnborg:
> > When building imgag200 for the alpha architecture it fails like this:
> > mgag200_drv.c:233:9: error: implicit declaration of function ‘vmalloc’
> >   233 |  bios = vmalloc(size);
> >       |         ^~~~~~~
> >       |         kmalloc
> > 
> > When building for other architectures vmalloc.h is pulled in via some
> > other header file - for example asm-generic/io.h.
> > Use an explicit include of vmalloc.h to fix the build.
> > 
> > Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
> 
> Sorry for being (too) late, but still:
> 
> Reviewed-by: Thomas Zimmermann <tzimmermann@suse.de>
> 
> Best regards
> Thomas
> 
> > Fixes: e20dfd27f7aa ("drm/mgag200: Add support for G200 desktop cards")
> > Cc: Thomas Zimmermann <tzimmermann@suse.de>
> > Cc: Egbert Eich <eich@suse.com>
> > Cc: Takashi Iwai <tiwai@suse.de>
> > Cc: Lyude Paul <lyude@redhat.com>
> > ---
> > I have lost track if the offending commit is on the way to upstream or
> > it is just in drm-misc-next. But I think it is the latter so we can
> > apply this to drm-misc-next.
> > 
> > 	Sam
> > 
> >  drivers/gpu/drm/mgag200/mgag200_drv.c | 1 +
> >  1 file changed, 1 insertion(+)
> > 
> > diff --git a/drivers/gpu/drm/mgag200/mgag200_drv.c b/drivers/gpu/drm/mgag200/mgag200_drv.c
> > index 09170d46aa53..b282b0e42c2d 100644
> > --- a/drivers/gpu/drm/mgag200/mgag200_drv.c
> > +++ b/drivers/gpu/drm/mgag200/mgag200_drv.c
> > @@ -9,6 +9,7 @@
> >  #include <linux/console.h>
> >  #include <linux/module.h>
> >  #include <linux/pci.h>
> > +#include <linux/vmalloc.h>
> >  
> >  #include <drm/drm_drv.h>
> >  #include <drm/drm_file.h>
> > 
> 
> -- 
> Thomas Zimmermann
> Graphics Driver Developer
> SUSE Software Solutions Germany GmbH
> Maxfeldstr. 5, 90409 Nürnberg, Germany
> (HRB 36809, AG Nürnberg)
> Geschäftsführer: Felix Imendörffer
> 



_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

Re: [PATCH] drm/mgag200: fix build on alpha arch

[Daniel Vetter]

On Tue, Aug 11, 2020 at 10:12:01AM +0200, Sam Ravnborg wrote:
> Hi Thomas.
> 
> On Tue, Aug 11, 2020 at 08:59:13AM +0200, Thomas Zimmermann wrote:
> > Hi Sam
> > 
> > thanks for taken care of this issue. Alpha is a rare architecture these
> > days. How do you build and test for it?
> 
> I am on ubuntu here so I have installed:
> apt install gcc-alpha-linux-gnu
> 
> And then alpha is one of the architectures I build for.
> My full list:
> archs="alpha arm arm64 sparc64 i386 x86 powerpc s390 riscv sh"
> 
> I build these configs:
> configs="allmodconfig allyesconfig allnoconfig defconfig"
> 
> It takes a while so usually I build only the relevant sub-set such as
> drivers/gpu/drm/bridge/
> 
> I have, unfortunately, no testing. Would love something qemu based but
> have never invested time into this.

So the long term idea here was that we'd do at least all the build testing
on gitlab. But moving the drm repos over to gitlab has kinda been on hold
for a bit, with all the gitlab.freedesktop.org struggles past few months.
It does look like it's on a really good track though.

Then we could do build testing for pretty much any reasonable
architecture, and also qemu based runs (within reasons at least). And
committers wouldn't have to duplicate it all for their own setup, since
it's all shared.
-Daniel

> 
> 	Sam
> 
> > 
> > Am 07.08.20 um 20:05 schrieb Sam Ravnborg:
> > > When building imgag200 for the alpha architecture it fails like this:
> > > mgag200_drv.c:233:9: error: implicit declaration of function ‘vmalloc’
> > >   233 |  bios = vmalloc(size);
> > >       |         ^~~~~~~
> > >       |         kmalloc
> > > 
> > > When building for other architectures vmalloc.h is pulled in via some
> > > other header file - for example asm-generic/io.h.
> > > Use an explicit include of vmalloc.h to fix the build.
> > > 
> > > Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
> > 
> > Sorry for being (too) late, but still:
> > 
> > Reviewed-by: Thomas Zimmermann <tzimmermann@suse.de>
> > 
> > Best regards
> > Thomas
> > 
> > > Fixes: e20dfd27f7aa ("drm/mgag200: Add support for G200 desktop cards")
> > > Cc: Thomas Zimmermann <tzimmermann@suse.de>
> > > Cc: Egbert Eich <eich@suse.com>
> > > Cc: Takashi Iwai <tiwai@suse.de>
> > > Cc: Lyude Paul <lyude@redhat.com>
> > > ---
> > > I have lost track if the offending commit is on the way to upstream or
> > > it is just in drm-misc-next. But I think it is the latter so we can
> > > apply this to drm-misc-next.
> > > 
> > > 	Sam
> > > 
> > >  drivers/gpu/drm/mgag200/mgag200_drv.c | 1 +
> > >  1 file changed, 1 insertion(+)
> > > 
> > > diff --git a/drivers/gpu/drm/mgag200/mgag200_drv.c b/drivers/gpu/drm/mgag200/mgag200_drv.c
> > > index 09170d46aa53..b282b0e42c2d 100644
> > > --- a/drivers/gpu/drm/mgag200/mgag200_drv.c
> > > +++ b/drivers/gpu/drm/mgag200/mgag200_drv.c
> > > @@ -9,6 +9,7 @@
> > >  #include <linux/console.h>
> > >  #include <linux/module.h>
> > >  #include <linux/pci.h>
> > > +#include <linux/vmalloc.h>
> > >  
> > >  #include <drm/drm_drv.h>
> > >  #include <drm/drm_file.h>
> > > 
> > 
> > -- 
> > Thomas Zimmermann
> > Graphics Driver Developer
> > SUSE Software Solutions Germany GmbH
> > Maxfeldstr. 5, 90409 Nürnberg, Germany
> > (HRB 36809, AG Nürnberg)
> > Geschäftsführer: Felix Imendörffer
> > 
> 
> 
> 
> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel

-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel