Messages in this thread | | | Date | Thu, 29 Mar 2012 00:03:36 -0700 | From | Josh Triplett <> | Subject | Re: [PATCH] kvm-amd: Auto-load on CPUs with SVM. |
| |
On Thu, Mar 29, 2012 at 10:33:25AM +0400, Michael Tokarev wrote: > On 28.03.2012 22:32, Josh Triplett wrote: > > Enable x86 feature-based autoloading for the kvm-amd module on CPUs > > with X86_FEATURE_SVM. > > > > Signed-off-by: Josh Triplett <josh@joshtriplett.org> > > It appears to work fine on my amd boxen.
Thanks for testing it. Can I interpret that as a Tested-by/Acked-by?
> The only problem > is that I've no idea when it will be possible to finally > disable the init script (shipped in Debian and other distros) > to load the module based on /proc/cpuinfo contents. It will > do nothing on kernels with this (or kvm-intel) change applied, > but we still need to support older kernels without that feature... :)
Each stable release of Debian has a specific minimum kernel version that it supports, typically determined by the requirements of packages such as udev. As far as I know, the next stable release will ship with 3.2, and I don't think the necessary infrastructure exists in 3.2 to backport this patch (though I'd love to hear otherwise if it does), so that release will probably need to include the init script. However, the next release should have no problem safely dropping it and requiring a sufficiently new kernel.
In unstable (after the release), qemu-kvm could just go ahead and drop the script with a NEWS.Debian entry telling people to add the necessary module to /etc/modules if they have an older kernel.
> Josh, and I'm sure you will be the first who'll file a bugreport > against qemu-kvm in Debian requesting the startup script to be > removed!.. ;))
I don't know where you could possibly get that idea. ;)
- Josh Triplett
| |