[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] Re: [PATCH 5/6] qemu fixes
From: |
M. Warner Losh |
Subject: |
Re: [Qemu-devel] Re: [PATCH 5/6] qemu fixes |
Date: |
Fri, 28 Nov 2008 09:46:04 -0700 (MST) |
In message: <address@hidden>
Christoph Egger <address@hidden> writes:
: On Friday 28 November 2008 12:34:54 Jan Kiszka wrote:
: > Christoph Egger wrote:
: > > Hi,
: > >
: > > This is a series of four patches which improve support
: > > for qemu on NetBSD.
: > >
: > > sys-queue.h defines _SYS_QUEUE_H_ which is also defined by
: > > the <sys/queue.h> system header. <sys/disk.h> uses SLIST_ENTRY
: > > on NetBSD, which doesn't exist in sys-queue.h. Therefore,
: > > include <sys/queue.h> before including sys-queue.h.
: >
: > Does this patch (which my mail client unfortunately refuses to cite...)
: > take into account that we now have TAILQ_FOREACH_SAFE which NetBSD may
: > lack?
:
: Yes, NetBSD has it:
:
: #define TAILQ_FOREACH_SAFE(var, head, field, next) \
: for ((var) = ((head)->tqh_first); \
: (var) != NULL && ((next) = TAILQ_NEXT(var, field), 1); \
: (var) = (next))
:
: Another option to this patch would be:
:
: Renaming _SYS_QUEUE_H_ to QEMU_SYS_QUEUE_H_
: at least allows to include both sys-queue.h and <sys/queue.h>.
That would be the better option, because _XXX is reserved for the
compiler/library/toolchain, not for the application...
Warner