xen-devel.lists.xenproject.org archive mirror
 help / color / mirror / Atom feed
From: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
To: Bruce Edge <bruce.edge@gmail.com>
Cc: xen-devel@lists.xensource.com
Subject: Re: 3GB memory restriction for pv domU with PCI
Date: Thu, 14 Oct 2010 13:58:37 -0400	[thread overview]
Message-ID: <20101014175837.GC7312@dumpdata.com> (raw)
In-Reply-To: <AANLkTikHW=jQDRA9U5v6ktDNxS-6G=8NL4g88pzskUHv@mail.gmail.com>

> sticking points are this and the 2MB contig alloc for the tachyon 2 MB
> block for the FC SEST. I feel like if we don't get pvops in now, it'll
> be a hard sell later on.

The 2MB contig alloc ought to work with the pci_alloc_coherent function.

> 
> Anyway, thanks for the info and I'll keep my fingers crossed that 5
> days is all it takes.
> 
> >
> > If this is a show-stopper right now, you could come with a hack where
> > the resource check is just ignored.
> >
> I assume one could do this only if not using PCI passthrough?

Well, it is basically relaxing the criteria. In the olden days (2.6.18)
the System RAM region wa not accounted for. Nowadays it is. So when
a PCI device requests a region that the kernel thinks is System RAM
it tells it to buzz of. In 2.6.18, the kernel would have no trouble
giving you that region. So the hack is just to relax the criteria
and maybe just do:

 if (strncmp(r->name,"System RAM") == 0) {
	return OK;
 }

Or something like that in the resource.c file. I do't remember the
details of that file, but that looked like one of those quick-and-fast
ways of doing this. The right way is to actually make the guest aware of the
PCI hole the Dom0 has and just shift the memory as required. It sounds
easy :-)

      reply	other threads:[~2010-10-14 17:58 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-06-11 15:54 3GB memory restriction for pv domU with PCI Bruce Edge
2010-06-11 16:42 ` Konrad Rzeszutek Wilk
2010-10-14  0:10   ` Bruce Edge
2010-10-14 14:23     ` Konrad Rzeszutek Wilk
2010-10-14 14:39       ` Bruce Edge
2010-10-14 17:58         ` Konrad Rzeszutek Wilk [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=20101014175837.GC7312@dumpdata.com \
    --to=konrad.wilk@oracle.com \
    --cc=bruce.edge@gmail.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).