Re: [Qemu-devel] [PATCH] oslib-posix: check for posix_memalign in configure script

2018-02-09 Thread Michael Tokarev
04.01.2018 20:39, Andreas Gustafsson wrote: > Check for the presence of posix_memalign() in the configure script, > not using "defined(_POSIX_C_SOURCE) && !defined(__sun__)". This > lets qemu use posix_memalign() on NetBSD versions that have it, > instead of falling back to valloc() which is waste

Re: [Qemu-devel] [PATCH] oslib-posix: check for posix_memalign in configure script

2018-01-08 Thread Kamil Rytarowski
On 08.01.2018 17:01, Andreas Gustafsson wrote: > Peter Maydell wrote: >> Can we put this through the -trivial tree? (cc'd) > > I'm not sufficiently familiar with the intenal workflows of the qemu > project to give a meaningful answer to that question. > I don't maintain a NetBSD merge queue and

Re: [Qemu-devel] [PATCH] oslib-posix: check for posix_memalign in configure script

2018-01-08 Thread Andreas Gustafsson
Peter Maydell wrote: > Can we put this through the -trivial tree? (cc'd) I'm not sufficiently familiar with the intenal workflows of the qemu project to give a meaningful answer to that question. -- Andreas Gustafsson, g...@gson.org

Re: [Qemu-devel] [PATCH] oslib-posix: check for posix_memalign in configure script

2018-01-08 Thread Peter Maydell
On 8 January 2018 at 16:01, Andreas Gustafsson wrote: > Peter Maydell wrote: >> Can we put this through the -trivial tree? (cc'd) > > I'm not sufficiently familiar with the intenal workflows of the qemu > project to give a meaningful answer to that question. Sorry, I should have been clearer, tha

Re: [Qemu-devel] [PATCH] oslib-posix: check for posix_memalign in configure script

2018-01-08 Thread Peter Maydell
On 4 January 2018 at 17:39, Andreas Gustafsson wrote: > Check for the presence of posix_memalign() in the configure script, > not using "defined(_POSIX_C_SOURCE) && !defined(__sun__)". This > lets qemu use posix_memalign() on NetBSD versions that have it, > instead of falling back to valloc() whi

Re: [Qemu-devel] [PATCH] oslib-posix: check for posix_memalign in configure script

2018-01-05 Thread Kamil Rytarowski
On 04.01.2018 18:39, Andreas Gustafsson wrote: > Check for the presence of posix_memalign() in the configure script, > not using "defined(_POSIX_C_SOURCE) && !defined(__sun__)". This > lets qemu use posix_memalign() on NetBSD versions that have it, > instead of falling back to valloc() which is wa

[Qemu-devel] [PATCH] oslib-posix: check for posix_memalign in configure script

2018-01-04 Thread Andreas Gustafsson
Check for the presence of posix_memalign() in the configure script, not using "defined(_POSIX_C_SOURCE) && !defined(__sun__)". This lets qemu use posix_memalign() on NetBSD versions that have it, instead of falling back to valloc() which is wasteful when the required alignment is smaller than a pa