All of lore.kernel.org
 help / color / mirror / Atom feed
From: Arun Sharma <arun.sharma@intel.com>
To: Mike Wray <mike.wray@hp.com>
Cc: Ian Pratt <m+Ian.Pratt@cl.cam.ac.uk>, xen-devel@lists.xensource.com
Subject: Re: VMX device models not getting created anymore?
Date: Tue, 21 Jun 2005 14:01:12 -0700	[thread overview]
Message-ID: <42B88018.1010702@intel.com> (raw)
In-Reply-To: <42B6A4EE.4040203@hp.com>

Mike Wray wrote:

>> However, if I subtract -1, everything magically works :)
> 
> 
> Having to add or subtract 1 to make something work is almost always
> not the right way to fix a problem - because it dosen't address
> the real issue.
> 

You probably missed the smiley :)

>> I think the quick fix is to redefine IOPACKET_PORT to be 3. Will send 
>> a patch to remove the hard coding ASAP.
> 
> 
> That's not the fix. That just replaces one hard-coded constant with 
> another, when
> it shouldn't be hard-coded at all. It's going to break again if any 
> other interdomain
> port is allocated.
> 
> The port to use should be passed to the domain as a parameter.
> This is what is done with the domain controller port,
> and the xenstore port.
> 

Yes of course, I promised to send a patch as well. The quick fix was for 
those who couldn't wait.

> The correct fix is to add the device model port as a parameter to
> xc_vmx_build. In fact this already has control_evtchn as a parameter,
> but ignores it - so you could use that. You could probably re-use
> the start_info field for it too as vmx domains don't seem to be using
> the control channel otherwise:
> 

The problem is that:

self.construct_image()
	...
		xc_vmx_build()
self.configure()
	...
		self.image.createDeviceModel()

The event channel hasn't been created at the time of xc_vmx_build().

	-Arun

  reply	other threads:[~2005-06-21 21:01 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-06-10 21:48 VMX device models not getting created anymore? Ian Pratt
2005-06-15 14:30 ` Mike Wray
2005-06-15 17:09   ` Arun Sharma
2005-06-16  8:56     ` Mike Wray
2005-06-16 17:35       ` Arun Sharma
2005-06-17 13:07         ` Mike Wray
2005-06-17 19:28           ` Arun Sharma
2005-06-17 20:17             ` Arun Sharma
2005-06-17 21:54               ` cannot concatenate 'str' and 'Nonetype' object with creating domain Jurgen Stroo
2005-06-17 22:16                 ` Jurgen Stroo
2005-06-20 11:13               ` VMX device models not getting created anymore? Mike Wray
2005-06-21 21:01                 ` Arun Sharma [this message]
2005-06-22  8:34                   ` Mike Wray
2005-06-20 10:52             ` Mike Wray
  -- strict thread matches above, loose matches on Subject: below --
2005-06-15 17:12 Ian Pratt
2005-06-15 18:02 ` Arun Sharma
2005-06-10 21:22 Arun Sharma

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=42B88018.1010702@intel.com \
    --to=arun.sharma@intel.com \
    --cc=m+Ian.Pratt@cl.cam.ac.uk \
    --cc=mike.wray@hp.com \
    --cc=xen-devel@lists.xensource.com \
    /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.