kvm-83 release

The biggest change this time around is support for the AMD IOMMU for device assignment. As it requires extensive core kernel support, it is only available on Linux 2.6.29rc1 or higher hosts.

The biggest change this time around is support for the AMD IOMMU for device assignment. As it requires extensive core kernel support, it is only available on Linux 2.6.29rc1 or higher hosts.

Changes from kvm-82:
- amd iommu support (with Linux 2.6.29+ hosts) (Joerg Roedel)
- merge qemu-svn
- qcow2 correctness fixes
- '-drive serial=...' option
- virtio-net migration fixes
- fix '--with-patched-kernel' build on Ubuntu (Nolan Leake)
- improved tsc handling (Marcelo Tosatti)
- should reduce timing errors on older Linux guests
- remove cpuid code duplication due to upstream qemu work (Amit Shah)
- guest S3 sleep (Gleb Natapov)
- load assigned devices option rom (Kechao Liu)
- '-no-kvm-pit-reinjection' option to improve timing on RHEL 3 era guests (Marcelo Tosatti)
- remove duplicate firmware config interface probe in bios (Sebastian Herbszt)
- backward compatibility for marker_synchronize_register() (Eduardo Habkost)
- fix compile warning in tpr patching (Mark McLoughlin)
- don't install kvm headers to /usr/local
- fix smbios memory size in certain cases (Bill Rieske)
- simplify mmu shadow walking function
- stop using vmap() on guest pages (annoys ksm) (Izik Eidus)
- fix warning on MSR_VM_HSAVE_PA in dmesg
- fix i8254 timer count read (Marcelo Tosatti)
- ppc e500 core support (Hollis Blanchard)
- fix vmx real mode guest state validity checks
- make pop instruction emulation a little more generic
- implement 'ret far' instruction emulation
- properly lock i8354 timer
- remove ppc debug support (Hollis Blanchard)
- fix race freeing assigned device interrupt (Sheng Yang)
- fix pci interrupt in freeing in device assignment (Sheng Yang)
- allow disabling msi for assigned devices (Sheng Yang)
- emulate amd microcode level msr (Alexander Graf)
- helps esx boot on kvm
- make page unshadowing a little faster
- clear pending pit interrupts when unmasking the timer irq
- fixes xen-on-kvm
- mmu notifiers release method (Marcelo Tosatti)
- fixes large pages support
- expose ia64 registers to userspace (Jes Sorensen)
- fix type in has_svm() (Joe Perches)
- drop unnecessary memory clearing when allocating shadow page (Marcelo Tosatti)
- rename "metaphysical" page attribute to "direct"
- fix irq routing with -no-kernel-irqchip

Notes:
If you use the modules bundled with kvm-83, you can use any version
of Linux from 2.6.16 upwards. You may also use kvm-83 userspace with the kvm modules provided by Linux 2.6.25 or above. Some features may only be available in newer releases.

http://kvm.qumranet.com

Hi, i installed kvm-83 on xubuntu-amd64/kernel-2.6.7-11. i notice some problem with -net downscript.
my kvm start up script like this:
qemu-system-x86_64 ... ... -net tap,ifname=tap0,script=/path/kvm-ifup,downscript=/path/kvm-ifdown when the guest shutdown, kvm pass all argument to downscript.

the command like this (kvm-83):
/path/kvm-ifdown "tap0,script=/path/kvm-ifup,downscript=/path/kvm-ifdown"

kvm-82 only pass tap0 to downscript like this:
/path/kvm-ifdown "tap0"

Hi John,

Quoted 7 lines · John Wong

Hi, i installed kvm-83 on xubuntu-amd64/kernel-2.6.7-11. i notice some problem with -net downscript.
my kvm start up script like this:
qemu-system-x86_64 ... ... -net tap,ifname=tap0,script=/path/kvm-ifup,downscript=/path/kvm-ifdown when the guest shutdown, kvm pass all argument to downscript.

Quoted 7 lines

the command like this (kvm-83):
/path/kvm-ifdown "tap0,script=/path/kvm-ifup,downscript=/path/kvm-ifdown"

kvm-82 only pass tap0 to downscript like this:
/path/kvm-ifdown "tap0"

Thanks for the report. Can you try this simple change and see if it fixes it for you?

net.c +1 -1

@@ -1740,7 +1740,7 @@ void net_cleanup(void)
                 TAPState *s = vc->opaque;
 
                 if (strcmp(vc->model, "tap") == 0 &&
-                    sscanf(vc->info_str, "ifname=%63s ", ifname) == 1 &&
+                    sscanf(vc->info_str, "ifname=%63s,", ifname) == 1 &&
                     s->down_script[0])
                     launch_script(s->down_script, ifname, s->fd);
             }

Thanks,
Mark.

--
To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html

Hi Mark, i want to try, but i don't know where can download snapshot source code,
i'm long time no use linux.
Where can i download kvm snapshot source code? kernel.org? or?

Hi John,

Quoted 7 lines · John Wong

Hi, i installed kvm-83 on xubuntu-amd64/kernel-2.6.7-11. i notice some problem with -net downscript.
my kvm start up script like this:
qemu-system-x86_64 ... ... -net tap,ifname=tap0,script=/path/kvm-ifup,downscript=/path/kvm-ifdown when the guest shutdown, kvm pass all argument to downscript.

Quoted 6 lines

the command like this (kvm-83):
/path/kvm-ifdown "tap0,script=/path/kvm-ifup,downscript=/path/kvm-ifdown"

kvm-82 only pass tap0 to downscript like this:
/path/kvm-ifdown "tap0"

Thanks for the report. Can you try this simple change and see if it fixes it for you?

Mark McLoughlin wrote:

net.c void net_cleanup(void) · line 1743

Quoted 8 lines

net.c +1 -1

@@ -1740,7 +1740,7 @@ void net_cleanup(void)
                 TAPState *s = vc->opaque;
 
                 if (strcmp(vc->model, "tap") == 0 &&
-                    sscanf(vc->info_str, "ifname=%63s ", ifname) == 1 &&
+                    sscanf(vc->info_str, "ifname=%63s,", ifname) == 1 &&

Hmm. What if ifname=foo is the last parameter?

Quoted 2 lines · Michael Tokarev

Mark McLoughlin wrote:

[]

Quoted 11 lines

net.c void net_cleanup(void) · line 1744

Quoted 9 lines

net.c +1 -1

@@ -1740,7 +1740,7 @@ void net_cleanup(void)
                 TAPState *s = vc->opaque;
 
                 if (strcmp(vc->model, "tap") == 0 &&
-                    sscanf(vc->info_str, "ifname=%63s ", ifname) == 1 &&
+                    sscanf(vc->info_str, "ifname=%63s,", ifname) == 1 &&
 

Hmm. What if ifname=foo is the last parameter?

And while we're at it.. Why the parsing isn't done like this:

  for(tok = strtok(arg, ","); tok; tok = strtok(NULL, ",")) {
    if (strncmp(tok, "ifname=", 7) == 0)
      strcpy(ifname, tok+7);
    else if ...
  }

-- here and everywhere else?

/mjt
--
To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html

Quoted 1 line

Michael Tokarev wrote:

Quoted 6 lines

Quoted 3 lines

net.c void net_cleanup(void) · line 1743

Quoted 3 lines

       if (strcmp(vc->model, "tap") == 0 &&

net.c +1 -1

-          sscanf(vc->info_str, "ifname=%63s ", ifname) == 1 &&
+          sscanf(vc->info_str, "ifname=%63s,", ifname) == 1 &&

And while we're at it.. Why the parsing isn't done like this:

Quoted 5 lines

  for(tok = strtok(arg, ","); tok; tok = strtok(NULL, ",")) {
    if (strncmp(tok, "ifname=", 7) == 0)
      strcpy(ifname, tok+7);
    else if ...
  }

Because strtok is extremely evil.

Quoted 1 line · Anthony Liguori

Quoted 1 line

Michael Tokarev wrote:

...

Quoted 12 lines

Quoted 11 lines

Quoted 3 lines

net.c void net_cleanup(void) · line 1743

Quoted 3 lines

       if (strcmp(vc->model, "tap") == 0 &&

net.c +1 -1

-          sscanf(vc->info_str, "ifname=%63s ", ifname) == 1 &&
+          sscanf(vc->info_str, "ifname=%63s,", ifname) == 1 &&

And while we're at it.. Why the parsing isn't done like this:

  for(tok = strtok(arg, ","); tok; tok = strtok(NULL, ",")) {
    if (strncmp(tok, "ifname=", 7) == 0)
      strcpy(ifname, tok+7);
    else if ...
  }

Quoted 1 line

Because strtok is extremely evil.

I didn't mean strtok, but the way -- something LIKE that, i.e, by splitting the string into args and parsing each in turn.

Quoted 4 lines · Avi Kivity

The biggest change this time around is support for the AMD IOMMU for device assignment. As it requires extensive core kernel support, it is only available on Linux 2.6.29rc1 or higher hosts.

status report on rhel/centos-5.2 host:
- fedora-9 latest kernel now boot again
- fedora-10 installer stop at "waiting for hardware..." so it's currently not able to upgrade fedora-9 guest neither from cdrom nor form preupgrade (ie hard disk).

and these dozens of warning still comes during compilation on rhel/centos 5.2:
-----------------------------------------
In file included from /home/robot/rpm/BUILD/kvm-83/kernel/include/asm/kvm_host.h:65,
from
/home/robot/rpm/BUILD/kvm-83/kernel/include/linux/kvm_host.h:67,

                 from /home/robot/rpm/BUILD/kvm-83/kernel/x86/lapic.c:60:

/home/robot/rpm/BUILD/kvm-83/kernel/include-compat/asm/msr-index.h:30:1:
warning: "MSR_IA32_PERFCTR0" redefined

Quoted 7 lines · Farkas Levente

Quoted 4 lines · Avi Kivity

The biggest change this time around is support for the AMD IOMMU for device assignment. As it requires extensive core kernel support, it is only available on Linux 2.6.29rc1 or higher hosts.

Quoted 6 lines

status report on rhel/centos-5.2 host:
- fedora-9 latest kernel now boot again
- fedora-10 installer stop at "waiting for hardware..." so it's currently not able to upgrade fedora-9 guest neither from cdrom nor form preupgrade (ie hard disk).

Works here. What's your host cpu type? Does the error occur if you pin the guest to one host cpu?