Re: [Qemu-devel] [PATCH 1/4] Introduce strtosz() library function to convert a string to a byte count.

2010-10-12 Thread Jes Sorensen
On 10/12/10 17:52, Markus Armbruster wrote: > Still not entirely happy, but maybe we can commit it as is, and fix it > up later. No worries, I think this is the most serious review I have ever received for any piece of code, but you're finding valid points so it's good. If all of QEMU had been rev

Re: [Qemu-devel] [RFT] qemu 0.13.0-rc3

2010-10-12 Thread Juergen Lock
On Tue, Oct 12, 2010 at 05:00:34PM -0500, Anthony Liguori wrote: > On 10/12/2010 04:34 PM, Juergen Lock wrote: > > In article<4cb38c82.1090...@linux.vnet.ibm.com> you write: > > > >> After suffering from a prolonged maintainer softlockup, I'm attempting > >> to get 0.13.0 release process back

[Qemu-devel] Trace all instrucdtions in loop

2010-10-12 Thread vanson . dang
Dear All. Currently, QEMU doesn't trace all instructions in loop for example: main.c -- main() { int a = 1,b = 2,c = 3, i; int sum,sub,mul; for(i = 0; i < 3; i++) { sum = b+c; sub = b-c; mul = b*c; } r

Re: [Qemu-devel] [PATCH] ceph/rbd block driver for qemu-kvm (v6)

2010-10-12 Thread Anthony Liguori
On 10/12/2010 06:18 PM, Christian Brunner wrote: Hi Anthony, Hi Kevin, Hi Stefan, here is an updated version of the ceph/rbd block driver. It includes all the changes Yehuda made after the discussion on the list last weekend. As far as I can tell all the issues discussed are fixed. Please let u

Re: [Qemu-devel] [RFT] qemu 0.13.0-rc3

2010-10-12 Thread Anthony Liguori
On 10/12/2010 04:34 PM, Juergen Lock wrote: In article<4cb38c82.1090...@linux.vnet.ibm.com> you write: After suffering from a prolonged maintainer softlockup, I'm attempting to get 0.13.0 release process back on track. I've tagged qemu-0.13.0-rc3 in git which only carries a few changes sin

Re: [Qemu-devel] [RFT] qemu 0.13.0-rc3

2010-10-12 Thread Juergen Lock
In article <4cb38c82.1090...@linux.vnet.ibm.com> you write: >After suffering from a prolonged maintainer softlockup, I'm attempting >to get 0.13.0 release process back on track. > >I've tagged qemu-0.13.0-rc3 in git which only carries a few changes >since 0.13.0-rc1. Most notably, a series of up

[Qemu-devel] [PATCH] ceph/rbd block driver for qemu-kvm (v6)

2010-10-12 Thread Christian Brunner
Hi Anthony, Hi Kevin, Hi Stefan, here is an updated version of the ceph/rbd block driver. It includes all the changes Yehuda made after the discussion on the list last weekend. As far as I can tell all the issues discussed are fixed. Please let us know if the driver is ready for inclusion now. T

[Qemu-devel] [Bug 654913] Re: Windows XP uses 200% CPU when given 2 VCPUs

2010-10-12 Thread .:. brainsik
It turns out, that if you use the ACPI Multiprocessor, this problem goes away. If you use MPS Multiprocessor, then this problem occurs. The worst part is if you originally built the VM without ACPI, then there is *no way* to use on the ACPI HALs in Windows XP without reinstalling. There's a ton of

Re: [Qemu-devel] [PATCH 36/39] add autoconfy --with-arch= option, compatible with --sparc-cpu

2010-10-12 Thread Blue Swirl
On Tue, Oct 12, 2010 at 1:00 PM, Paolo Bonzini wrote: > Signed-off-by: Paolo Bonzini > --- >        Please test this on SPARC. :) > >  configure |   72 > + >  1 files changed, 39 insertions(+), 33 deletions(-) > > diff --git a/configure

Re: [Qemu-devel] [PATCH 17/39] introduce CPPFLAGS configure variable

2010-10-12 Thread Blue Swirl
On Tue, Oct 12, 2010 at 1:00 PM, Paolo Bonzini wrote: > Be consistent with Autoconf-generated scripts. > > Signed-off-by: Paolo Bonzini > --- >  configure |    8 >  rules.mak |    8 >  2 files changed, 12 insertions(+), 4 deletions(-) > > diff --git a/configure b/configure > ind

[Qemu-devel] where is qemu.git/contrib

2010-10-12 Thread Zhi Wang
Hi, I am trying to test the ivshmem in KVM. The shared memory without server works. But I can not find the ivshmem-server. The server pulled from http://www.gitorious.org/nahanni does not seem to work with the ivshmem implementation in qemu.git (outdated?). Could any one please kindly tell me how

Re: [Qemu-devel] [PATCH 07/39] do not pass bogus $(SRC_PATH) include paths to cc during configure

2010-10-12 Thread Blue Swirl
On Tue, Oct 12, 2010 at 1:00 PM, Paolo Bonzini wrote: > Non-existent -I paths are dropped silently by the compiler, but still > it is not polite to pass bogus options.  Configure-time tests do not > need any include files from the source path, so only include -I flags > at make time (when they're

Re: [SeaBIOS] [Qemu-devel] [RFC] Passing boot order from qemu to seabios

2010-10-12 Thread Gleb Natapov
On Tue, Oct 12, 2010 at 10:45:58AM -0700, H. Peter Anvin wrote: > On 10/12/2010 10:41 AM, Gleb Natapov wrote: > > On Tue, Oct 12, 2010 at 10:35:51AM -0700, H. Peter Anvin wrote: > >> On real hardware it is shared between BIOS and the OS, actually. > >> > > Guest OS can write in qemu CMOS too. But w

Re: [Qemu-devel] [PATCH 13/39] properly detect compiler in tests/Makefile

2010-10-12 Thread Blue Swirl
On Tue, Oct 12, 2010 at 1:00 PM, Paolo Bonzini wrote: > `make speed' only makes sense if not cross-compiling, so sha1 can > use the CC for the system that is hosting qemu.  sha1-i386 is also > wrong, since there is usually no variable for the target CC; guess > some plausible values. > > Signed-of

Re: [Qemu-devel] [PATCH 04/39] fix sparse support (?)

2010-10-12 Thread Blue Swirl
On Tue, Oct 12, 2010 at 1:00 PM, Paolo Bonzini wrote: > I didn't test with sparse, but the old code using += before a variable > was set was wrong.  Sparse support should probably be ripped out or > redone, but this at least keeps some sanity. > > Signed-off-by: Paolo Bonzini > --- >  configure |

[Qemu-devel] where is qemu.git/contrib

2010-10-12 Thread Zhi Wang
Hi, I am trying to test the ivshmem in KVM. The shared memory without server works. But I can not find the ivshmem-server. The server pulled from http://www.gitorious.org/nahanni does not seem to work with the ivshmem implementation in qemu.git (outdated?). Could any one please kindly tell me how

Re: [Qemu-devel] [PATCH 0/2] usb-ccid device (v2)

2010-10-12 Thread Anthony Liguori
On 10/12/2010 12:09 PM, Alon Levy wrote: The smart card is not being migrated. It is running on the client machine, which is not being migrated/shutdown (same as vncviewer isn't migrated). Ok, let's look at this compared to another similar use-case: USB passthrough of a webcam device that

[Qemu-devel] Re: [PATCH 2/3] Replace remaining gcc format attribute by macro GCC_FMT_ATTR (format checking)

2010-10-12 Thread Blue Swirl
On Thu, Sep 23, 2010 at 7:28 PM, Stefan Weil wrote: > Replace the remaining format attribute printf by macro > GCC_FMT_ATTR which uses gnu_printf (if supported). > > This needs additional code changes: > > * Add qemu-common.h (which defined GCC_FMT_ATTR) were needed. > > * Remove standard includes

Re: [Qemu-devel] [RFC] Can we remove special handling of standard headers (introduced for dyngen / OSX?)

2010-10-12 Thread Blue Swirl
On Mon, Oct 11, 2010 at 4:22 PM, Stefan Weil wrote: > Am 10.10.2010 00:46, schrieb Andreas Färber: >> >> Am 04.10.2010 um 21:29 schrieb Stefan Weil: >> >>> Am 25.09.2010 09:46, schrieb Blue Swirl: On Thu, Sep 23, 2010 at 8:44 PM, Stefan Weil wrote: > > Am 23.09.2010 22:33,

Re: [SeaBIOS] [Qemu-devel] [RFC] Passing boot order from qemu to seabios

2010-10-12 Thread H. Peter Anvin
On 10/12/2010 10:41 AM, Gleb Natapov wrote: > On Tue, Oct 12, 2010 at 10:35:51AM -0700, H. Peter Anvin wrote: >> On real hardware it is shared between BIOS and the OS, actually. >> > Guest OS can write in qemu CMOS too. But what is it useful for? Most of > its content is not standard AFAIK. > Thi

[Qemu-devel] Re: kvm networking todo wiki

2010-10-12 Thread Michael S. Tsirkin
On Sun, Oct 10, 2010 at 01:37:45PM +0200, Dragos Tatulea wrote: > Hi, > > > More importantly: anyone's going to work on this? > > I'd like to work on this. Might need some assistance though. > > Thanks, > Dragos BTW, as in some situations hardware might not be able satisfy requirements, a subse

Re: [SeaBIOS] [Qemu-devel] [RFC] Passing boot order from qemu to seabios

2010-10-12 Thread Gleb Natapov
On Tue, Oct 12, 2010 at 10:35:51AM -0700, H. Peter Anvin wrote: > On real hardware it is shared between BIOS and the OS, actually. > Guest OS can write in qemu CMOS too. But what is it useful for? Most of its content is not standard AFAIK. > "Gleb Natapov" wrote: > > >On Tue, Oct 12, 2010 at 09

Re: [SeaBIOS] [Qemu-devel] [RFC] Passing boot order from qemu to seabios

2010-10-12 Thread H. Peter Anvin
On real hardware it is shared between BIOS and the OS, actually. "Gleb Natapov" wrote: >On Tue, Oct 12, 2010 at 09:33:16AM -0700, H. Peter Anvin wrote: >> On 10/12/2010 01:01 AM, Gleb Natapov wrote: >> > On Mon, Oct 11, 2010 at 02:15:26PM -0700, H. Peter Anvin wrote: >> >>> I don't disagree. >>

[Qemu-devel] Re: [PATCH] Suppress warning: zero-length gnu_printf format string

2010-10-12 Thread Blue Swirl
On Mon, Oct 11, 2010 at 12:52 PM, Markus Armbruster wrote: > Warns about this line in check-qjson.c: >    QObject *obj = qobject_from_json(""); > > The obvious fix (add -Wno-format-zero-length to gcc_flags) doesn't > work, because -Wall switches it on again.  Fix by putting configured > flags last

[Qemu-devel] [Bug 659351] [NEW] QEMU uses obsolete gethostbyname and inet_aton rather than getaddrinfo

2010-10-12 Thread Benjamin Jencks
Public bug reported: In several places, including the tcp migration code, qemu uses gethostbyname and inet_aton to construct a sockaddr_in. These should be replaced by the more modern getaddrinfo for both connect and accept, along with a loop to try all returned addrinfo structs on the connecting

Re: [Qemu-devel] [PATCH 0/2] usb-ccid device (v2)

2010-10-12 Thread Alon Levy
- "Anthony Liguori" wrote: > On 10/12/2010 11:43 AM, Alon Levy wrote: > > - "Anthony Liguori" wrote: > > > > > >> On 10/12/2010 11:03 AM, Alon Levy wrote: > >> > >>> - "Anthony Liguori" wrote: > >>> > >>> > >>> > On 10/12/2010 07:58 AM, Alon Levy wrote: > >>

Re: [SeaBIOS] [Qemu-devel] [RFC] Passing boot order from qemu to seabios

2010-10-12 Thread Gleb Natapov
On Tue, Oct 12, 2010 at 09:33:16AM -0700, H. Peter Anvin wrote: > On 10/12/2010 01:01 AM, Gleb Natapov wrote: > > On Mon, Oct 11, 2010 at 02:15:26PM -0700, H. Peter Anvin wrote: > >>> I don't disagree. > >>> > >>> I think the best thing to do is to let SeaBIOS create a boot order table > >>> that

Re: [Qemu-devel] [PATCH 0/2] usb-ccid device (v2)

2010-10-12 Thread Anthony Liguori
On 10/12/2010 11:43 AM, Alon Levy wrote: - "Anthony Liguori" wrote: On 10/12/2010 11:03 AM, Alon Levy wrote: - "Anthony Liguori" wrote: On 10/12/2010 07:58 AM, Alon Levy wrote: This patch adds a new device, it is described in full in the

Re: [Qemu-devel] [PATCH 0/2] usb-ccid device (v2)

2010-10-12 Thread Alon Levy
- "Anthony Liguori" wrote: > On 10/12/2010 11:03 AM, Alon Levy wrote: > > - "Anthony Liguori" wrote: > > > > > >> On 10/12/2010 07:58 AM, Alon Levy wrote: > >> > >>> This patch adds a new device, it is described in full in the > second > >>> > >> patch > >> > >>>

[Qemu-devel] Re: [PATCH v2 6/7] qed: Read/write support

2010-10-12 Thread Anthony Liguori
On 10/12/2010 10:59 AM, Stefan Hajnoczi wrote: On Tue, Oct 12, 2010 at 05:39:48PM +0200, Kevin Wolf wrote: Am 12.10.2010 17:22, schrieb Anthony Liguori: On 10/12/2010 10:08 AM, Kevin Wolf wrote: Otherwise we might destroy data that isn't even touched by the guest request i

Re: [SeaBIOS] [Qemu-devel] [RFC] Passing boot order from qemu to seabios

2010-10-12 Thread H. Peter Anvin
On 10/12/2010 01:01 AM, Gleb Natapov wrote: > On Mon, Oct 11, 2010 at 02:15:26PM -0700, H. Peter Anvin wrote: >>> I don't disagree. >>> >>> I think the best thing to do is to let SeaBIOS create a boot order table >>> that contains descriptive information and then advertise that to QEMU. >>> >>> QE

[Qemu-devel] Re: Where's gpxe-eepro100-80862449.rom ?

2010-10-12 Thread Stefan Weil
Am 12.10.2010 14:41, schrieb Markus Armbruster: Commit db667a12 added a reference to ROM file gpxe-eepro100-80862449.rom, but no such file. Intentional? Yes. See http://lists.nongnu.org/archive/html/qemu-devel/2010-05/msg00418.html Citation from my former mail: Yes, it is missing. I did

Re: [Qemu-devel] [RFT] qemu 0.13.0-rc3

2010-10-12 Thread Stefan Weil
Am 12.10.2010 00:15, schrieb Anthony Liguori: After suffering from a prolonged maintainer softlockup, I'm attempting to get 0.13.0 release process back on track. I've tagged qemu-0.13.0-rc3 in git which only carries a few changes since 0.13.0-rc1. Most notably, a series of updates from Kevin

Re: [Qemu-devel] [PATCH 0/2] usb-ccid device (v2)

2010-10-12 Thread Anthony Liguori
On 10/12/2010 11:03 AM, Alon Levy wrote: - "Anthony Liguori" wrote: On 10/12/2010 07:58 AM, Alon Levy wrote: This patch adds a new device, it is described in full in the second patch intro and also in the documentation in docs. In brief it provides a sta

[Qemu-devel] Re: [PATCH v2 6/7] qed: Read/write support

2010-10-12 Thread Avi Kivity
On 10/12/2010 06:16 PM, Anthony Liguori wrote: It's fairly simple to add a sync to this path. It's probably worth checking the prefill/postfill for zeros and avoiding the write/sync if that's the case. That should optimize the common cases of allocating new space within a file. My intuit

Re: [Qemu-devel] [PATCH 0/2] usb-ccid device (v2)

2010-10-12 Thread Alon Levy
- "Anthony Liguori" wrote: > On 10/12/2010 07:58 AM, Alon Levy wrote: > > This patch adds a new device, it is described in full in the second > patch > > intro and also in the documentation in docs. In brief it provides a > standard > > smart card reader device. > > > > The first patch is th

[Qemu-devel] Re: [PATCH v2 6/7] qed: Read/write support

2010-10-12 Thread Stefan Hajnoczi
On Tue, Oct 12, 2010 at 05:39:48PM +0200, Kevin Wolf wrote: > Am 12.10.2010 17:22, schrieb Anthony Liguori: > > On 10/12/2010 10:08 AM, Kevin Wolf wrote: > >> Otherwise we might destroy data that isn't > >> even touched by the guest request in case of a crash. > >> > > > > The failure scenar

Re: [Qemu-devel] [PATCH 1/4] Introduce strtosz() library function to convert a string to a byte count.

2010-10-12 Thread Markus Armbruster
Still not entirely happy, but maybe we can commit it as is, and fix it up later. jes.soren...@redhat.com writes: > From: Jes Sorensen > > strtosz() returns -1 on error. It now supports human unit formats in > eg. 1.0G, with better error handling. > > This version lets strtod() do the actual pars

Re: [Qemu-devel] [RFT] qemu 0.13.0-rc3

2010-10-12 Thread Rick Vernam
I encountered two issues with 0.13.0-rc1, not sure how to tell if the patches have been applied? one is this patch: http://patchwork.ozlabs.org/patch/62420/ discussed here: http://lists.nongnu.org/archive/html/qemu-devel/2010-09/msg01114.html the other was http://lists.nongnu.org/archive/html/qe

[Qemu-devel] [Bug 659276] [NEW] Accessing Century byte via RTC

2010-10-12 Thread Ian Seyler
Public bug reported: With the RTC configure for Binary instead of BCD I am getting the incorrect result for Century. mov al, 0x0B; Set RTC to binary mode out 0x70, al in al, 0x71 bts ax, 2 mov bl, al mov al, 0x0B out 0x70

[Qemu-devel] Re: [PATCH v2 6/7] qed: Read/write support

2010-10-12 Thread Kevin Wolf
Am 12.10.2010 17:22, schrieb Anthony Liguori: > On 10/12/2010 10:08 AM, Kevin Wolf wrote: >> Otherwise we might destroy data that isn't >> even touched by the guest request in case of a crash. >> > > The failure scenarios are either that the cluster is leaked in which > case, the old versio

[Qemu-devel] Re: [PATCH v2 6/7] qed: Read/write support

2010-10-12 Thread Kevin Wolf
Am 08.10.2010 17:48, schrieb Stefan Hajnoczi: > This patch implements the read/write state machine. Operations are > fully asynchronous and multiple operations may be active at any time. > > Allocating writes lock tables to ensure metadata updates do not > interfere with each other. If two alloc

Re: [Qemu-devel] [PATCH 08/39] provide portable sizeof(long) test

2010-10-12 Thread malc
On Tue, 12 Oct 2010, Paolo Bonzini wrote: > On 10/12/2010 04:41 PM, malc wrote: > > > > > Gives wrong results on Win64. > > > > > > Then it's not HOST_LONG_BITS, it's HOST_POINTER_BITS. > > > > Not quite, [s]size_t/ptrdiff_t are 64 bits wide udner Win64, little > > to do with pointers. > > B

Re: [Qemu-devel] [PATCH 08/39] provide portable sizeof(long) test

2010-10-12 Thread Paolo Bonzini
On 10/12/2010 05:12 PM, malc wrote: Bottom line: QEMU assumes that long is large enough to hold a pointer, this assumption is wrong, and your patch is wrong (not because it miscalculates size of ABI long) but because of the assumptions current code has. Whether two wrongs makes right is an open

[Qemu-devel] Re: [PATCH v2 6/7] qed: Read/write support

2010-10-12 Thread Anthony Liguori
On 10/12/2010 10:08 AM, Kevin Wolf wrote: Otherwise we might destroy data that isn't even touched by the guest request in case of a crash. The failure scenarios are either that the cluster is leaked in which case, the old version of the data is still present or the cluster is orphaned b

Re: [Qemu-devel] qdev: Some ISA devices don't handle second instantiation gracefully

2010-10-12 Thread Anthony Liguori
On 10/12/2010 08:54 AM, Markus Armbruster wrote: I think the proper thing to do is remove all exit(1)s and propagate errors instead. exit() is good enough during startup, i.e. -device. It's wrong for hot plug; anything to be used in a hot plug path must propagate errors. We could keep

Re: [Qemu-devel] [PATCH 08/39] provide portable sizeof(long) test

2010-10-12 Thread malc
On Tue, 12 Oct 2010, Paolo Bonzini wrote: > On 10/12/2010 04:38 PM, malc wrote: > > On Tue, 12 Oct 2010, Paolo Bonzini wrote: > > > > > On 10/12/2010 03:47 PM, malc wrote: > > > > ># host long bits test > > > > > -hostlongbits="32" > > > > > -case "$cpu" in > > > > > - x86_64|alpha|ia64|spar

Re: [Qemu-devel] [PATCH 08/39] provide portable sizeof(long) test

2010-10-12 Thread Paolo Bonzini
On 10/12/2010 04:41 PM, malc wrote: > > > > Gives wrong results on Win64. > > > > Then it's not HOST_LONG_BITS, it's HOST_POINTER_BITS. > > Not quite, [s]size_t/ptrdiff_t are 64 bits wide udner Win64, little > to do with pointers. Before (on Win64): sizeof(long) == 4, HOST_LONG_BITS == 64 Af

[Qemu-devel] Re: [PATCH v2 5/7] qed: Table, L2 cache, and cluster functions

2010-10-12 Thread Kevin Wolf
Am 08.10.2010 17:48, schrieb Stefan Hajnoczi: > This patch adds code to look up data cluster offsets in the image via > the L1/L2 tables. The L2 tables are writethrough cached in memory for > performance (each read/write requires a lookup so it is essential to > cache the tables). > > With cluste

Re: [Qemu-devel] [PATCH 08/39] provide portable sizeof(long) test

2010-10-12 Thread Paolo Bonzini
On 10/12/2010 04:38 PM, malc wrote: On Tue, 12 Oct 2010, Paolo Bonzini wrote: On 10/12/2010 03:47 PM, malc wrote: # host long bits test -hostlongbits="32" -case "$cpu" in - x86_64|alpha|ia64|sparc64|ppc64|s390x) -hostlongbits=64 - ;; -esac +cat> $TMPC<< EOF +int sizeof_long_is_8[s

[Qemu-devel] Re: qdev: Some ISA devices don't handle second instantiation gracefully

2010-10-12 Thread Gerd Hoffmann
Hi, They call register_ioport_write(), which aborts via hw_error() when the port is already in use. This is okay for non-configurable parts of a board emulation, but not okay for a qdev device, unless it has no_user set. Related: when isa_init_irq() finds the requested IRQ already in use, it

[Qemu-devel] Re: [Bug 638955] Re: emulated netcards don't work with recent sunos kernel

2010-10-12 Thread Stefan Hajnoczi
On Sat, Oct 2, 2010 at 8:23 PM, daniel pecka <638...@bugs.launchpad.net> wrote: > well, i did some more investigations and here come a results .. > > this patch http://patchwork.ozlabs.org/patch/65137/raw/ solves problem > partially .. NICs are working with that but after a deeper look, > connectio

Re: [Qemu-devel] [PATCH 08/39] provide portable sizeof(long) test

2010-10-12 Thread malc
On Tue, 12 Oct 2010, Paolo Bonzini wrote: > On 10/12/2010 03:47 PM, malc wrote: > > > # host long bits test > > > -hostlongbits="32" > > > -case "$cpu" in > > > - x86_64|alpha|ia64|sparc64|ppc64|s390x) > > > -hostlongbits=64 > > > - ;; > > > -esac > > > +cat> $TMPC<< EOF > > > +int sizeo

Re: [Qemu-devel] [PATCH 00/39] Make configure command line autoconf-compatible

2010-10-12 Thread Anthony Liguori
On 10/12/2010 08:32 AM, malc wrote: On Tue, 12 Oct 2010, Paolo Bonzini wrote: On 10/12/2010 03:22 PM, malc wrote: 19 files changed, 3729 insertions(+), 456 deletions(-) I don't think the explanation above justified net plus of 3273 lines of new code. The part y

Re: [Qemu-devel] [PATCH 00/39] Make configure command line autoconf-compatible

2010-10-12 Thread Paolo Bonzini
On 10/12/2010 04:30 PM, Anthony Liguori wrote: I think there's a fair bit of value in looking like autoconf but a 39 patch series is a bit to digest at once. I'll try to do a thorough review but I'd suggest keeping patch series down to about 20 patches whenever possible and do multiple rounds as

Re: [Qemu-devel] [PATCH 23/39] deprecate --audio-card-list

2010-10-12 Thread Paolo Bonzini
On 10/12/2010 03:49 PM, malc wrote: On Tue, 12 Oct 2010, Paolo Bonzini wrote: We have config files now which provide fine-grained control. Signed-off-by: Paolo Bonzini --- configure |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/configure b/configure index 57867

[Qemu-devel] Invitación a conectarnos en LinkedIn

2010-10-12 Thread Ernesto Domato via LinkedIn
LinkedIn Ernesto Domato requested to add you as a connection on LinkedIn: -- Jiajun, Me gustaría añadirte a mi red profesional en LinkedIn. -Ernesto Accept invitation from Ernesto Domato http://www.linkedin.com/e/-kkb1ec-gf6v999o-6x/qTMmi8QEI

Re: [Qemu-devel] [PATCH 08/39] provide portable sizeof(long) test

2010-10-12 Thread Paolo Bonzini
On 10/12/2010 03:47 PM, malc wrote: # host long bits test -hostlongbits="32" -case "$cpu" in - x86_64|alpha|ia64|sparc64|ppc64|s390x) -hostlongbits=64 - ;; -esac +cat> $TMPC<< EOF +int sizeof_long_is_8[sizeof(long) == 8 ? 1 : -1]; +EOF +if compile_object; then +hostlongbits=64 +else +ho

Re: [Qemu-devel] [PATCH 23/39] deprecate --audio-card-list

2010-10-12 Thread malc
On Tue, 12 Oct 2010, Paolo Bonzini wrote: > We have config files now which provide fine-grained control. > > Signed-off-by: Paolo Bonzini > --- > configure |4 ++-- > 1 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/configure b/configure > index 5786729..efebaa9 100755 > -

Re: [Qemu-devel] [PATCH 08/39] provide portable sizeof(long) test

2010-10-12 Thread malc
On Tue, 12 Oct 2010, Paolo Bonzini wrote: > Do not hardcode the list of 64-bit CPUs. > > Signed-off-by: Paolo Bonzini > --- > configure | 14 -- > 1 files changed, 8 insertions(+), 6 deletions(-) > > diff --git a/configure b/configure > index cb76177..873e4a6 100755 > --- a/confi

Re: [Qemu-devel] [PATCH 00/39] Make configure command line autoconf-compatible

2010-10-12 Thread Paolo Bonzini
On 10/12/2010 03:22 PM, malc wrote: 19 files changed, 3729 insertions(+), 456 deletions(-) I don't think the explanation above justified net plus of 3273 lines of new code. The part you forgot to quote is: Makefile | 54 +- Makefile.dis |6 +- Makefi

[Qemu-devel] [PATCH 39/39] use host triplets for feature detection

2010-10-12 Thread Paolo Bonzini
Signed-off-by: Paolo Bonzini --- configure | 64 - 1 files changed, 21 insertions(+), 43 deletions(-) diff --git a/configure b/configure index 0340cfe..69b3406 100755 --- a/configure +++ b/configure @@ -290,49 +290,25 @@ case "$cpu" i

Re: [Qemu-devel] Re: [PATCH v2 3/7] docs: Add QED image format specification

2010-10-12 Thread Anthony Liguori
On 10/12/2010 08:16 AM, Stefan Hajnoczi wrote: Well, the protocol is currently encoded in the file name, separated by a colon. Of course, we want to get rid of that, but we still don't know what we want instead. It's completely unrelated to the backing file format, though, it's about the format

Re: [Qemu-devel] [PATCH 0/2] usb-ccid device (v2)

2010-10-12 Thread Anthony Liguori
On 10/12/2010 07:58 AM, Alon Levy wrote: This patch adds a new device, it is described in full in the second patch intro and also in the documentation in docs. In brief it provides a standard smart card reader device. The first patch is the configure change and docs. The second patch contains th

Re: [Qemu-devel] [PATCH 00/39] Make configure command line autoconf-compatible

2010-10-12 Thread malc
On Tue, 12 Oct 2010, Paolo Bonzini wrote: > Here is the patch series implementing the idea I mentioned a while ago. > > The first 14 introduce small changes to group pieces of code together > and help understanding/reviewing the next changes. Most of them are > just cosmetic changes, though some

Re: [Qemu-devel] qdev: Some ISA devices don't handle second instantiation gracefully

2010-10-12 Thread Markus Armbruster
Anthony Liguori writes: > On 10/12/2010 08:00 AM, Markus Armbruster wrote: >> Markus Armbruster writes: >> >> >>> When I try -device isa-applesmc -device isa-applesmc, I get >>> >>> WARNING: Using AppleSMC with invalid key >>> qemu: hardware error: register_ioport_read: invalid opa

[Qemu-devel] [PATCH] e1000: Handle IO Port.

2010-10-12 Thread anthony . perard
From: Anthony PERARD This patch introduces the two IOPorts on e1000, IOADDR and IODATA. The IOADDR is used to specify which register we want to access when we read or write on IODATA. It also check the RDLEN register when a packet is received, if the value is 0, the receive descriptor buffer is

[Qemu-devel] [PATCH 36/39] add autoconfy --with-arch= option, compatible with --sparc-cpu

2010-10-12 Thread Paolo Bonzini
Signed-off-by: Paolo Bonzini --- Please test this on SPARC. :) configure | 72 + 1 files changed, 39 insertions(+), 33 deletions(-) diff --git a/configure b/configure index d6f472b..99de45b 100755 --- a/configure +++ b/config

Re: [Qemu-devel] [PATCH 00/39] Make configure command line autoconf-compatible

2010-10-12 Thread malc
On Tue, 12 Oct 2010, Paolo Bonzini wrote: > On 10/12/2010 03:22 PM, malc wrote: > > > 19 files changed, 3729 insertions(+), 456 deletions(-) > > > > I don't think the explanation above justified net plus of 3273 lines of new > > code. > > The part you forgot to quote is: Yes, failed to notice

Re: [Qemu-devel] qdev: Some ISA devices don't handle second instantiation gracefully

2010-10-12 Thread Anthony Liguori
On 10/12/2010 08:00 AM, Markus Armbruster wrote: Markus Armbruster writes: When I try -device isa-applesmc -device isa-applesmc, I get WARNING: Using AppleSMC with invalid key qemu: hardware error: register_ioport_read: invalid opaque [...] and a core dump. I know nothing

Re: [Qemu-devel] Re: [PATCH v2 3/7] docs: Add QED image format specification

2010-10-12 Thread Stefan Hajnoczi
On Tue, Oct 12, 2010 at 10:07:53AM +0200, Kevin Wolf wrote: > Am 11.10.2010 19:14, schrieb Anthony Liguori: > > On 10/11/2010 11:18 AM, Anthony Liguori wrote: > >> On 10/11/2010 10:46 AM, Stefan Hajnoczi wrote: > >>> On Mon, Oct 11, 2010 at 05:39:01PM +0200, Avi Kivity wrote: > On 10/11/2010

[Qemu-devel] [PATCH 33/39] rename source_path to srcdir

2010-10-12 Thread Paolo Bonzini
Signed-off-by: Paolo Bonzini --- configure | 48 1 files changed, 24 insertions(+), 24 deletions(-) diff --git a/configure b/configure index 432bd3b..8ed9736 100755 --- a/configure +++ b/configure @@ -182,8 +182,8 @@ for opt do ;; CC=*|--c

[Qemu-devel] [PATCH 30/39] add autoconfy alias --with-headers for --kerneldir

2010-10-12 Thread Paolo Bonzini
This name is already used by glibc. Signed-off-by: Paolo Bonzini --- configure |5 +++-- 1 files changed, 3 insertions(+), 2 deletions(-) diff --git a/configure b/configure index e1eeda2..39a7119 100755 --- a/configure +++ b/configure @@ -726,7 +726,7 @@ for opt do ;; --disable-blobs)

[Qemu-devel] [PATCH 38/39] provide a more gnuish default sysroot

2010-10-12 Thread Paolo Bonzini
the default sys-root already hads "gnemul" in it, so I do not feel ashamed to follow the GNU conventions completely. Signed-off-by: Paolo Bonzini --- configure | 20 ++-- create_config |2 +- 2 files changed, 19 insertions(+), 3 deletions(-) diff --git a/configure b/co

[Qemu-devel] [PATCH 28/39] rename interp_prefix to sysroot

2010-10-12 Thread Paolo Bonzini
Signed-off-by: Paolo Bonzini --- bsd-user/main.c| 10 +- configure | 14 +++--- darwin-user/machload.c | 12 ++-- darwin-user/main.c |6 +++--- darwin-user/syscall.c |2 +- linux-user/main.c | 10 +- 6 files changed, 2

[Qemu-devel] [PATCH 25/39] add autoconfy alias --enable-block-drivers for --block-drv-whitelist

2010-10-12 Thread Paolo Bonzini
Signed-off-by: Paolo Bonzini --- configure |6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/configure b/configure index b144b21..86b1835 100755 --- a/configure +++ b/configure @@ -574,7 +574,7 @@ for opt do ;; --enable-audio-drivers=*|--audio-drv-list=*) audio_d

[Qemu-devel] [PATCH 27/39] add autoconfy alias --with-sysroot for --interp-prefix

2010-10-12 Thread Paolo Bonzini
Signed-off-by: Paolo Bonzini --- configure |8 +--- 1 files changed, 5 insertions(+), 3 deletions(-) diff --git a/configure b/configure index d677b53..581b91e 100755 --- a/configure +++ b/configure @@ -514,7 +514,7 @@ for opt do ;; --prefix=*) prefix="$optarg" ;; - --interp-pre

[Qemu-devel] [PATCH 31/39] add autoconfy alias --srcdir= for --source-path

2010-10-12 Thread Paolo Bonzini
Signed-off-by: Paolo Bonzini --- configure |7 --- 1 files changed, 4 insertions(+), 3 deletions(-) diff --git a/configure b/configure index 39a7119..44b5be9 100755 --- a/configure +++ b/configure @@ -182,7 +182,7 @@ for opt do ;; CC=*|--cc=*) CC="$optarg" ;; - --source-path=*)

[Qemu-devel] Re: qdev: Some ISA devices don't handle second instantiation gracefully (was: isa-applesmc doesn't handle second instantiation gracefully)

2010-10-12 Thread Alexander Graf
On 12.10.2010, at 15:00, Markus Armbruster wrote: > Markus Armbruster writes: > >> When I try -device isa-applesmc -device isa-applesmc, I get >> >>WARNING: Using AppleSMC with invalid key >>qemu: hardware error: register_ioport_read: invalid opaque >>[...] >> >> and a core dump.

[Qemu-devel] [PATCH 37/39] make more options "standard"

2010-10-12 Thread Paolo Bonzini
Signed-off-by: Paolo Bonzini --- configure | 16 1 files changed, 8 insertions(+), 8 deletions(-) diff --git a/configure b/configure index 99de45b..27fc67a 100755 --- a/configure +++ b/configure @@ -806,6 +806,14 @@ EOF echo "Standard options:" echo " --help

[Qemu-devel] [PATCH 24/39] add autoconfy alias --enable-audio-drivers alias for --audio-drv-list

2010-10-12 Thread Paolo Bonzini
Signed-off-by: Paolo Bonzini --- configure |5 +++-- 1 files changed, 3 insertions(+), 2 deletions(-) diff --git a/configure b/configure index efebaa9..b144b21 100755 --- a/configure +++ b/configure @@ -572,7 +572,7 @@ for opt do ;; --audio-card-list=*) audio_card_list=`echo "$optarg"

[Qemu-devel] [PATCH 23/39] deprecate --audio-card-list

2010-10-12 Thread Paolo Bonzini
We have config files now which provide fine-grained control. Signed-off-by: Paolo Bonzini --- configure |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/configure b/configure index 5786729..efebaa9 100755 --- a/configure +++ b/configure @@ -856,8 +856,6 @@ echo " --en

[Qemu-devel] [PATCH 32/39] rename SRC_PATH to srcdir

2010-10-12 Thread Paolo Bonzini
Signed-off-by: Paolo Bonzini --- Makefile | 52 ++-- Makefile.dis |6 ++-- Makefile.hw|8 +++--- Makefile.objs |4 +- Makefile.target| 32 +- Ma

[Qemu-devel] [PATCH 19/39] add CPP variable

2010-10-12 Thread Paolo Bonzini
There is already a CPP makefile variable, treat it consistently with every other tool. Signed-off-by: Paolo Bonzini --- configure | 10 +- 1 files changed, 9 insertions(+), 1 deletions(-) diff --git a/configure b/configure index 1ea56f7..8273f84 100755 --- a/configure +++ b/configure

[Qemu-devel] [PATCH 15/39] let --host-cc slide into oblivion

2010-10-12 Thread Paolo Bonzini
It is not really needed anymore since dyngen's death, and the previous patches removed the last occurrences. Signed-off-by: Paolo Bonzini --- configure |8 +--- create_config |3 --- 2 files changed, 1 insertions(+), 10 deletions(-) diff --git a/configure b/configure index 068b4

[Qemu-devel] [PATCH 34/39] add autoconfy --host= option deprecating --cross-prefix

2010-10-12 Thread Paolo Bonzini
Unlike --cross-prefix, if you specify a tool by environment variables --host will not affect it. This allows SDL_CONFIG=/usr/.../bin/sdl-config ./configure --host=i686-pc-mingw32 to work. Signed-off-by: Paolo Bonzini --- configure | 41 - 1 files ch

[Qemu-devel] [PATCH 16/39] introduce CFLAGS= and LDFLAGS= configure command-line options

2010-10-12 Thread Paolo Bonzini
Also, run checks with -O2. Some headers may provide different code based on __OPTIMIZE__, so it's good to be consistent. Signed-off-by: Paolo Bonzini --- configure | 31 +-- 1 files changed, 21 insertions(+), 10 deletions(-) diff --git a/configure b/configure inde

[Qemu-devel] [PATCH 08/39] provide portable sizeof(long) test

2010-10-12 Thread Paolo Bonzini
Do not hardcode the list of 64-bit CPUs. Signed-off-by: Paolo Bonzini --- configure | 14 -- 1 files changed, 8 insertions(+), 6 deletions(-) diff --git a/configure b/configure index cb76177..873e4a6 100755 --- a/configure +++ b/configure @@ -1088,12 +1088,14 @@ esac fi # host

[Qemu-devel] [PATCH 22/39] make trace options use autoconfy names

2010-10-12 Thread Paolo Bonzini
These are not in any release, so I am just renaming them. Signed-off-by: Paolo Bonzini --- configure |8 1 files changed, 4 insertions(+), 4 deletions(-) diff --git a/configure b/configure index 76626a4..5786729 100755 --- a/configure +++ b/configure @@ -540,9 +540,9 @@ for opt do

[Qemu-devel] [PATCH 29/39] add autoconfy alias --enable-targets for --target-list

2010-10-12 Thread Paolo Bonzini
Signed-off-by: Paolo Bonzini --- configure |5 +++-- 1 files changed, 3 insertions(+), 2 deletions(-) diff --git a/configure b/configure index b439bc2..e1eeda2 100755 --- a/configure +++ b/configure @@ -538,7 +538,7 @@ for opt do ;; --cpu=*) ;; - --target-list=*) target_list="$opta

[Qemu-devel] [PATCH 14/39] remove HOST_CC mention from roms/{sea, vga}bios/config.mak

2010-10-12 Thread Paolo Bonzini
Not used in the submodules. Signed-off-by: Paolo Bonzini --- configure |1 - 1 files changed, 0 insertions(+), 1 deletions(-) diff --git a/configure b/configure index e059525..068b40a 100755 --- a/configure +++ b/configure @@ -3129,7 +3129,6 @@ for rom in seabios vgabios ; do echo "CPP

[Qemu-devel] [PATCH 10/39] do not default to non-prefixed pkg-config when cross compiling

2010-10-12 Thread Paolo Bonzini
This can still be requested with PKG_CONFIG=/path/to/pkg-config. Just do not use it as a default, and print a warning. Signed-off-by: Paolo Bonzini --- configure |5 +++-- 1 files changed, 3 insertions(+), 2 deletions(-) diff --git a/configure b/configure index ac11d4e..e072dde 100755 --- a

[Qemu-devel] [PATCH 21/39] add autoconfy aliases --with-* for audio library options

2010-10-12 Thread Paolo Bonzini
Signed-off-by: Paolo Bonzini --- configure | 17 +++-- 1 files changed, 11 insertions(+), 6 deletions(-) diff --git a/configure b/configure index 547f8e9..76626a4 100755 --- a/configure +++ b/configure @@ -564,11 +564,11 @@ for opt do ;; --enable-sdl) sdl="yes" ;; - --fmo

[Qemu-devel] [PATCH 26/39] add libtooly alias --enable-static for --static

2010-10-12 Thread Paolo Bonzini
Signed-off-by: Paolo Bonzini --- configure |5 +++-- 1 files changed, 3 insertions(+), 2 deletions(-) diff --git a/configure b/configure index 86b1835..d677b53 100755 --- a/configure +++ b/configure @@ -546,7 +546,7 @@ for opt do ;; --enable-gprof) gprof="yes" ;; - --static) + --e

[Qemu-devel] [PATCH 11/39] reorganize sdl-config tests

2010-10-12 Thread Paolo Bonzini
This also allows overriding it with SDL_CONFIG, and warning in suspicious cross-compilation scenarios. Signed-off-by: Paolo Bonzini --- configure | 30 +- 1 files changed, 17 insertions(+), 13 deletions(-) diff --git a/configure b/configure index e072dde..f61c5b8 1

[Qemu-devel] [PATCH 09/39] fix spelling of $pkg_config, move default together with other cross tools

2010-10-12 Thread Paolo Bonzini
Signed-off-by: Paolo Bonzini --- configure | 34 +- 1 files changed, 17 insertions(+), 17 deletions(-) diff --git a/configure b/configure index 873e4a6..ac11d4e 100755 --- a/configure +++ b/configure @@ -216,6 +216,7 @@ objcopy="${cross_prefix}${OBJCOPY-objcopy}

[Qemu-devel] [PATCH 20/39] add autoconfy aliases MAKE=/INSTALL= for --make and --install

2010-10-12 Thread Paolo Bonzini
Signed-off-by: Paolo Bonzini --- configure | 14 -- 1 files changed, 8 insertions(+), 6 deletions(-) diff --git a/configure b/configure index 8273f84..547f8e9 100755 --- a/configure +++ b/configure @@ -526,9 +526,9 @@ for opt do ;; --host-cc=*) ;; - --make=*) make="$optar

[Qemu-devel] [PATCH 17/39] introduce CPPFLAGS configure variable

2010-10-12 Thread Paolo Bonzini
Be consistent with Autoconf-generated scripts. Signed-off-by: Paolo Bonzini --- configure |8 rules.mak |8 2 files changed, 12 insertions(+), 4 deletions(-) diff --git a/configure b/configure index 2a2f64e..7f2e2a4 100755 --- a/configure +++ b/configure @@ -189,6 +189

[Qemu-devel] [PATCH 18/39] add autoconfy alias CC= for --cc

2010-10-12 Thread Paolo Bonzini
Signed-off-by: Paolo Bonzini --- configure |7 --- 1 files changed, 4 insertions(+), 3 deletions(-) diff --git a/configure b/configure index 7f2e2a4..1ea56f7 100755 --- a/configure +++ b/configure @@ -180,7 +180,7 @@ for opt do case "$opt" in --cross-prefix=*) cross_prefix="$optarg

[Qemu-devel] [PATCH 07/39] do not pass bogus $(SRC_PATH) include paths to cc during configure

2010-10-12 Thread Paolo Bonzini
Non-existent -I paths are dropped silently by the compiler, but still it is not polite to pass bogus options. Configure-time tests do not need any include files from the source path, so only include -I flags at make time (when they're properly expanded). Signed-off-by: Paolo Bonzini --- configu

  1   2   >