| Message ID | 20201221005318.11866-8-jiaxun.yang@flygoat.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | Alpine Linux build fix and CI pipeline | expand |
On 21/12/2020 01.53, Jiaxun Yang wrote: > As per POSIX specification of limits.h [1], OS libc may define > PAGE_SIZE in limits.h. > > To prevent collosion of definition, we discard PAGE_SIZE from > defined by libc and take QEMU's variable. > > [1]: https://pubs.opengroup.org/onlinepubs/7908799/xsh/limits.h.html > > Signed-off-by: Jiaxun Yang <jiaxun.yang@flygoat.com> > --- > accel/kvm/kvm-all.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/accel/kvm/kvm-all.c b/accel/kvm/kvm-all.c > index 389eaace72..3feb17d965 100644 > --- a/accel/kvm/kvm-all.c > +++ b/accel/kvm/kvm-all.c > @@ -58,6 +58,9 @@ > /* KVM uses PAGE_SIZE in its definition of KVM_COALESCED_MMIO_MAX. We > * need to use the real host PAGE_SIZE, as that's what KVM will use. > */ > +#ifdef PAGE_SIZE > +#undef PAGE_SIZE > +#endif > #define PAGE_SIZE qemu_real_host_page_size If I get that right, the PAGE_SIZE macro is only used one time in this file... so it's maybe easier to get rid of the macro completely and replace the single occurance with qemu_real_host_page_size directly? Thomas
On Wed, Jan 13, 2021, at 3:19 PM, Thomas Huth wrote: > On 21/12/2020 01.53, Jiaxun Yang wrote: > > As per POSIX specification of limits.h [1], OS libc may define > > PAGE_SIZE in limits.h. > > > > To prevent collosion of definition, we discard PAGE_SIZE from > > defined by libc and take QEMU's variable. > > > > [1]: https://pubs.opengroup.org/onlinepubs/7908799/xsh/limits.h.html > > > > Signed-off-by: Jiaxun Yang <jiaxun.yang@flygoat.com> > > --- > > accel/kvm/kvm-all.c | 3 +++ > > 1 file changed, 3 insertions(+) > > > > diff --git a/accel/kvm/kvm-all.c b/accel/kvm/kvm-all.c > > index 389eaace72..3feb17d965 100644 > > --- a/accel/kvm/kvm-all.c > > +++ b/accel/kvm/kvm-all.c > > @@ -58,6 +58,9 @@ > > /* KVM uses PAGE_SIZE in its definition of KVM_COALESCED_MMIO_MAX. We > > * need to use the real host PAGE_SIZE, as that's what KVM will use. > > */ > > +#ifdef PAGE_SIZE > > +#undef PAGE_SIZE > > +#endif > > #define PAGE_SIZE qemu_real_host_page_size > > If I get that right, the PAGE_SIZE macro is only used one time in this > file... so it's maybe easier to get rid of the macro completely and replace > the single occurance with qemu_real_host_page_size directly? It is also used in kernel headers included here. Thanks. - Jiaxun > > Thomas > >
diff --git a/accel/kvm/kvm-all.c b/accel/kvm/kvm-all.c index 389eaace72..3feb17d965 100644 --- a/accel/kvm/kvm-all.c +++ b/accel/kvm/kvm-all.c @@ -58,6 +58,9 @@ /* KVM uses PAGE_SIZE in its definition of KVM_COALESCED_MMIO_MAX. We * need to use the real host PAGE_SIZE, as that's what KVM will use. */ +#ifdef PAGE_SIZE +#undef PAGE_SIZE +#endif #define PAGE_SIZE qemu_real_host_page_size //#define DEBUG_KVM
As per POSIX specification of limits.h [1], OS libc may define PAGE_SIZE in limits.h. To prevent collosion of definition, we discard PAGE_SIZE from defined by libc and take QEMU's variable. [1]: https://pubs.opengroup.org/onlinepubs/7908799/xsh/limits.h.html Signed-off-by: Jiaxun Yang <jiaxun.yang@flygoat.com> --- accel/kvm/kvm-all.c | 3 +++ 1 file changed, 3 insertions(+)