Re: [Qemu-devel] [PATCH][RFC] Fix bugs in the ATAPI cdrom driver

2007-08-19 Thread Thiemo Seufer
Brandon Philips wrote: > On 17:42 Fri 17 Aug 2007, Matthew Kent wrote: > > On Fri, 2007-17-08 at 16:43 -0700, Brandon Philips wrote: > > > The new libata-eh in the Linux kernel is throwing a fit over the QEMU > > > cdrom device for two reasons: > > > > Nice thanks, was suffering from this issue as

[Qemu-devel] qemu vl.c

2007-08-19 Thread Thiemo Seufer
CVSROOT:/sources/qemu Module name:qemu Changes by: Thiemo Seufer 07/08/19 22:09:40 Modified files: . : vl.c Log message: Add -clock option, by Luca Tettamanti. CVSWeb URLs: http://cvs.savannah.gnu.org/viewcvs/qemu/vl.c?cvsroot=qemu&r1=1.324&r2=1

[Qemu-devel] qemu vl.c vl.h

2007-08-19 Thread Thiemo Seufer
CVSROOT:/sources/qemu Module name:qemu Changes by: Thiemo Seufer 07/08/19 21:56:03 Modified files: . : vl.c vl.h Log message: Rework alarm timer infrastrucure, by Luca Tettamanti. CVSWeb URLs: http://cvs.savannah.gnu.org/viewcvs/qemu/vl.c?cvsroo

[Qemu-devel] qemu/hw ide.c

2007-08-19 Thread Thiemo Seufer
CVSROOT:/sources/qemu Module name:qemu Changes by: Thiemo Seufer 07/08/19 21:46:53 Modified files: hw : ide.c Log message: Fix bugs in the ATAPI cdrom driver, by Brandon Philips. CVSWeb URLs: http://cvs.savannah.gnu.org/viewcvs/qemu/hw/ide.c?cvsr

[Qemu-devel] qemu/darwin-user main.c

2007-08-19 Thread Thiemo Seufer
CVSROOT:/sources/qemu Module name:qemu Changes by: Thiemo Seufer 07/08/19 21:43:54 Modified files: darwin-user: main.c Log message: Darwin-user: Compile fix for ppc targets, by Pierre d'Herbemont. CVSWeb URLs: http://cvs.savannah.gnu.org/viewcvs/qemu/dar

[Qemu-devel] Sparc guest

2007-08-19 Thread Nigel Horne
On the CVS version, networking has stopped working again with Sparc guests running Debian Linux. It had been working fine for a few weeks, but the latest version is broken again, with the device not being found. -Nigel

[Qemu-devel] Re: [kvm-devel] [PATCH 0/4] Rework alarm timer infrastrucure - take2

2007-08-19 Thread Avi Kivity
Luca wrote: > On 8/19/07, Luca Tettamanti <[EMAIL PROTECTED]> wrote: > >> +static uint64_t qemu_next_deadline(void) { >> +uint64_t nearest_delta_us = ULLONG_MAX; >> +uint64_t vmdelta_us; >> > > Hum, I introduced a bug here... those vars should be signed. > > On the overhead introduc

Re: [kvm-devel] [Qemu-devel] Re: [PATCH 0/4] Rework alarm timer infrastrucure - take2

2007-08-19 Thread Avi Kivity
Jamie Lokier wrote: > Paul Brook wrote: > >>> Yes, good thinking, but this should only be done if it actually impacts >>> something. Reducing overhead from 0.1% to 0.05% is not worthwhile if it >>> introduces extra complexity. >>> >> If the overhead is that small, why are we touching thi

RE: [kvm-devel] [Qemu-devel] Re: [PATCH 0/4] Rework alarmtimer infrastrucure - take2

2007-08-19 Thread Dor Laor
>Paul Brook wrote: >> > Yes, good thinking, but this should only be done if it actually >impacts >> > something. Reducing overhead from 0.1% to 0.05% is not worthwhile >if it >> > introduces extra complexity. >> >> If the overhead is that small, why are we touching this code in the >first >> place

Re: [Qemu-devel] sparc kqemu?

2007-08-19 Thread Blue Swirl
On 8/16/07, Jonathan Kalbfeld <[EMAIL PROTECTED]> wrote: > Is this on the horizon? Is there any interest in it? For Sparc32 this could be possible, the page table structures are not unlike i386 ones. Given the death of Sparc32 distros I don't think there would be much interest. On Sparc64 there

Re: [Qemu-devel] Re: PATCH, RFC: Generic DMA framework

2007-08-19 Thread Blue Swirl
In 8/16/07, malc <[EMAIL PROTECTED]> wrote: > Very long time ago i changed the ISA DMA API to address some of the > critique that Fabrice expressed, i can't remember offhand if that > included removal of explicit position passing or not (the patch is on > some off-line HDD so it's not easy to check

Re: [kvm-devel] [Qemu-devel] Re: [PATCH 0/4] Rework alarm timer infrastrucure - take2

2007-08-19 Thread Jamie Lokier
Paul Brook wrote: > > Yes, good thinking, but this should only be done if it actually impacts > > something. Reducing overhead from 0.1% to 0.05% is not worthwhile if it > > introduces extra complexity. > > If the overhead is that small, why are we touching this code in the first > place? Insig

[Qemu-devel] Re: [kvm-devel] [PATCH 0/4] Rework alarm timer infrastrucure - take2

2007-08-19 Thread Luca
On 8/19/07, Luca Tettamanti <[EMAIL PROTECTED]> wrote: > +static uint64_t qemu_next_deadline(void) { > +uint64_t nearest_delta_us = ULLONG_MAX; > +uint64_t vmdelta_us; Hum, I introduced a bug here... those vars should be signed. On the overhead introduced: how do you measure it? Luca

RE: [kvm-devel] [Qemu-devel] Re: [PATCH 0/4] Rework alarm timer infrastrucure - take2

2007-08-19 Thread Dor Laor
>>> Yes, good thinking, but this should only be done if it actually >impacts >>> something. Reducing overhead from 0.1% to 0.05% is not worthwhile if >it >>> introduces extra complexity. >>> >> >> >> If the overhead is that small, why are we touching this code in the >first >> place? >> > >Accurac

Re: [kvm-devel] [Qemu-devel] Re: [PATCH 0/4] Rework alarm timer infrastrucure - take2

2007-08-19 Thread Avi Kivity
Paul Brook wrote: Yes, good thinking, but this should only be done if it actually impacts something. Reducing overhead from 0.1% to 0.05% is not worthwhile if it introduces extra complexity. If the overhead is that small, why are we touching this code in the first place? Accuracy i

Re: [kvm-devel] [Qemu-devel] Re: [PATCH 0/4] Rework alarm timer infrastrucure - take2

2007-08-19 Thread Paul Brook
> Yes, good thinking, but this should only be done if it actually impacts > something. Reducing overhead from 0.1% to 0.05% is not worthwhile if it > introduces extra complexity. If the overhead is that small, why are we touching this code in the first place? Paul

Re: [kvm-devel] [Qemu-devel] Re: [PATCH 0/4] Rework alarm timer infrastrucure - take2

2007-08-19 Thread Avi Kivity
Jamie Lokier wrote: Avi Kivity wrote: In this case the dyn-tick minimum res will be 1msec. I believe it should work ok since this is the case without any dyn-tick. Actually minimum resolution depends on host HZ setting, but - yes - essentially you have the same behaviour of

Re: [kvm-devel] [Qemu-devel] Making qemu images executable (and store command line arguments in them =P)

2007-08-19 Thread Christian Brunschen
On 19 Aug 2007, at 12:08, Avi Kivity wrote: Markus Hitter wrote: Am 19.08.2007 um 12:35 schrieb Avi Kivity: - this deviates from standard Unix (and Windows) practice. Arguably the OS X way is superior, but qemu is perhaps not the best vehicle to introduce it to other OSes. NeXTstep i

Re: [Qemu-devel] Re: [kvm-devel] [PATCH 0/4] Rework alarm timer infrastrucure - take2

2007-08-19 Thread Jamie Lokier
Avi Kivity wrote: > >>>In this case the dyn-tick minimum res will be 1msec. I believe it should > >>>work ok since this is the case without any dyn-tick. > >>> > >>Actually minimum resolution depends on host HZ setting, but - yes - > >>essentially you have the same behaviour of the "unix" tim

[Qemu-devel] CASIMIR - last e-mail about it :)

2007-08-19 Thread GUERRAZ Francois
Hi Qemu people. After my last mail about CASIMIR, I had answers of people complaining that it was not it English and they can't read my article... Well, the software is still not in English, but there are some improvements :) ยง There is a translation of the previous article at at this address

Re: [Qemu-devel] [PATCH] Set apic instance number for savevm

2007-08-19 Thread Avi Kivity
Avi Kivity wrote: This allows save/restore (and live migration on kvm) of smp guests. Index: hw/apic.c === RCS file: /sources/qemu/qemu/hw/apic.c,v retrieving revision 1.13 diff -u -r1.13 apic.c --- hw/apic.c 3 Apr 2007 16:38:34 -

Re: [kvm-devel] [Qemu-devel] Making qemu images executable (and store command line arguments in them =P)

2007-08-19 Thread Avi Kivity
Markus Hitter wrote: Am 19.08.2007 um 12:35 schrieb Avi Kivity: - this deviates from standard Unix (and Windows) practice. Arguably the OS X way is superior, but qemu is perhaps not the best vehicle to introduce it to other OSes. NeXTstep introduced this into the Unix world and GN

Re: [kvm-devel] [Qemu-devel] Making qemu images executable (and store command line arguments in them =P)

2007-08-19 Thread Markus Hitter
Am 19.08.2007 um 12:35 schrieb Avi Kivity: - this deviates from standard Unix (and Windows) practice. Arguably the OS X way is superior, but qemu is perhaps not the best vehicle to introduce it to other OSes. NeXTstep introduced this into the Unix world and GNUstep uses it today. So, th

Re: [kvm-devel] [Qemu-devel] Making qemu images executable (and store command line arguments in them =P)

2007-08-19 Thread Avi Kivity
Christian Brunschen wrote: On 17 Aug 2007, at 13:40, Avi Kivity wrote: It's not easy to use: if you move the image, you need to move the file. I'd like to have exactly one entity to worry about when using a virtual machine. As I wrote previously, there is already such a thing on every modern

[Qemu-devel] Re: [kvm-devel] [PATCH 0/4] Rework alarm timer infrastrucure - take2

2007-08-19 Thread Avi Kivity
Anthony Liguori wrote: I think this is a really nice and important patch set. Just a couple things: On Sun, 2007-08-19 at 00:02 +0200, Luca Tettamanti wrote: In this case the dyn-tick minimum res will be 1msec. I believe it should work ok since this is the case without any dyn-tick.

[Qemu-devel] Re: [kvm-devel] [PATCH/RFC 4/4] Add support for dynamic ticks.

2007-08-19 Thread Avi Kivity
Matthew Kent wrote: If DYNAMIC_TICKS is defined qemu does not attepmt to generate SIGALRM at a constant rate. Rather, the system timer is set to generate SIGALRM only when it is needed. DYNAMIC_TICKS reduces the number of SIGALRMs sent to idle dynamic-ticked guests. Original patch from Dan Kenigs

[Qemu-devel] RE: [kvm-devel] [PATCH 0/4] Rework alarm timer infrastrucure -take2

2007-08-19 Thread Dor Laor
>I think this is a really nice and important patch set. Just a couple >things: > >On Sun, 2007-08-19 at 00:02 +0200, Luca Tettamanti wrote: > >> > In this case the dyn-tick minimum res will be 1msec. I believe it >should >> > work ok since this is the case without any dyn-tick. >> >> Actually mini