Re: Proper support for PCI-based option rom loading (was Re: [Qemu-devel] Re: qdev property bug?)

2009-12-14 Thread Kevin O'Connor
On Mon, Dec 14, 2009 at 08:37:44PM -0600, Anthony Liguori wrote: > Okay, I think I've figured out how this is supposed to work. With these > two patches to SeaBIOS and the patch to qemu, I can run: I'm not sure why "Do not guard qemu shadow ram work around in CONFIG_OPTIONROMS_DEPLOYED" patch i

Re: [Qemu-devel] Re: qdev property bug?

2009-12-14 Thread Kevin O'Connor
On Mon, Dec 14, 2009 at 03:05:22PM -0600, Anthony Liguori wrote: > I believe the way this works with real BIOSes is that the rom is > initially loaded somewhere outside of the 1M region and it's init is > executed. It's then the rom's job to execute it's initialization. > Afterwards, the bi

Re: Proper support for PCI-based option rom loading (was Re: [Qemu-devel] Re: qdev property bug?)

2009-12-14 Thread Kevin O'Connor
On Mon, Dec 14, 2009 at 08:37:44PM -0600, Anthony Liguori wrote: > Okay, I think I've figured out how this is supposed to work. With these > two patches to SeaBIOS and the patch to qemu, I can run: Yes - I think this is the best way to fix this. Space optimizations can be most easily done by a

Re: [Qemu-devel] target-alpha: An approach to fp insn qualifiers

2009-12-14 Thread Richard Henderson
On 12/14/2009 04:31 PM, Richard Henderson wrote: On 12/14/2009 12:11 PM, Laurent Desnogues wrote: I'll take a closer look at your patch tomorrow. For the record, I believe this finishes what I had in mind for the exception handling there in op_handler.c. Hmph. One more patch for correctness

[Qemu-devel] [ANNOUNCE] SeaBIOS mailing list

2009-12-14 Thread Kevin O'Connor
There is now a mailing list for discussions on SeaBIOS. To join the mailing list, please visit: http://www.seabios.org/mailman/listinfo/seabios Mails can be sent to seab...@seabios.org. To reduce spam, non-member posts to the list will be moderated. -Kevin

Proper support for PCI-based option rom loading (was Re: [Qemu-devel] Re: qdev property bug?)

2009-12-14 Thread Anthony Liguori
Michael S. Tsirkin wrote: On Mon, Dec 14, 2009 at 02:44:34PM -0600, Anthony Liguori wrote: Michael S. Tsirkin wrote: Or, we could have a very small ROM, that loads more actual code from card or from qemu directly when it is run. It's not as simple as it sounds but it's possib

Re: [Qemu-devel] [FOR 0.12][PATCH] target-i386: Fix evaluation of DR7 register

2009-12-14 Thread TeLeMan
raise_exception() in op_helper.c should not be used outside op_helper.c because of tcg. This patch uses raise_exception2() instead of raise_exception(). target-i386-fix-raise_exception-for-hw-breakpoints.patch Description: Binary data

Re: [Qemu-devel] vmware vga + kvm interaction

2009-12-14 Thread Dave Airlie
On Tue, Dec 15, 2009 at 12:28 AM, Anthony Liguori wrote: > Dave Airlie wrote: >> >> I actually reinvented at least one of the patches locally and it >> didn't seem to help, >> but I'll try and take a closer look today, >> > > http://repo.or.cz/w/qemu/aliguori-queue.git vmware-vga-for-dave > > Is t

[Qemu-devel] [PATCH] Fix TEXTREL in user targets when compiled with --enable-user-pie

2009-12-14 Thread Kirill A. Shutemov
We really need compile _all_ sources for user target with -fpie when use --enable-user-pie. It's regression introduced by commit add16157d72454. Please, do not break it any more. Signed-off-by: Kirill A. Shutemov Cc: Blue Swirl --- Makefile.user |4 ++-- configure |1 + 2 files cha

Re: [Qemu-devel] target-alpha: An approach to fp insn qualifiers

2009-12-14 Thread Richard Henderson
On 12/14/2009 12:11 PM, Laurent Desnogues wrote: I'll take a closer look at your patch tomorrow. For the record, I believe this finishes what I had in mind for the exception handling there in op_handler.c. r~ commit ce6c2abc1d5d437dde980b4addc7da0f0f5de252 Author: Richard Henderson Date:

Re: [Qemu-devel] target-alpha: An approach to fp insn qualifiers

2009-12-14 Thread Richard Henderson
On 12/14/2009 12:11 PM, Laurent Desnogues wrote: I don't really like passing parts of opcodes to helpers, but as you say that prevents explosion of helpers. OTOH you could do lazy calls to helpers that set rounding modes with my approach of separating them from computation. I don't particularl

Re: [Qemu-devel] Re: qdev property bug?

2009-12-14 Thread Sebastian Herbszt
Anthony Liguori wrote: Sebastian Herbszt wrote: Anthony Liguori wrote: Michael S. Tsirkin wrote: On Mon, Dec 14, 2009 at 02:43:38PM -0600, Anthony Liguori wrote: Because it can be selected by the user via the menu and because it can be selected at runtime via the boot_set monitor command.

[Qemu-devel] FreeBSD qemu-devel 0.12.0-rc2 port update available for testing

2009-12-14 Thread Juergen Lock
Hi! I updated my git head snapshot qemu-devel port update to 0.12.0-rc2 today (that was just announced: http://lists.gnu.org/archive/html/qemu-devel/2009-12/msg01514.html - the Subject says rc1 but in fact its rc2) so people can test that version on FreeBSD more easily: http://peo

[Qemu-devel] [PATCH:] Fix a make -j race

2009-12-14 Thread Juergen Lock
Make libuser.a depend on $(GENERATED_HEADERS) too so make -j won't start building it before the headers exist. (There may be more bugs like this but at least this makes (g)make -j4 started from scratch on a quadcore now always complete here again.) Signed-off-by: Juergen Lock --- a/Makefile +++

[Qemu-devel] Re: [ANNOUNCE] qemu-kvm-0.12.0-rc2 released

2009-12-14 Thread Dustin Kirkland
On Mon, Dec 14, 2009 at 12:33 PM, Avi Kivity wrote: > qemu-kvm-0.12.0-rc2 is now available.  This release is is based on the > upstream qemu 0.12.0-rc2, plus kvm-specific enhancements.  Please see the > original qemu 0.12.0-rc2 release announcement for details. > > This release can be used with th

Re: [Qemu-devel] Re: qdev property bug?

2009-12-14 Thread Anthony Liguori
Sebastian Herbszt wrote: Anthony Liguori wrote: Michael S. Tsirkin wrote: On Mon, Dec 14, 2009 at 02:43:38PM -0600, Anthony Liguori wrote: Because it can be selected by the user via the menu and because it can be selected at runtime via the boot_set monitor command. Yes, but it's not

Re: [Qemu-devel] cpuid problem in upstream qemu with kvm

2009-12-14 Thread Anthony Liguori
Michael S. Tsirkin wrote: On Mon, Dec 14, 2009 at 02:54:49PM -0600, Anthony Liguori wrote: Michael S. Tsirkin wrote: On Mon, Dec 14, 2009 at 02:18:33PM -0600, Anthony Liguori wrote: Michael S. Tsirkin wrote: This might help 32 bit guests, but not guests with 64

Re: [Qemu-devel] Re: qdev property bug?

2009-12-14 Thread Sebastian Herbszt
Anthony Liguori wrote: Michael S. Tsirkin wrote: On Mon, Dec 14, 2009 at 02:43:38PM -0600, Anthony Liguori wrote: Because it can be selected by the user via the menu and because it can be selected at runtime via the boot_set monitor command. Yes, but it's not like we have nowhere to

Re: [Qemu-devel] [PATCH 09/11] Cocoa: Shutdown when window is closed

2009-12-14 Thread Andreas Färber
Hi, Am 14.12.2009 um 08:04 schrieb >: On Dec 13, 2009, at 04:55, ext Andreas Färber wrote: The application is not very useful once the guest window is closed. QEMU is not a document-based application; terminating it automatically saves the user another action and resembles SDL behavior.

[Qemu-devel] [PATCH v2 09/11] Cocoa: Shutdown when window is closed

2009-12-14 Thread Andreas Färber
The application is not very useful once the guest window is closed. QEMU is not a document-based application; terminating it automatically saves the user another action and resembles SDL behavior. v2: - Use delegate method, suggested by Juha Riihimäki. Signed-off-by: Andreas Färber Cc: Juha Riih

Re: [Qemu-devel] Re: [PATCH RFC] virtio: add features qdev property

2009-12-14 Thread Gerd Hoffmann
On 12/14/09 22:14, Michael S. Tsirkin wrote: On Mon, Dec 14, 2009 at 10:12:14PM +0100, Gerd Hoffmann wrote: On 12/14/09 21:43, Michael S. Tsirkin wrote: What do we put in e.g. 0.11 compat? Any features we enable there might not be supported by host. compat properties as usual? Sorry, I still

Re: [Qemu-devel] Re: [PATCH RFC] virtio: add features qdev property

2009-12-14 Thread Michael S. Tsirkin
On Mon, Dec 14, 2009 at 10:12:14PM +0100, Gerd Hoffmann wrote: > On 12/14/09 21:43, Michael S. Tsirkin wrote: >> What do we put in e.g. 0.11 compat? Any features we enable >> there might not be supported by host. > > compat properties as usual? > > Sorry, I still fail to see your problem. > > You'l

Re: [Qemu-devel] cpuid problem in upstream qemu with kvm

2009-12-14 Thread Michael S. Tsirkin
On Mon, Dec 14, 2009 at 02:54:49PM -0600, Anthony Liguori wrote: > Michael S. Tsirkin wrote: >> On Mon, Dec 14, 2009 at 02:18:33PM -0600, Anthony Liguori wrote: >> >>> Michael S. Tsirkin wrote: >>> This might help 32 bit guests, but not guests with 64 bit kernel and 32 bit userspa

Re: [Qemu-devel] Re: [PATCH RFC] virtio: add features qdev property

2009-12-14 Thread Gerd Hoffmann
On 12/14/09 21:43, Michael S. Tsirkin wrote: What do we put in e.g. 0.11 compat? Any features we enable there might not be supported by host. compat properties as usual? Sorry, I still fail to see your problem. You'll have a 'disable' bitmap. Fill it via 'features='. Or using separate prope

Re: [Qemu-devel] Re: qdev property bug?

2009-12-14 Thread Anthony Liguori
Michael S. Tsirkin wrote: On Mon, Dec 14, 2009 at 02:43:38PM -0600, Anthony Liguori wrote: Because it can be selected by the user via the menu and because it can be selected at runtime via the boot_set monitor command. Yes, but it's not like we have nowhere to store them. We could sha

Re: [Qemu-devel] Re: qdev property bug?

2009-12-14 Thread Anthony Liguori
Sebastian Herbszt wrote: Some DOS-based software might need the UMA (RAM UMB). I don't think that's the workload to optimize for in terms of the default settings. Certainly, it's important to allow such workloads to disable all unnecessary rom loading but I don't think it's justification fo

Re: [Qemu-devel] cpuid problem in upstream qemu with kvm

2009-12-14 Thread Anthony Liguori
Michael S. Tsirkin wrote: On Mon, Dec 14, 2009 at 02:18:33PM -0600, Anthony Liguori wrote: Michael S. Tsirkin wrote: This might help 32 bit guests, but not guests with 64 bit kernel and 32 bit userspace (my case) because all 64 bit CPUs advertise syscall bit in cpuid. Thus 64 bit guests

Re: [Qemu-devel] Re: qdev property bug?

2009-12-14 Thread Michael S. Tsirkin
On Mon, Dec 14, 2009 at 02:44:34PM -0600, Anthony Liguori wrote: > Michael S. Tsirkin wrote: >> Or, we could have a very small ROM, that loads >> more actual code from card or from qemu directly >> when it is run. >> > > It's not as simple as it sounds but it's possible, in theory at least. > >

[Qemu-devel] [PATCH 5/5] monitor: Catch printing to non-existent monitor

2009-12-14 Thread Luiz Capitulino
The monitor_vprintf() function now touches the 'mon' pointer before calling monitor_puts(), this causes block migration to segfault as its functions call monitor_printf() with a NULL 'mon'. To fix the problem this commit moves the 'mon' NULL check from monitor_puts() to monitor_vprintf(). This ca

[Qemu-devel] [PATCH 4/5] monitor: Avoid readline functions in QMP

2009-12-14 Thread Luiz Capitulino
The monitor_read_command() function is readline specific and should only be used when readline is available. Signed-off-by: Luiz Capitulino --- monitor.c |3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) diff --git a/monitor.c b/monitor.c index 920ccff..b518cc4 100644 --- a/monitor.c

[Qemu-devel] [PATCH 3/5] monitor: do_balloon(): Check for errors

2009-12-14 Thread Luiz Capitulino
do_balloon() should check for ballooning availability as do_info_balloon() does. Noted by Daniel P. Berrange . Signed-off-by: Luiz Capitulino --- monitor.c | 35 ++- 1 files changed, 26 insertions(+), 9 deletions(-) diff --git a/monitor.c b/monitor.c index d50

[Qemu-devel] [PATCH 2/5] monitor: Use 'device' in eject

2009-12-14 Thread Luiz Capitulino
Monitor's eject command uses 'filename' for the device name argument, but 'device' is a better name. Signed-off-by: Luiz Capitulino --- monitor.c |2 +- qemu-monitor.hx |2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/monitor.c b/monitor.c index d97d529..d5041d

[Qemu-devel] [PATCH 1/5] QDict: Fix size update

2009-12-14 Thread Luiz Capitulino
Key replacement should not update the dictionary's size. This commit also adds a test for the bug. Signed-off-by: Luiz Capitulino --- check-qdict.c |2 ++ qdict.c |3 +-- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/check-qdict.c b/check-qdict.c index c37d448..f2

[Qemu-devel] [FOR 0.12 0/5]: Misc QMP related fixes

2009-12-14 Thread Luiz Capitulino
This small series contains some QMP related fixes. Thanks.

Re: [Qemu-devel] Re: qdev property bug?

2009-12-14 Thread Michael S. Tsirkin
On Mon, Dec 14, 2009 at 02:43:38PM -0600, Anthony Liguori wrote: > Michael S. Tsirkin wrote: >> On Mon, Dec 14, 2009 at 09:32:30PM +0100, Gerd Hoffmann wrote: >> >>> On 12/14/09 21:20, Michael S. Tsirkin wrote: >>> > So far, it's not clear to me that anyone has demonstrated how this is

Re: [Qemu-devel] Re: qdev property bug?

2009-12-14 Thread Sebastian Herbszt
Michael S. Tsirkin wrote: On Mon, Dec 14, 2009 at 09:32:30PM +0100, Gerd Hoffmann wrote: On 12/14/09 21:20, Michael S. Tsirkin wrote: So far, it's not clear to me that anyone has demonstrated how this is harmful for people that don't want pxe booting. Assuming we fix the bug about rom loading,

Re: [Qemu-devel] Re: [PATCH RFC] virtio: add features qdev property

2009-12-14 Thread Michael S. Tsirkin
On Mon, Dec 14, 2009 at 09:40:28PM +0100, Gerd Hoffmann wrote: > On 12/14/09 20:17, Michael S. Tsirkin wrote: >> On Mon, Dec 14, 2009 at 06:18:29PM +0100, Gerd Hoffmann wrote: >>> On 12/14/09 17:23, Michael S. Tsirkin wrote: On Mon, Dec 14, 2009 at 04:01:33PM +0100, Gerd Hoffmann wrote: S

Re: [Qemu-devel] Re: qdev property bug?

2009-12-14 Thread Anthony Liguori
Michael S. Tsirkin wrote: Or, we could have a very small ROM, that loads more actual code from card or from qemu directly when it is run. It's not as simple as it sounds but it's possible, in theory at least. But I think the question really is, what problem are we trying to solve? Regards,

Re: [Qemu-devel] Re: qdev property bug?

2009-12-14 Thread Anthony Liguori
Michael S. Tsirkin wrote: On Mon, Dec 14, 2009 at 09:32:30PM +0100, Gerd Hoffmann wrote: On 12/14/09 21:20, Michael S. Tsirkin wrote: So far, it's not clear to me that anyone has demonstrated how this is harmful for people that don't want pxe booting. Assuming we fix the bug about rom

[Qemu-devel] Re: [PATCH RFC] virtio: add features qdev property

2009-12-14 Thread Gerd Hoffmann
-device 'virtio-blk-pci,disable=blk-identify|ring-indirect' The driver will just do 'vdev->host_features&= ~disable'. cheers, Gerd Is there an example of an existing property that is like this? No. A new property type is needed for this. cheers, Gerd

Re: [Qemu-devel] Re: [PATCH RFC] virtio: add features qdev property

2009-12-14 Thread Gerd Hoffmann
On 12/14/09 20:17, Michael S. Tsirkin wrote: On Mon, Dec 14, 2009 at 06:18:29PM +0100, Gerd Hoffmann wrote: On 12/14/09 17:23, Michael S. Tsirkin wrote: On Mon, Dec 14, 2009 at 04:01:33PM +0100, Gerd Hoffmann wrote: So how do you do this? Assume we have -disable_hw_csum. We want new machine typ

Re: [Qemu-devel] Re: qdev property bug?

2009-12-14 Thread Sebastian Herbszt
Gerd Hoffmann wrote: On 12/14/09 21:20, Michael S. Tsirkin wrote: So far, it's not clear to me that anyone has demonstrated how this is harmful for people that don't want pxe booting. Assuming we fix the bug about rom loading, then there's really no impact to users. Fix (v2) is on the list al

Re: [Qemu-devel] Re: qdev property bug?

2009-12-14 Thread Michael S. Tsirkin
On Mon, Dec 14, 2009 at 10:34:28PM +0200, Michael S. Tsirkin wrote: > On Mon, Dec 14, 2009 at 09:32:30PM +0100, Gerd Hoffmann wrote: > > On 12/14/09 21:20, Michael S. Tsirkin wrote: > >>> So far, it's not clear to me that anyone has demonstrated how this is > >>> harmful for people that don't want

Re: [Qemu-devel] Re: qdev property bug?

2009-12-14 Thread Michael S. Tsirkin
On Mon, Dec 14, 2009 at 09:32:30PM +0100, Gerd Hoffmann wrote: > On 12/14/09 21:20, Michael S. Tsirkin wrote: >>> So far, it's not clear to me that anyone has demonstrated how this is >>> harmful for people that don't want pxe booting. Assuming we fix the bug >>> about rom loading, then there's re

Re: [Qemu-devel] cpuid problem in upstream qemu with kvm

2009-12-14 Thread Michael S. Tsirkin
On Mon, Dec 14, 2009 at 02:18:33PM -0600, Anthony Liguori wrote: > Michael S. Tsirkin wrote: >> This might help 32 bit guests, but not guests with 64 bit >> kernel and 32 bit userspace (my case) because all 64 bit >> CPUs advertise syscall bit in cpuid. Thus 64 bit guests >> do not seem to even bot

Re: [Qemu-devel] Re: qdev property bug?

2009-12-14 Thread Gerd Hoffmann
On 12/14/09 21:20, Michael S. Tsirkin wrote: So far, it's not clear to me that anyone has demonstrated how this is harmful for people that don't want pxe booting. Assuming we fix the bug about rom loading, then there's really no impact to users. Fix (v2) is on the list already. PXE booting m

Re: [Qemu-devel] Re: qdev property bug?

2009-12-14 Thread Michael S. Tsirkin
On Mon, Dec 14, 2009 at 01:59:47PM -0600, Anthony Liguori wrote: > Michael S. Tsirkin wrote: >> At some level this is wrong. ROM presence is a hardware feature and you >> definitely need to select your hardware when you create a VM. For real >> hardware, cards with PXE are sometimes more expensive

[Qemu-devel] [PATCH] vnc: hextile: do not generate ForegroundSpecified and SubrectsColoured tiles

2009-12-14 Thread Anthony Liguori
This violates the RFB specification (section 6.6.4). It happens to work with most clients but it's still wrong. Reported-by: Yaniv Kaul Signed-off-by: Anthony Liguori --- vnchextile.h |4 +--- 1 files changed, 1 insertions(+), 3 deletions(-) diff --git a/vnchextile.h b/vnchextile.h index

Re: [Qemu-devel] cpuid problem in upstream qemu with kvm

2009-12-14 Thread Anthony Liguori
Michael S. Tsirkin wrote: This might help 32 bit guests, but not guests with 64 bit kernel and 32 bit userspace (my case) because all 64 bit CPUs advertise syscall bit in cpuid. Thus 64 bit guests do not seem to even bother checking this bit: AMD + 64 bit -> syscall. Okay, I don't see a grea

Re: [Qemu-devel] cpuid problem in upstream qemu with kvm

2009-12-14 Thread Michael S. Tsirkin
On Mon, Dec 14, 2009 at 10:08:38PM +0200, Gleb Natapov wrote: > On Mon, Dec 14, 2009 at 02:02:03PM -0600, Anthony Liguori wrote: > > Gleb Natapov wrote: > > >>I thought KVM emulates the syscall instruction? I swear I've seen > > >>patches for that. > > >> > > >It is. Starting from 2.6.32. > > > >

Re: [Qemu-devel] cpuid problem in upstream qemu with kvm

2009-12-14 Thread Michael S. Tsirkin
On Mon, Dec 14, 2009 at 02:02:03PM -0600, Anthony Liguori wrote: > Gleb Natapov wrote: >>> I thought KVM emulates the syscall instruction? I swear I've seen >>> patches for that. >>> >>> >> It is. Starting from 2.6.32. >> > > Okay, so this is a performance vs. migration compatibility issue

Re: [Qemu-devel] target-alpha: An approach to fp insn qualifiers

2009-12-14 Thread Laurent Desnogues
On Mon, Dec 14, 2009 at 7:02 PM, Richard Henderson wrote: > One of the other folks here (I'm sorry, I've forgotten who as I misplaced > the mail) passed me a preliminary patch to tackle the missing fp rounding > mode support.  That patch added helpers to change the rounding mode, and > injected th

Re: [Qemu-devel] cpuid problem in upstream qemu with kvm

2009-12-14 Thread Gleb Natapov
On Mon, Dec 14, 2009 at 02:02:03PM -0600, Anthony Liguori wrote: > Gleb Natapov wrote: > >>I thought KVM emulates the syscall instruction? I swear I've seen > >>patches for that. > >> > >It is. Starting from 2.6.32. > > Okay, so this is a performance vs. migration compatibility issue then? > > B

Re: [Qemu-devel] cpuid problem in upstream qemu with kvm

2009-12-14 Thread Anthony Liguori
Gleb Natapov wrote: I thought KVM emulates the syscall instruction? I swear I've seen patches for that. It is. Starting from 2.6.32. Okay, so this is a performance vs. migration compatibility issue then? BTW, couldn't we just not advertise syscall in cpuid? That should fix it too

Re: [Qemu-devel] Re: qdev property bug?

2009-12-14 Thread Sebastian Herbszt
Anthony Liguori wrote: Sebastian Herbszt wrote: Anthony Liguori wrote: Michael S. Tsirkin wrote: Further, we should error out when device is added. Doing this during boot is way too late, management won't be able to understand such errors and won't be able to recover. I don't quite under

Re: [Qemu-devel] cpuid problem in upstream qemu with kvm

2009-12-14 Thread Gleb Natapov
On Mon, Dec 14, 2009 at 01:57:29PM -0600, Anthony Liguori wrote: > Michael S. Tsirkin wrote: > >On Mon, Dec 14, 2009 at 01:44:22PM -0600, Anthony Liguori wrote: > >>Michael S. Tsirkin wrote: > >>>Hi! > >>>I have a linux guest where init crashes during boot with invalid opcode > >>>error > >>>with

Re: [Qemu-devel] Re: qdev property bug?

2009-12-14 Thread Anthony Liguori
Michael S. Tsirkin wrote: At some level this is wrong. ROM presence is a hardware feature and you definitely need to select your hardware when you create a VM. For real hardware, cards with PXE are sometimes more expensive than without as they need ROM memory. People do select hardware based on

Re: [Qemu-devel] cpuid problem in upstream qemu with kvm

2009-12-14 Thread Anthony Liguori
Michael S. Tsirkin wrote: On Mon, Dec 14, 2009 at 01:44:22PM -0600, Anthony Liguori wrote: Michael S. Tsirkin wrote: Hi! I have a linux guest where init crashes during boot with invalid opcode error with kernel 2.6.31, but works fine with kernel 2.6.32. It also works fine even with 2.6.

Re: [Qemu-devel] cpuid problem in upstream qemu with kvm

2009-12-14 Thread Michael S. Tsirkin
On Mon, Dec 14, 2009 at 01:44:22PM -0600, Anthony Liguori wrote: > Michael S. Tsirkin wrote: >> Hi! >> I have a linux guest where init crashes during boot with invalid opcode error >> with kernel 2.6.31, but works fine with kernel 2.6.32. >> It also works fine even with 2.6.31 with qemu-kvm. >> >>

Re: [Qemu-devel] Re: qdev property bug?

2009-12-14 Thread Michael S. Tsirkin
On Mon, Dec 14, 2009 at 01:33:50PM -0600, Anthony Liguori wrote: > Sebastian Herbszt wrote: >> Anthony Liguori wrote: >>> Michael S. Tsirkin wrote: Further, we should error out when device is added. Doing this during boot is way too late, management won't be able to understand s

Re: [Qemu-devel] cpuid problem in upstream qemu with kvm

2009-12-14 Thread Anthony Liguori
Michael S. Tsirkin wrote: Hi! I have a linux guest where init crashes during boot with invalid opcode error with kernel 2.6.31, but works fine with kernel 2.6.32. It also works fine even with 2.6.31 with qemu-kvm. With Gleb's help, we figured out that the opcode in question is syscall, which is

Re: [Qemu-devel] Re: qdev property bug?

2009-12-14 Thread Sebastian Herbszt
Michael S. Tsirkin wrote: On Mon, Dec 14, 2009 at 08:12:48PM +0100, Sebastian Herbszt wrote: Anthony Liguori wrote: Michael S. Tsirkin wrote: Further, we should error out when device is added. Doing this during boot is way too late, management won't be able to understand such errors and won't

[Qemu-devel] cpuid problem in upstream qemu with kvm

2009-12-14 Thread Michael S. Tsirkin
Hi! I have a linux guest where init crashes during boot with invalid opcode error with kernel 2.6.31, but works fine with kernel 2.6.32. It also works fine even with 2.6.31 with qemu-kvm. With Gleb's help, we figured out that the opcode in question is syscall, which is unsupported by hardware on k

Re: [Qemu-devel] Re: qdev property bug?

2009-12-14 Thread Anthony Liguori
Sebastian Herbszt wrote: Anthony Liguori wrote: Michael S. Tsirkin wrote: Further, we should error out when device is added. Doing this during boot is way too late, management won't be able to understand such errors and won't be able to recover. I don't quite understand this. In 0.11, we

[Qemu-devel] Re: [PATCH RFC] virtio: add features qdev property

2009-12-14 Thread Michael S. Tsirkin
On Mon, Dec 14, 2009 at 12:37:29PM +0100, Gerd Hoffmann wrote: > On 12/14/09 12:10, Michael S. Tsirkin wrote: >> On Mon, Dec 14, 2009 at 11:24:41AM +0100, Gerd Hoffmann wrote: >> for block: >> if (strcmp(s->serial_str, "0")) >> features |= 1<< VIRTIO_BLK_F_IDENTIFY; >> >> if (bd

Re: [Qemu-devel] Re: qdev property bug?

2009-12-14 Thread Michael S. Tsirkin
On Mon, Dec 14, 2009 at 08:12:48PM +0100, Sebastian Herbszt wrote: > Anthony Liguori wrote: >> Michael S. Tsirkin wrote: >>> >>> Further, we should error out when device is added. >>> Doing this during boot is way too late, management >>> won't be able to understand such errors and >>> won't be abl

Re: [Qemu-devel] Re: [PATCH RFC] virtio: add features qdev property

2009-12-14 Thread Michael S. Tsirkin
On Mon, Dec 14, 2009 at 06:18:29PM +0100, Gerd Hoffmann wrote: > On 12/14/09 17:23, Michael S. Tsirkin wrote: >> On Mon, Dec 14, 2009 at 04:01:33PM +0100, Gerd Hoffmann wrote: >> So how do you do this? >> Assume we have -disable_hw_csum. >> We want new machine type to have it off, right? >> But now

Re: [Qemu-devel] Re: qdev property bug?

2009-12-14 Thread Sebastian Herbszt
Anthony Liguori wrote: Michael S. Tsirkin wrote: Further, we should error out when device is added. Doing this during boot is way too late, management won't be able to understand such errors and won't be able to recover. I don't quite understand this. In 0.11, we never loaded option roms

Re: [Qemu-devel] RFC: exit on incoming exec migrate failure

2009-12-14 Thread Andrew Farmer
On 11 Dec 2009, at 13:19, Daniel P. Berrange wrote: > On Wed, Dec 09, 2009 at 01:10:18PM -0800, Andrew Farmer wrote: >> Right now, if an incoming migrate through exec fails, the qemu process >> will end up chewing CPU indefinitely - it looks like it closes the >> migration FD but doesn't remove i

[Qemu-devel] target-alpha: An approach to fp insn qualifiers

2009-12-14 Thread Richard Henderson
One of the other folks here (I'm sorry, I've forgotten who as I misplaced the mail) passed me a preliminary patch to tackle the missing fp rounding mode support. That patch added helpers to change the rounding mode, and injected them before and after every fp insn that forced a rounding mode.

Re: [Qemu-devel] Re: Spice project is now open

2009-12-14 Thread Mark McLoughlin
On Mon, 2009-12-14 at 15:10 +, Daniel P. Berrange wrote: > The model I had in mind was for the proxy to define a VNC extension that > allows the client to query what 'desktops' are available and request > switching between them at any time. The list of desktop would of course > be authorized p

Re: [Qemu-devel] [ANNOUNCE][Call-For-Testing] Release 0.12.0-rc1 of QEMU

2009-12-14 Thread Anthony Liguori
Avi Kivity wrote: On 12/14/2009 04:06 PM, Anthony Liguori wrote: The QEMU team is pleased to announce the availability of the 0.12.0-rc2 release. This is the second release candidate for the 0.12.0 release. This release is not intended for production use. I'm missing the tag for this release?

Re: [Qemu-devel] Re: [PATCH RFC] virtio: add features qdev property

2009-12-14 Thread Gerd Hoffmann
On 12/14/09 17:23, Michael S. Tsirkin wrote: On Mon, Dec 14, 2009 at 04:01:33PM +0100, Gerd Hoffmann wrote: So how do you do this? Assume we have -disable_hw_csum. We want new machine type to have it off, right? But now you run qemu on host which does not support hw_csum. With your suggestion it

[Qemu-devel] Re: -serial stdio broken

2009-12-14 Thread Gerd Hoffmann
It looks like vl.c was a poor place to do the mux choice. I just wonder why mux option can't be automatically enabled for stdio in qemu-char.c:qemu_chr_parse_compat() or somewhere nearby. The mux driver grabs the 'Ctrl-a' hotkey, so I wouldn't enable that unconditionally because the chardev isn

Re: [Qemu-devel] Re: qdev property bug?

2009-12-14 Thread Glauber Costa
On Mon, Dec 14, 2009 at 08:22:12AM -0600, Anthony Liguori wrote: > Michael S. Tsirkin wrote: > >Well I am pretty sure that I used virtio + e1000 with 0.11 > >and apparently I can't now. > >So it does look like a regression to me ... > > That's what I said, we should make sure that we stop loading

Re: [Qemu-devel] Re: qdev property bug?

2009-12-14 Thread Glauber Costa
On Mon, Dec 14, 2009 at 04:11:43PM +0200, Michael S. Tsirkin wrote: > On Mon, Dec 14, 2009 at 08:11:59AM -0600, Anthony Liguori wrote: > > Alexander Graf wrote: > >> Michael S. Tsirkin wrote: > >> > >>> On Mon, Dec 14, 2009 at 12:55:28PM +0100, Alexander Graf wrote: > >>> > Am 14.12.

Re: [Qemu-devel] Re: qdev property bug?

2009-12-14 Thread Glauber Costa
On Mon, Dec 14, 2009 at 04:01:43PM +0200, Michael S. Tsirkin wrote: > On Mon, Dec 14, 2009 at 02:35:31PM +0100, Alexander Graf wrote: > > Michael S. Tsirkin wrote: > > > On Mon, Dec 14, 2009 at 12:55:28PM +0100, Alexander Graf wrote: > > > > > >> Am 14.12.2009 um 11:59 schrieb "Michael S. Tsirki

Re: [Qemu-devel] [ANNOUNCE][Call-For-Testing] Release 0.12.0-rc1 of QEMU

2009-12-14 Thread Avi Kivity
On 12/14/2009 04:06 PM, Anthony Liguori wrote: The QEMU team is pleased to announce the availability of the 0.12.0-rc2 release. This is the second release candidate for the 0.12.0 release. This release is not intended for production use. I'm missing the tag for this release? -- error compilin

Re: [Qemu-devel] [ANNOUNCE][Call-For-Testing] Release 0.12.0-rc1 of QEMU

2009-12-14 Thread Blue Swirl
On Mon, Dec 14, 2009 at 4:20 PM, Anthony Liguori wrote: > Artyom Tarasenko wrote: >> >> 2009/12/14 Anthony Liguori : >> >>> >>> The current plan is to release 0.12.0 on Friday, Dec 18th.  Depending on >>> the >>> severity of bug fixes we get between now and then, we may do another >>> release >>>

[Qemu-devel] Re: -serial stdio broken

2009-12-14 Thread Blue Swirl
On Mon, Dec 14, 2009 at 10:55 AM, Gerd Hoffmann wrote: > On 12/13/09 10:38, Blue Swirl wrote: >> >> On Sun, Dec 13, 2009 at 8:24 AM, Blue Swirl  wrote: >>> >>> I guess e1c09175bc00dd8dfb2ad1b26e1858dcdc109b59 or >>> 998bbd74b9d813b14a3a3b5009a5d5a48c7dce51 broke -serial stdio for all >>> targets:

Re: [Qemu-devel] Re: [PATCH RFC] virtio: add features qdev property

2009-12-14 Thread Michael S. Tsirkin
On Mon, Dec 14, 2009 at 04:01:33PM +0100, Gerd Hoffmann wrote: > On 12/14/09 14:59, Michael S. Tsirkin wrote: >> It's not an easy quesiton. >> If we do it as disable bits, then we get incompatible >> machines when running on different hosts. > > In case that one host supports feature which the othe

Re: [Qemu-devel] Re: qdev property bug?

2009-12-14 Thread Michael S. Tsirkin
On Mon, Dec 14, 2009 at 08:25:53AM -0600, Anthony Liguori wrote: > Michael S. Tsirkin wrote: >> >> Further, we should error out when device is added. >> Doing this during boot is way too late, management >> won't be able to understand such errors and >> won't be able to recover. >> > > I don't q

[Qemu-devel] Re: [PATCH] Seabios: Fix PkgLength calculation for the SSDT.

2009-12-14 Thread Magnus Christensson
On 12/13/2009 04:14 PM, Kevin O'Connor wrote: - Forwarded message from Gleb Natapov - From: Gleb Natapov To: Kevin O'Connor Date: Sun, 13 Dec 2009 15:18:08 +0200 Subject: Re: [...@virtutech.com: [coreboot] [PATCH] Seabios: Fix PkgLength calculation for the SSDT.] On Thu, Dec 10

[Qemu-devel] Re: Seabios: PCI interrupt routing question

2009-12-14 Thread Magnus Christensson
On 12/13/2009 04:12 PM, Kevin O'Connor wrote: - Forwarded message from Gleb Natapov - From: Gleb Natapov To: Kevin O'Connor Date: Sun, 13 Dec 2009 17:07:48 +0200 Subject: Re: [...@virtutech.com: [coreboot] Seabios: PCI interrupt routing question] On Thu, Dec 10, 2009 at 08:55:1

Re: [Qemu-devel] [ANNOUNCE][Call-For-Testing] Release 0.12.0-rc1 of QEMU

2009-12-14 Thread Anthony Liguori
Artyom Tarasenko wrote: 2009/12/14 Anthony Liguori : The current plan is to release 0.12.0 on Friday, Dec 18th. Depending on the severity of bug fixes we get between now and then, we may do another release candidate in the middle of the week. I see that the carry flag fix for sparc is

Re: [Qemu-devel] Re: Spice project is now open

2009-12-14 Thread Anthony Liguori
Avi Kivity wrote: You can still implement this with SCM_RIGHTS. Authenticate, select guest, drop tls, pass fd to qemu, authenticate, hack hack hack, drop tls, pass fd back to proxy, goto 10. Here's how I'd envision this working. Start qemu with: qemu -vnc proxy:/path/to/unix/domain/socket

Re: [Qemu-devel] Qemu terminating with SIGABRT

2009-12-14 Thread David S. Ahern
I'm using a variant of the KVM source from RHEL5 plus a few cherry-picked patches. Host OS is RHEL 5.3. The servers are using E5540 or E5504 processors. The host OS is running from a small USB key, and there is no place to write a core file. Other accommodations have to be made to get it. Hopefully

Re: [Qemu-devel] [ANNOUNCE][Call-For-Testing] Release 0.12.0-rc1 of QEMU

2009-12-14 Thread Artyom Tarasenko
2009/12/14 Anthony Liguori : > The current plan is to release 0.12.0 on Friday, Dec 18th.  Depending on the > severity of bug fixes we get between now and then, we may do another release > candidate in the middle of the week. I see that the carry flag fix for sparc is included. Are you planning to

Re: [Qemu-devel] Qemu terminating with SIGABRT

2009-12-14 Thread Luiz Capitulino
On Sat, 12 Dec 2009 11:09:38 -0700 "David S. Ahern" wrote: > Thanks for the responses. I had forgotten that SIGABRT==abort() which > means I have to get the core file to get to the root cause. To date the > only information I have is a shell exit status of 134 which from the > bash man pages mean

Re: [Qemu-devel] Re: Spice project is now open

2009-12-14 Thread Avi Kivity
On 12/14/2009 05:10 PM, Daniel P. Berrange wrote: The model I had in mind was for the proxy to define a VNC extension that allows the client to query what 'desktops' are available and request switching between them at any time. The list of desktop would of course be authorized per client, and st

Re: [Qemu-devel] [PATCH] A different way to ask for readonly drive

2009-12-14 Thread Stefan Weil
Naphtali Sprei schrieb: > Hi, > After feedback from Red Hat guys, I've decided to slightly modify the > approach to drive's readonly. > The new approach also addresses the silent fall-back to open the drives' file > as read-only when read-write fails > (permission denied) that causes unexpected b

Re: [Qemu-devel] Re: Spice project is now open

2009-12-14 Thread Anthony Liguori
Daniel P. Berrange wrote: On Mon, Dec 14, 2009 at 08:42:12AM -0600, Anthony Liguori wrote: Avi Kivity wrote: On 12/13/2009 01:46 AM, Anthony Liguori wrote: Dan Berrange and I have been talking about being able to move VNC server into a central process such that all of the VMs ca

Re: [Qemu-devel] Re: Spice project is now open

2009-12-14 Thread Anthony Liguori
Avi Kivity wrote: On 12/14/2009 05:17 PM, Daniel P. Berrange wrote: Yes - need to pass the encryption state. Hopefully the crypto stacks support this. There's no mechanism for this in the SASL libraries. With GNUTLS there is the ability to preserve negotiated session state from one TLS

Re: [Qemu-devel] Re: Spice project is now open

2009-12-14 Thread Avi Kivity
On 12/14/2009 05:17 PM, Daniel P. Berrange wrote: Yes - need to pass the encryption state. Hopefully the crypto stacks support this. There's no mechanism for this in the SASL libraries. With GNUTLS there is the ability to preserve negotiated session state from one TLS conenection and us

Re: [Qemu-devel] Re: Spice project is now open

2009-12-14 Thread Daniel P. Berrange
On Mon, Dec 14, 2009 at 04:53:07PM +0200, Avi Kivity wrote: > On 12/14/2009 04:42 PM, Anthony Liguori wrote: > >I think it's a bit trickier though because ideally you would want to > >use the vnc protocol to negotiate which vm you're connecting to. > > Right, of course. If the client can no lo

Re: [Qemu-devel] Re: Spice project is now open

2009-12-14 Thread Daniel P. Berrange
On Mon, Dec 14, 2009 at 08:42:12AM -0600, Anthony Liguori wrote: > Avi Kivity wrote: > >On 12/13/2009 01:46 AM, Anthony Liguori wrote: > >> > >>Dan Berrange and I have been talking about being able to move VNC > >>server into a central process such that all of the VMs can have a > >>single VNC po

[Qemu-devel] [FOR 0.12 PATCH v2] Check rom_load_all() return value.

2009-12-14 Thread Gerd Hoffmann
Check rom_load_all() return value. Also don't make option rom loading failure fatal. Signed-off-by: Gerd Hoffmann --- hw/loader.c |2 +- vl.c|5 - 2 files changed, 5 insertions(+), 2 deletions(-) diff --git a/hw/loader.c b/hw/loader.c index 2d7a2c4..89491a2 100644 --- a/hw/l

Re: [Qemu-devel] Re: [PATCH RFC] virtio: add features qdev property

2009-12-14 Thread Gerd Hoffmann
On 12/14/09 14:59, Michael S. Tsirkin wrote: It's not an easy quesiton. If we do it as disable bits, then we get incompatible machines when running on different hosts. In case that one host supports feature which the other doesn't and the feature isn't masked out? Well, management failure I'd

Re: [Qemu-devel] Re: [PATCH RFC] virtio: add features qdev property

2009-12-14 Thread Gerd Hoffmann
On 12/14/09 14:30, Markus Armbruster wrote: Then the property parser would accepts strings such as 'bit1|bit2' and you can have -device 'virtio-blk-pci,disable=blk-identify|ring-indirect' The driver will just do 'vdev->host_features&= ~disable'. Use of '|' in option argument syntax is user

Re: [Qemu-devel] Re: qdev property bug?

2009-12-14 Thread Anthony Liguori
Gerd Hoffmann wrote: On 12/14/09 15:10, Anthony Liguori wrote: The old behavior with two different nic types and -boot n was "undefined". The old etherboot roms were quite large. To large to fit more than one (certainly not two). Two worked with the etherboot roms. Yes, they were 32k. Two

  1   2   >