[PATCH v3 0/9] Build fixes for Haiku

Peter Maydell posted 9 patches 3 years, 9 months ago
Test FreeBSD passed
Test docker-quick@centos7 passed
Test checkpatch passed
Test docker-mingw@fedora passed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20200703145614.16684-1-peter.maydell@linaro.org
Maintainers: Paolo Bonzini <pbonzini@redhat.com>, Paul Durrant <paul@xen.org>, Anthony Perard <anthony.perard@citrix.com>, Stefano Stabellini <sstabellini@kernel.org>
configure                   | 36 ++++++++++++++++++++++++++++++++++--
include/qemu/bswap.h        |  2 ++
include/qemu/osdep.h        |  6 +++++-
hw/xen/xen-legacy-backend.c |  1 -
os-posix.c                  |  4 ++++
util/compatfd.c             |  2 ++
util/drm.c                  | 19 ++++++++++++++-----
util/oslib-posix.c          | 20 +++++++++++++++++++-
util/qemu-openpty.c         |  2 +-
9 files changed, 81 insertions(+), 11 deletions(-)
[PATCH v3 0/9] Build fixes for Haiku
Posted by Peter Maydell 3 years, 9 months ago
This patchset is essentially a resend of David Carlier's build fixes
for the Haiku platform. I've taken David's patches and put them together
into a set of emails threaded in the way our CI tools expect, as the
easiest way to get the patchew robot to run the build tests so we can
check they didn't accidentally break one of the BSDs. I've also put
in the patch from Gerd that fixes the drm.c issue. I've also expanded
on the commit messages for all the patches so that we have a record
of why we made the changes if we need to look back at it in future.

Assuming no issues turn up, I'll arrange to get these into master
at some point before the 5.1 release.

Thanks for reporting these problems and providing the fixes, David.

-- PMM

David CARLIER (8):
  build: Enable BSD symbols for Haiku
  util/qemu-openpty.c: Don't assume pty.h is glibc-only
  build: Check that mlockall() exists
  osdep.h: Always include <sys/signal.h> if it exists
  osdep.h: For Haiku, define SIGIO as equivalent to SIGPOLL
  bswap.h: Include <endian.h> on Haiku for bswap operations
  util/compatfd.c: Only include <sys/syscall.h> if CONFIG_SIGNALFD
  util/oslib-posix.c: Implement qemu_init_exec_dir() for Haiku

Gerd Hoffmann (1):
  util/drm: make portable by avoiding struct dirent d_type

 configure                   | 36 ++++++++++++++++++++++++++++++++++--
 include/qemu/bswap.h        |  2 ++
 include/qemu/osdep.h        |  6 +++++-
 hw/xen/xen-legacy-backend.c |  1 -
 os-posix.c                  |  4 ++++
 util/compatfd.c             |  2 ++
 util/drm.c                  | 19 ++++++++++++++-----
 util/oslib-posix.c          | 20 +++++++++++++++++++-
 util/qemu-openpty.c         |  2 +-
 9 files changed, 81 insertions(+), 11 deletions(-)

-- 
2.20.1


Re: [PATCH v3 0/9] Build fixes for Haiku
Posted by Peter Maydell 3 years, 9 months ago
On Fri, 3 Jul 2020 at 15:56, Peter Maydell <peter.maydell@linaro.org> wrote:
>
> This patchset is essentially a resend of David Carlier's build fixes
> for the Haiku platform. I've taken David's patches and put them together
> into a set of emails threaded in the way our CI tools expect, as the
> easiest way to get the patchew robot to run the build tests so we can
> check they didn't accidentally break one of the BSDs. I've also put
> in the patch from Gerd that fixes the drm.c issue. I've also expanded
> on the commit messages for all the patches so that we have a record
> of why we made the changes if we need to look back at it in future.
>
> Assuming no issues turn up, I'll arrange to get these into master
> at some point before the 5.1 release.

I've made the minor renames from CONFIG_ to HAVE_ where suggested
and will put these into the tree via a target-arm pullreq I'm
planning for Monday.

thanks
-- PMM

Re: [PATCH v3 0/9] Build fixes for Haiku
Posted by David CARLIER 3 years, 9 months ago
Alright thanks for the assistance. Regards.

On Sat, 11 Jul 2020 at 19:44, Peter Maydell <peter.maydell@linaro.org>
wrote:

> On Fri, 3 Jul 2020 at 15:56, Peter Maydell <peter.maydell@linaro.org>
> wrote:
> >
> > This patchset is essentially a resend of David Carlier's build fixes
> > for the Haiku platform. I've taken David's patches and put them together
> > into a set of emails threaded in the way our CI tools expect, as the
> > easiest way to get the patchew robot to run the build tests so we can
> > check they didn't accidentally break one of the BSDs. I've also put
> > in the patch from Gerd that fixes the drm.c issue. I've also expanded
> > on the commit messages for all the patches so that we have a record
> > of why we made the changes if we need to look back at it in future.
> >
> > Assuming no issues turn up, I'll arrange to get these into master
> > at some point before the 5.1 release.
>
> I've made the minor renames from CONFIG_ to HAVE_ where suggested
> and will put these into the tree via a target-arm pullreq I'm
> planning for Monday.
>
> thanks
> -- PMM
>