All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Daniel P. Berrange" <berrange-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
To: Anthony Liguori <anthony-rdkfGonbjUSkNkDKm+mE6A@public.gmane.org>
Cc: kvm-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org,
	Ian Jackson <Ian.Jackson-mvvWK6WmYclDPfheJLI6IQ@public.gmane.org>,
	qemu-devel-qX2TKyscuCcdnm+yROfE0A@public.gmane.org,
	Avi Kivity <avi-atKUWr5tajBWk0Htik3J/w@public.gmane.org>,
	Paul Brook <paul-qD8j1LwMmJjtCj0u4l0SBw@public.gmane.org>
Subject: Re: [Qemu-devel] Re: [PATCH 1/6] Use correct types to enable > 2G support
Date: Fri, 1 Feb 2008 17:57:50 +0000	[thread overview]
Message-ID: <20080201175750.GD22981@redhat.com> (raw)
In-Reply-To: <47A35C7E.8050309-rdkfGonbjUSkNkDKm+mE6A@public.gmane.org>

On Fri, Feb 01, 2008 at 11:53:02AM -0600, Anthony Liguori wrote:
> Ian Jackson wrote:
> > Anthony Liguori writes ("[Qemu-devel] Re: [kvm-devel] [PATCH 1/6] Use correct types to enable > 2G support"):
> >   
> >> The alternative is to change all the places that assume phys_ram_base + 
> >> PA which I don't like very much.
> >>     
> >
> > We would ideally like to do this for Xen, at least in the places we
> > care about.  (Xen uses less of the qemu tree than KVM, I think.)
> >   
> 
> Support for the map cache in the Xen tree is a rather big change that 
> I'm not going to attempt to support it in this patch series.
> 
> I'd rather preserve the phys_ram_base + PA assumption because it allows 
> us to be able to do support > 1 page DMA operations for our virtual IO 
> drivers.  If you break the assumption that physically contiguous memory 
> in the guest is virtual contiguous memory in the host, things get pretty 
> ugly.

Well Xen i386 has no choice but to use the map cache, since PAE lets 
i386 guests have as much as 100 GB of memory & there's no way you can
map that into QEMU's 32-bit userspace. So if virt IO has a dependancy
on contigious memory access in QEMU its not going to play nice with
Xen.

Dan.
-- 
|=- Red Hat, Engineering, Emerging Technologies, Boston.  +1 978 392 2496 -=|
|=-           Perl modules: http://search.cpan.org/~danberr/              -=|
|=-               Projects: http://freshmeat.net/~danielpb/               -=|
|=-  GnuPG: 7D3B9505   F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505  -=| 

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

WARNING: multiple messages have this Message-ID (diff)
From: "Daniel P. Berrange" <berrange@redhat.com>
To: Anthony Liguori <anthony@codemonkey.ws>
Cc: kvm-devel@lists.sourceforge.net,
	Ian Jackson <Ian.Jackson@eu.citrix.com>,
	qemu-devel@nongnu.org, Paul Brook <paul@codesourcery.com>
Subject: Re: [kvm-devel] [Qemu-devel] Re: [PATCH 1/6] Use correct types to enable > 2G support
Date: Fri, 1 Feb 2008 17:57:50 +0000	[thread overview]
Message-ID: <20080201175750.GD22981@redhat.com> (raw)
In-Reply-To: <47A35C7E.8050309@codemonkey.ws>

On Fri, Feb 01, 2008 at 11:53:02AM -0600, Anthony Liguori wrote:
> Ian Jackson wrote:
> > Anthony Liguori writes ("[Qemu-devel] Re: [kvm-devel] [PATCH 1/6] Use correct types to enable > 2G support"):
> >   
> >> The alternative is to change all the places that assume phys_ram_base + 
> >> PA which I don't like very much.
> >>     
> >
> > We would ideally like to do this for Xen, at least in the places we
> > care about.  (Xen uses less of the qemu tree than KVM, I think.)
> >   
> 
> Support for the map cache in the Xen tree is a rather big change that 
> I'm not going to attempt to support it in this patch series.
> 
> I'd rather preserve the phys_ram_base + PA assumption because it allows 
> us to be able to do support > 1 page DMA operations for our virtual IO 
> drivers.  If you break the assumption that physically contiguous memory 
> in the guest is virtual contiguous memory in the host, things get pretty 
> ugly.

Well Xen i386 has no choice but to use the map cache, since PAE lets 
i386 guests have as much as 100 GB of memory & there's no way you can
map that into QEMU's 32-bit userspace. So if virt IO has a dependancy
on contigious memory access in QEMU its not going to play nice with
Xen.

Dan.
-- 
|=- Red Hat, Engineering, Emerging Technologies, Boston.  +1 978 392 2496 -=|
|=-           Perl modules: http://search.cpan.org/~danberr/              -=|
|=-               Projects: http://freshmeat.net/~danielpb/               -=|
|=-  GnuPG: 7D3B9505   F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505  -=| 

  parent reply	other threads:[~2008-02-01 17:57 UTC|newest]

Thread overview: 66+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-01-31 22:36 [PATCH 0/6] Support for the Kernel Virtual Machine interface Anthony Liguori
2008-01-31 22:36 ` [Qemu-devel] " Anthony Liguori
     [not found] ` <1201818980-27534-1-git-send-email-aliguori-r/Jw6+rmf7HQT0dZR+AlfA@public.gmane.org>
2008-01-31 22:36   ` [PATCH 1/6] Use correct types to enable > 2G support Anthony Liguori
2008-01-31 22:36     ` [Qemu-devel] " Anthony Liguori
     [not found]     ` <1201818980-27534-2-git-send-email-aliguori-r/Jw6+rmf7HQT0dZR+AlfA@public.gmane.org>
2008-01-31 23:54       ` Paul Brook
2008-01-31 23:54         ` [Qemu-devel] " Paul Brook
     [not found]         ` <200801312354.24382.paul-qD8j1LwMmJjtCj0u4l0SBw@public.gmane.org>
2008-02-01  0:25           ` Anthony Liguori
2008-02-01  0:25             ` [Qemu-devel] " Anthony Liguori
     [not found]             ` <47A26714.4000204-r/Jw6+rmf7HQT0dZR+AlfA@public.gmane.org>
2008-02-01  0:37               ` Paul Brook
2008-02-01  0:37                 ` [Qemu-devel] " Paul Brook
     [not found]                 ` <200802010037.30731.paul-qD8j1LwMmJjtCj0u4l0SBw@public.gmane.org>
2008-02-01  0:40                   ` Anthony Liguori
2008-02-01  0:40                     ` [Qemu-devel] " Anthony Liguori
2008-02-01 10:26       ` Fabrice Bellard
2008-02-01 10:26         ` [Qemu-devel] " Fabrice Bellard
     [not found]         ` <47A2F3C7.6060409-+Yn2HnRVxb5AfugRpC6u6w@public.gmane.org>
2008-02-01 14:35           ` Anthony Liguori
2008-02-01 14:35             ` [Qemu-devel] " Anthony Liguori
     [not found]             ` <47A32E40.3000204-r/Jw6+rmf7HQT0dZR+AlfA@public.gmane.org>
2008-02-01 15:13               ` Avi Kivity
2008-02-01 15:13                 ` [Qemu-devel] " Avi Kivity
2008-02-01 11:56                 ` Robert William Fuller
2008-02-01 16:09                   ` M. Warner Losh
2008-02-01 16:47                     ` Philip Boulain
2008-02-01 17:35                   ` Jamie Lokier
     [not found]                 ` <47A33721.4020600-atKUWr5tajBWk0Htik3J/w@public.gmane.org>
2008-02-01 15:33                   ` Anthony Liguori
2008-02-01 15:33                     ` [Qemu-devel] Re: [kvm-devel] " Anthony Liguori
     [not found]                     ` <47A33BC6.3060705-r/Jw6+rmf7HQT0dZR+AlfA@public.gmane.org>
2008-02-01 15:40                       ` [Qemu-devel] " Ian Jackson
2008-02-01 15:40                         ` [Qemu-devel] Re: [kvm-devel] " Ian Jackson
     [not found]                         ` <18339.15712.411486.395238-msK/Ju9w1zmnROeE8kUsYhEHtJm+Wo+I@public.gmane.org>
2008-02-01 17:53                           ` [Qemu-devel] " Anthony Liguori
2008-02-01 17:53                             ` [kvm-devel] " Anthony Liguori
     [not found]                             ` <47A35C7E.8050309-rdkfGonbjUSkNkDKm+mE6A@public.gmane.org>
2008-02-01 17:57                               ` Daniel P. Berrange [this message]
2008-02-01 17:57                                 ` Daniel P. Berrange
     [not found]                                 ` <20080201175750.GD22981-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2008-02-01 20:31                                   ` Anthony Liguori
2008-02-01 20:31                                     ` [kvm-devel] " Anthony Liguori
2008-02-01 21:33                                     ` [kvm-devel] " Paul Brook
2008-02-01 21:33                                       ` [kvm-devel] [Qemu-devel] " Paul Brook
2008-02-01 16:00               ` Paul Brook
2008-02-01 16:00                 ` [Qemu-devel] " Paul Brook
     [not found]                 ` <200802011600.10877.paul-qD8j1LwMmJjtCj0u4l0SBw@public.gmane.org>
2008-02-01 16:21                   ` Fabrice Bellard
2008-02-01 16:21                     ` [Qemu-devel] " Fabrice Bellard
     [not found]                     ` <47A3471B.7030702-+Yn2HnRVxb5AfugRpC6u6w@public.gmane.org>
2008-02-05 11:34                       ` Ian Jackson
2008-02-05 11:34                         ` Ian Jackson
2008-02-01 17:49                 ` [kvm-devel] " Anthony Liguori
2008-02-01 17:49                   ` [Qemu-devel] " Anthony Liguori
2008-02-03  8:58       ` Izik Eidus
2008-02-03  8:58         ` [Qemu-devel] Re: [kvm-devel] " Izik Eidus
2008-01-31 22:36   ` [PATCH 2/6] SCI fixes Anthony Liguori
2008-01-31 22:36     ` [Qemu-devel] " Anthony Liguori
2008-01-31 22:36   ` [PATCH 3/6] Fix daemonize options Anthony Liguori
2008-01-31 22:36     ` [Qemu-devel] " Anthony Liguori
2008-01-31 22:36   ` [PATCH 4/6] Tell BIOS about the number of CPUs Anthony Liguori
2008-01-31 22:36     ` [Qemu-devel] " Anthony Liguori
     [not found]     ` <1201818980-27534-5-git-send-email-aliguori-r/Jw6+rmf7HQT0dZR+AlfA@public.gmane.org>
2008-02-01  0:14       ` Paul Brook
2008-02-01  0:14         ` [Qemu-devel] " Paul Brook
     [not found]         ` <200802010014.07960.paul-qD8j1LwMmJjtCj0u4l0SBw@public.gmane.org>
2008-02-01  0:28           ` Anthony Liguori
2008-02-01  0:28             ` [Qemu-devel] " Anthony Liguori
     [not found]             ` <47A26790.6010609-r/Jw6+rmf7HQT0dZR+AlfA@public.gmane.org>
2008-02-01  0:40               ` Paul Brook
2008-02-01  0:40                 ` [Qemu-devel] " Paul Brook
2008-01-31 22:36   ` [PATCH 5/6] Refactor option ROM loading Anthony Liguori
2008-01-31 22:36     ` [Qemu-devel] " Anthony Liguori
2008-01-31 22:36   ` [PATCH 6/6] QEMU support for the Kernel Virtual Machine interface Anthony Liguori
2008-01-31 22:36     ` [Qemu-devel] " Anthony Liguori
     [not found]     ` <1201818980-27534-7-git-send-email-aliguori-r/Jw6+rmf7HQT0dZR+AlfA@public.gmane.org>
2008-02-01  9:49       ` Fabrice Bellard
2008-02-01  9:49         ` [Qemu-devel] " Fabrice Bellard
     [not found]         ` <47A2EB14.8030804-+Yn2HnRVxb5AfugRpC6u6w@public.gmane.org>
2008-02-01 14:18           ` Anthony Liguori
2008-02-01 14:18             ` [Qemu-devel] " Anthony Liguori
2008-01-31 22:53   ` [qemu-devel] [PATCH 0/6] Support " Anthony Liguori
2008-01-31 22:53     ` Anthony Liguori

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=20080201175750.GD22981@redhat.com \
    --to=berrange-h+wxahxf7alqt0dzr+alfa@public.gmane.org \
    --cc=Ian.Jackson-mvvWK6WmYclDPfheJLI6IQ@public.gmane.org \
    --cc=anthony-rdkfGonbjUSkNkDKm+mE6A@public.gmane.org \
    --cc=avi-atKUWr5tajBWk0Htik3J/w@public.gmane.org \
    --cc=kvm-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org \
    --cc=paul-qD8j1LwMmJjtCj0u4l0SBw@public.gmane.org \
    --cc=qemu-devel-qX2TKyscuCcdnm+yROfE0A@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.