Re: Error in accel/tcg?

[Kenneth Adam Miller <kennethadammiller@gmail.com>]

[-- Attachment #1: Type: text/plain, Size: 1369 bytes --]

I saw that TARGET_PAGE_SIZE gets a value that depends on TARGET_PAGE_BITS
using tags and grep, but I did not see either of them take a value that is
beyond the idxmap size.

On Tue, Jul 20, 2021 at 5:07 AM Peter Maydell <peter.maydell@linaro.org>
wrote:

> On Mon, 19 Jul 2021 at 23:20, Kenneth Adam Miller
> <kennethadammiller@gmail.com> wrote:
> >
> > Hello,
> >
> > I get the following error:
> >
> > <long cmd here> -c ../accel/tcg/cputlb.c
> > ../qemu/accel/tcg/cputlb.c: In function 'tlb_flush_page_by_mmuidx':
> > ../qemu/accel/tcg/cputlb.c:602:23: error: comparison is always true due
> to limited range of data type [-Werror=type-limits]
> >     } else if (idxmap < TARGET_PAGE_SIZE) {
> >
> > I don't know why that suddenly shows up.
>
> So, which target are you building for, which host, and which
> compiler version? (TARGET_PAGE_SIZE gets a value that depends
> on the TARGET_PAGE_BITS setting for the target.)
>
> > In other parts of the code, I think somehow the meson build system
> >is triggering errors for warnings
>
> The QEMU build system defaults to warnings-are-errors when
> building from git. You can turn this off by passing
> configure '--disable-werror'. (Note that that's a bad idea if
> you're working on the QEMU source code, because you want to
> be able to see and fix the warnings in the code you're working on.)
>
> thanks
> -- PMM
>

[-- Attachment #2: Type: text/html, Size: 1895 bytes --]