public inbox for kvm@vger.kernel.org
 help / color / mirror / Atom feed
From: Carsten Otte <cotte-tA70FqPdS9bQT0dZR+AlfA@public.gmane.org>
To: "Zhang, Xiantao" <xiantao.zhang-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
Cc: carsteno-tA70FqPdS9bQT0dZR+AlfA@public.gmane.org,
	kvm-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org,
	borntrae-23VcF4HTsmIX0ybBhKVfKdBPR1lH4CV8@public.gmane.org,
	Avi Kivity <avi-atKUWr5tajBWk0Htik3J/w@public.gmane.org>
Subject: Re: [PATCH] Add the arg "module" for kvm_arch_init
Date: Mon, 26 Nov 2007 10:23:27 +0100	[thread overview]
Message-ID: <474A908F.9080803@de.ibm.com> (raw)
In-Reply-To: <42DFA526FC41B1429CE7279EF83C6BDC9E77AA-wq7ZOvIWXbMAbVU2wMM1CrfspsVTdybXVpNB7YpNyf8@public.gmane.org>

Zhang, Xiantao wrote:
> Sorry for confusing you. <snip> Is it clear now?
Thanks, yea this makes your intention clear to me. I think what you 
describe is an odd way to initialize both modules, and I'd like to 
suggest to try to come up with the reverse approach where kvm.ko loads 
your other module via request_module() in kvm_arch_init(). But as you 
said, this way does not confict with other archs, therefore I'm fine 
with it.

-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2005.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/

      parent reply	other threads:[~2007-11-26  9:23 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-11-22 23:32 [PATCH] Add the arg "module" for kvm_arch_init Zhang, Xiantao
     [not found] ` <42DFA526FC41B1429CE7279EF83C6BDC9E752C-wq7ZOvIWXbMAbVU2wMM1CrfspsVTdybXVpNB7YpNyf8@public.gmane.org>
2007-11-23  8:19   ` Christian Borntraeger
     [not found]     ` <200711230919.38608.borntraeger-tA70FqPdS9bQT0dZR+AlfA@public.gmane.org>
2007-11-23  8:25       ` Zhang, Xiantao
     [not found]         ` <42DFA526FC41B1429CE7279EF83C6BDC9E773F-wq7ZOvIWXbMAbVU2wMM1CrfspsVTdybXVpNB7YpNyf8@public.gmane.org>
2007-11-23  9:34           ` Carsten Otte
     [not found]             ` <47469EAF.8010301-tA70FqPdS9bQT0dZR+AlfA@public.gmane.org>
2007-11-25  3:23               ` Zhang, Xiantao
     [not found]                 ` <42DFA526FC41B1429CE7279EF83C6BDC9E77AA-wq7ZOvIWXbMAbVU2wMM1CrfspsVTdybXVpNB7YpNyf8@public.gmane.org>
2007-11-25  9:43                   ` Avi Kivity
     [not found]                     ` <474943A4.507-atKUWr5tajBWk0Htik3J/w@public.gmane.org>
2007-11-25 11:59                       ` Zhang, Xiantao
     [not found]                         ` <42DFA526FC41B1429CE7279EF83C6BDC9E77BF-wq7ZOvIWXbMAbVU2wMM1CrfspsVTdybXVpNB7YpNyf8@public.gmane.org>
2007-11-25 12:31                           ` Avi Kivity
     [not found]                             ` <47496B2E.2050101-atKUWr5tajBWk0Htik3J/w@public.gmane.org>
2007-11-26  4:24                               ` Zhang, Xiantao
2007-11-26  9:23                   ` Carsten Otte [this message]

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=474A908F.9080803@de.ibm.com \
    --to=cotte-ta70fqpds9bqt0dzr+alfa@public.gmane.org \
    --cc=avi-atKUWr5tajBWk0Htik3J/w@public.gmane.org \
    --cc=borntrae-23VcF4HTsmIX0ybBhKVfKdBPR1lH4CV8@public.gmane.org \
    --cc=carsteno-tA70FqPdS9bQT0dZR+AlfA@public.gmane.org \
    --cc=kvm-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org \
    --cc=xiantao.zhang-ral2JQCrhuEAvxtiuMwx3w@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox