Re: kernel build failure on amd64 -- genfb_machdep.o

[Paul Goyette <paul%whooppee.com@localhost>]

On Tue, 8 Feb 2011, Julian Fagir wrote:

> Hi,
>
> I have the same problem.
> Even after commenting out the failing positions (just debug-information), the
> kernel won't build because x86_genfb_mtrr_init has no previous prototype.
> After applying these two patches (appended), the kernel built fine for me.
>
> Don't treat these patches as useful, they're only suitable for this single
> problem. The first patch will fail to build on 32-bit-systems, but I know no
> useful substitute for displaying this uint64_t generically on 32- and
> 64-bit-systems of different platforms.

use PRIu64

        printf("%"PRIu64, uint64_t_value);

:)


> > cc1: warnings being treated as errors
> > /usr/src/sys/arch/x86/x86/genfb_machdep.c:79: warning: no previous
> > prototype for 'x86_genfb_mtrr_init'
> >   1
> > /usr/src/sys/arch/x86/x86/genfb_machdep.c: In function
> > 'x86_genfb_mtrr_init': /usr/src/sys/arch/x86/x86/genfb_machdep.c:96:
> > warning: format '%llx' expects type 'long long unsigned int', but argument
> > 3 has type '__uint64_t'
> > /usr/src/sys/arch/x86/x86/genfb_machdep.c:96: warning: format '%llx'
> > expects type 'long long unsigned int', but argument 4 has type
> > '__uint64_t'
>
>
> Regards, Julian

-------------------------------------------------------------------------
| Paul Goyette     | PGP Key fingerprint:     | E-mail addresses:       |
| Customer Service | FA29 0E3B 35AF E8AE 6651 | paul at whooppee.com    |
| Network Engineer | 0786 F758 55DE 53BA 7731 | pgoyette at juniper.net |
| Kernel Developer |                          | pgoyette at netbsd.org  |
-------------------------------------------------------------------------