From: malc <av1474@comtv.ru>
To: Juan Quintela <quintela@trasno.org>
Cc: qemu-devel@nongnu.org
Subject: [Qemu-devel] Re: Main loop
Date: Mon, 28 Sep 2009 14:19:01 +0400 (MSD) [thread overview]
Message-ID: <Pine.LNX.4.64.0909281416520.2434@linmac.oyster.ru> (raw)
In-Reply-To: <m3pr9bidy9.fsf@neno.mitica>
On Mon, 28 Sep 2009, Juan Quintela wrote:
> malc <av1474@comtv.ru> wrote:
> > On Mon, 28 Sep 2009, Juan Quintela wrote:
> >
> >> malc <av1474@comtv.ru> wrote:
> >> > At http://repo.or.cz/w/qemu/malc.git?a=shortlog;h=refs/heads/mtloop you
> >> > can find the branch which refactors main execution loop somewhat, effects
> >> > include:
> >> >
> >> > a. Host alarm timers are gone
> >> > b. IO thread is replaced (now Windows is supported too)
> >> >
> >> > I have no means of testing the KVM/Xen bits (both are likely to be broken
> >> > by this), and since testing was only done on Linux/X86-64[1],PPC and
> >> > Windows/i386 chances are good that something might be not so great in
> >> > BSD/Solairs/MacOS X lands.
> >>
> >> No cookie:
> >> It don't work in kvm mode.
>
> > Didn't i say that i have no means of testing it? Even x86 box yields:
>
> I read it as that you wanted people to test it :p
I must have expressed myself poorly, what was meant:
KVM/Xen are quite lilely broken (truth)
I have no means of testing _and_ fixing it
Testing _without_ those is nice, fixing those is better still
>
> > ~$ uname -a
> > Linux laptop 2.6.31 #7 SMP PREEMPT Sun Sep 13 10:59:44 MSD 2009 x86_64 AMD
> > Athlon(tm)64 X2 Dual Core Processor 3800+ AuthenticAMD GNU/Linux
> > ~$ egrep '^flags.*(vmx|svm)' /proc/cpuinfo
> > ~$
> >
> > Help is needed with that.
>
> on the "I like side" of the patch:
> - timers (clocks or whatever) are gone!!!!!
> - differences beteween io_thread/no io_thread also gone
> - pthread wrappers are gone
> on the "I don't like"
> - calling "minor refactoring" to a single patch that remove all the
> clocks, remove io thread, remove pthread wrappers, and some other
> things in one go
grumble, grubmble
--
mailto:av1474@comtv.ru
prev parent reply other threads:[~2009-09-28 10:19 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-09-26 23:55 [Qemu-devel] Main loop malc
2009-09-27 0:49 ` Anthony Liguori
2009-09-27 10:55 ` malc
2009-09-27 14:05 ` Anthony Liguori
2009-09-27 14:39 ` malc
2009-09-28 13:57 ` Glauber Costa
2009-09-28 18:50 ` malc
2009-09-28 19:35 ` Anthony Liguori
2009-09-28 21:21 ` Glauber Costa
2009-09-28 23:57 ` malc
2009-09-27 14:31 ` malc
2009-09-27 14:23 ` Blue Swirl
2009-09-27 14:35 ` malc
2009-09-27 17:43 ` malc
[not found] ` <m3fxa7jug0.fsf@neno.mitica>
2009-09-28 9:42 ` [Qemu-devel] " malc
[not found] ` <m3pr9bidy9.fsf@neno.mitica>
2009-09-28 10:19 ` malc [this message]
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=Pine.LNX.4.64.0909281416520.2434@linmac.oyster.ru \
--to=av1474@comtv.ru \
--cc=qemu-devel@nongnu.org \
--cc=quintela@trasno.org \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).