* [PATCH] Fix compiler issue with mmap_internal
@ 2021-10-28 21:27 Stafford Horne
2021-10-29 12:14 ` Adhemerval Zanella
0 siblings, 1 reply; 2+ messages in thread
From: Stafford Horne @ 2021-10-28 21:27 UTC (permalink / raw)
To: GLIBC patches
Compiling mmap_internal fails to compile when we use -1 for MMAP2_PAGE_UNIT
on 32 bit architectures. The error is as follows:
../sysdeps/unix/sysv/linux/mmap_internal.h:30:8: error: unknown type
name 'uint64_t'
|
30 | static uint64_t page_unit;
|
| ^~~~~~~~
Fix by adding including stdint.h.
---
sysdeps/unix/sysv/linux/mmap_internal.h | 2 ++
1 file changed, 2 insertions(+)
diff --git a/sysdeps/unix/sysv/linux/mmap_internal.h b/sysdeps/unix/sysv/linux/mmap_internal.h
index 5ca6976191..35e112cc77 100644
--- a/sysdeps/unix/sysv/linux/mmap_internal.h
+++ b/sysdeps/unix/sysv/linux/mmap_internal.h
@@ -19,6 +19,8 @@
#ifndef MMAP_INTERNAL_LINUX_H
#define MMAP_INTERNAL_LINUX_H 1
+#include <stdint.h>
+
/* This is the minimum mmap2 unit size accept by the kernel. An architecture
with multiple minimum page sizes (such as m68k) might define it as -1 and
thus it will queried at runtime. */
--
2.31.1
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [PATCH] Fix compiler issue with mmap_internal
2021-10-28 21:27 [PATCH] Fix compiler issue with mmap_internal Stafford Horne
@ 2021-10-29 12:14 ` Adhemerval Zanella
0 siblings, 0 replies; 2+ messages in thread
From: Adhemerval Zanella @ 2021-10-29 12:14 UTC (permalink / raw)
To: libc-alpha
On 28/10/2021 18:27, Stafford Horne via Libc-alpha wrote:
> Compiling mmap_internal fails to compile when we use -1 for MMAP2_PAGE_UNIT
> on 32 bit architectures. The error is as follows:
>
> ../sysdeps/unix/sysv/linux/mmap_internal.h:30:8: error: unknown type
> name 'uint64_t'
> |
> 30 | static uint64_t page_unit;
> |
> | ^~~~~~~~
>
> Fix by adding including stdint.h.
LGTM, thanks.
Reviewed-by: Adhemerval Zanella <adhemerval.zanella@linaro.org>
> ---
> sysdeps/unix/sysv/linux/mmap_internal.h | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/sysdeps/unix/sysv/linux/mmap_internal.h b/sysdeps/unix/sysv/linux/mmap_internal.h
> index 5ca6976191..35e112cc77 100644
> --- a/sysdeps/unix/sysv/linux/mmap_internal.h
> +++ b/sysdeps/unix/sysv/linux/mmap_internal.h
> @@ -19,6 +19,8 @@
> #ifndef MMAP_INTERNAL_LINUX_H
> #define MMAP_INTERNAL_LINUX_H 1
>
> +#include <stdint.h>
> +
> /* This is the minimum mmap2 unit size accept by the kernel. An architecture
> with multiple minimum page sizes (such as m68k) might define it as -1 and
> thus it will queried at runtime. */
>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2021-10-29 12:14 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-10-28 21:27 [PATCH] Fix compiler issue with mmap_internal Stafford Horne
2021-10-29 12:14 ` Adhemerval Zanella
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).