All of lore.kernel.org
 help / color / mirror / Atom feed
From: Avi Kivity <avi-atKUWr5tajBWk0Htik3J/w@public.gmane.org>
To: kvm-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org
Subject: Re: error modprobing
Date: Mon, 26 Feb 2007 16:34:06 +0200	[thread overview]
Message-ID: <45E2EFDE.2000807@qumranet.com> (raw)
In-Reply-To: <20070225221645.3adc491b.mailinglists-+AO6ZX++/Fdg9hUCZPvPmw@public.gmane.org>

[-- Attachment #1: Type: text/plain, Size: 466 bytes --]

richard lucassen wrote:
> Kernel: 2.6.20.1
> kvm-15
> AMD Athlon(tm) 64 X2 Dual Core Processor 3800+
> 1G memory
>
> After modprobing kvm-amd, the system becomes very very unstable and I
> have to reboot te get things right (after modprobing kvm-amd e.g. the
> ssh daemon is unaccessible)
>
>   

This was fixed by the attached patch.  Thanks to Richard for providing a 
machine for me to debug on.


-- 
error compiling committee.c: too many arguments to function


[-- Attachment #2: module-load-fix.patch --]
[-- Type: text/x-patch, Size: 1173 bytes --]

commit c1a8557e1da6e7d8bf8f77cb1b47c077f5c2a67d
Author: Avi Kivity <avi-atKUWr5tajBWk0Htik3J/w@public.gmane.org>
Date:   Mon Feb 26 16:29:43 2007 +0200

    KVM: Fix bogus failure in kvm.ko module initialization
    
    A bogus 'return r' can cause an otherwise successful module load to fail.
    This both denies users the use of kvm, and it also denies them the use of
    their machine, as it leaves a filesystem registered with its callbacks
    pointing into now-freed module memory.
    
    Fix by returning a zero like a good module.
    
    Thanks to Richard Lucassen <mailinglists-+AO6ZX++/Fdg9hUCZPvPmw@public.gmane.org> (?) for reporting
    the problem and for providing access to a machine which exhibited it.
    
    Signed-off-by: Avi Kivity <avi-atKUWr5tajBWk0Htik3J/w@public.gmane.org>

diff --git a/drivers/kvm/kvm_main.c b/drivers/kvm/kvm_main.c
index e48b4d7..ca82ba3 100644
--- a/drivers/kvm/kvm_main.c
+++ b/drivers/kvm/kvm_main.c
@@ -2540,7 +2540,7 @@ static __init int kvm_init(void)
 	bad_page_address = page_to_pfn(bad_page) << PAGE_SHIFT;
 	memset(__va(bad_page_address), 0, PAGE_SIZE);
 
-	return r;
+	return 0;
 
 out:
 	kvm_exit_debug();

[-- Attachment #3: Type: text/plain, Size: 345 bytes --]

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV

[-- Attachment #4: Type: text/plain, Size: 186 bytes --]

_______________________________________________
kvm-devel mailing list
kvm-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org
https://lists.sourceforge.net/lists/listinfo/kvm-devel

  parent reply	other threads:[~2007-02-26 14:34 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-02-25 21:16 error modprobing richard lucassen
     [not found] ` <20070225221645.3adc491b.mailinglists-+AO6ZX++/Fdg9hUCZPvPmw@public.gmane.org>
2007-02-26  6:39   ` Avi Kivity
     [not found]     ` <45E280A0.8050803-atKUWr5tajBWk0Htik3J/w@public.gmane.org>
2007-02-26 10:19       ` Andreas Hasenack
     [not found]         ` <200702260719.28511.ahasenack-y7mWNqJcIDpfJ/NunPodnw@public.gmane.org>
2007-02-26 10:23           ` Avi Kivity
2007-02-26 14:34   ` Avi Kivity [this message]
     [not found]     ` <45E2EFDE.2000807-atKUWr5tajBWk0Htik3J/w@public.gmane.org>
2007-02-28 15:52       ` Andreas Hasenack

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=45E2EFDE.2000807@qumranet.com \
    --to=avi-atkuwr5tajbwk0htik3j/w@public.gmane.org \
    --cc=kvm-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.