All of lore.kernel.org
 help / color / mirror / Atom feed
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

      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 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.