From: "Todd T. Fries" <qemu-devel@email.fries.net>
To: qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [RFC, PATCH] Host AIO support detection, OpenBSD host support
Date: Fri, 15 Aug 2008 15:29:43 -0500 [thread overview]
Message-ID: <20080815202943.GA11001@fries.net> (raw)
In-Reply-To: <f43fc5580808151305j7079e945n2f919f8f8409901a@mail.gmail.com>
Actually, the problem is that some driver lists (NetBSD in particular)
were changed to
list="<a> <b> <c>"
while others (OpenBSD in particular) were left with
list="a b c"
fun!
--
Todd Fries .. todd@fries.net
_____________________________________________
| \ 1.636.410.0632 (voice)
| Free Daemon Consulting, LLC \ 1.405.227.9094 (voice)
| http://FreeDaemonConsulting.com \ 1.866.792.3418 (FAX)
| "..in support of free software solutions." \ 250797 (FWD)
| \
\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\
37E7 D3EB 74D0 8D66 A68D B866 0326 204E 3F42 004A
http://todd.fries.net/pgp.txt
Penned by Blue Swirl on 20080815 23:05.40, we have:
| On 8/14/08, malc <av1474@comtv.ru> wrote:
| > On Wed, 13 Aug 2008, Blue Swirl wrote:
| >
| >
| > > On 8/12/08, Anthony Liguori <anthony@codemonkey.ws> wrote:
| > >
| > > > Blue Swirl wrote:
| > > >
| > > >
| > > > > Here are two patches, loosely based on patches in OpenBSD ports system
| > > > > that enable compiling Qemu. The first patch makes AIO use conditional
| > > > > on host AIO support. The second one adjusts various include files etc.
| > > > >
| > > > > I'd like to commit both of these, is that OK?
| > > > >
| > > > >
| > > > >
| > > >
| > > > The first patch is fine but the second patch has some bogus things in
| > it.
| > > >
| > >
| > > Thanks for the review. I removed the unneeded parts, made a separate
| > > patch for fixing OSS selection, and added a configure option for AIO
| > > as requested by Samuel Thibault.
| > >
| >
| > Thanks for the OSS bits, i have committed slightly modified patch that
| > takes all possible drivers into consideration.
|
| Great, but there is a bug in your commit, *) case seems to be checked
| before oss|sdl... case gets into play so the error is still present:
|
| Error: Unknown driver 'oss' selected
| Possible drivers are: oss sdl esd
|
next prev parent reply other threads:[~2008-08-15 20:31 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-08-12 18:45 [Qemu-devel] [RFC, PATCH] Host AIO support detection, OpenBSD host support Blue Swirl
2008-08-12 19:46 ` Anthony Liguori
2008-08-12 19:56 ` Todd T. Fries
2008-08-12 20:08 ` Anthony Liguori
2008-08-13 16:53 ` Blue Swirl
2008-08-13 16:48 ` Blue Swirl
2008-08-13 17:50 ` Anthony Liguori
2008-08-13 21:48 ` malc
2008-08-15 20:05 ` Blue Swirl
2008-08-15 20:29 ` Todd T. Fries [this message]
2008-08-16 16:26 ` malc
2008-08-16 16:36 ` Blue Swirl
2008-08-17 20:44 ` malc
2008-08-16 9:33 ` Blue Swirl
2008-08-12 21:07 ` Samuel Thibault
2008-08-12 21:17 ` Samuel Thibault
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20080815202943.GA11001@fries.net \
--to=qemu-devel@email.fries.net \
--cc=qemu-devel@nongnu.org \
--cc=todd@fries.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.