[Qemu-devel] Re: [PATCH 10/10] introduce qemu_ram_map

2010-04-27 Thread Marcelo Tosatti
On Tue, Apr 27, 2010 at 08:32:27AM -0600, Cam Macdonell wrote: > > > > +ram_addr_t qemu_ram_map(ram_addr_t size, void *host) > > +{ > > +    RAMBlock *new_block; > > + > > +    size = TARGET_PAGE_ALIGN(size); > > +    new_block = qemu_malloc(sizeof(*new_block)); > > + > > +    new_block->host = hos

[Qemu-devel] Re: [PATCH 10/10] introduce qemu_ram_map

2010-04-27 Thread Marcelo Tosatti
On Tue, Apr 27, 2010 at 08:28:15AM -0600, Cam Macdonell wrote: > On Mon, Apr 26, 2010 at 12:54 PM, Anthony Liguori > wrote: > > On 04/26/2010 01:49 PM, Marcelo Tosatti wrote: > >> > >> On Mon, Apr 26, 2010 at 01:27:30PM -0500, Anthony Liguori wrote: > >> > >>> > >>> On 04/26/2010 12:59 PM, Marcel

[Qemu-devel] Re: [PATCH 10/10] introduce qemu_ram_map

2010-04-27 Thread Cam Macdonell
On Mon, Apr 26, 2010 at 11:59 AM, Marcelo Tosatti wrote: > Which allows drivers to register an mmaped region into ram block mappings. > To be used by device assignment driver. > > CC: Cam Macdonell > Signed-off-by: Marcelo Tosatti > --- >  cpu-common.h |    1 + >  exec.c       |   28 +++

[Qemu-devel] Re: [PATCH 10/10] introduce qemu_ram_map

2010-04-27 Thread Cam Macdonell
On Mon, Apr 26, 2010 at 12:54 PM, Anthony Liguori wrote: > On 04/26/2010 01:49 PM, Marcelo Tosatti wrote: >> >> On Mon, Apr 26, 2010 at 01:27:30PM -0500, Anthony Liguori wrote: >> >>> >>> On 04/26/2010 12:59 PM, Marcelo Tosatti wrote: >>> Which allows drivers to register an mmaped region

Re: [Qemu-devel] Re: [PATCH 10/10] introduce qemu_ram_map

2010-04-26 Thread Marcelo Tosatti
On Mon, Apr 26, 2010 at 02:20:42PM -0500, Anthony Liguori wrote: > On 04/26/2010 02:14 PM, Marcelo Tosatti wrote: > >On Mon, Apr 26, 2010 at 01:57:37PM -0500, Anthony Liguori wrote: > >>On 04/26/2010 01:50 PM, Marcelo Tosatti wrote: > >>>On Mon, Apr 26, 2010 at 01:29:06PM -0500, Anthony Liguori wro

Re: [Qemu-devel] Re: [PATCH 10/10] introduce qemu_ram_map

2010-04-26 Thread Anthony Liguori
On 04/26/2010 02:14 PM, Marcelo Tosatti wrote: On Mon, Apr 26, 2010 at 01:57:37PM -0500, Anthony Liguori wrote: On 04/26/2010 01:50 PM, Marcelo Tosatti wrote: On Mon, Apr 26, 2010 at 01:29:06PM -0500, Anthony Liguori wrote: On 04/26/2010 12:59 PM, Marcelo Tosatti wrote:

[Qemu-devel] Re: [PATCH 10/10] introduce qemu_ram_map

2010-04-26 Thread Marcelo Tosatti
On Mon, Apr 26, 2010 at 01:57:37PM -0500, Anthony Liguori wrote: > On 04/26/2010 01:50 PM, Marcelo Tosatti wrote: > >On Mon, Apr 26, 2010 at 01:29:06PM -0500, Anthony Liguori wrote: > >>On 04/26/2010 12:59 PM, Marcelo Tosatti wrote: > >>>Which allows drivers to register an mmaped region into ram bl

[Qemu-devel] Re: [PATCH 10/10] introduce qemu_ram_map

2010-04-26 Thread Anthony Liguori
On 04/26/2010 01:50 PM, Marcelo Tosatti wrote: On Mon, Apr 26, 2010 at 01:29:06PM -0500, Anthony Liguori wrote: On 04/26/2010 12:59 PM, Marcelo Tosatti wrote: Which allows drivers to register an mmaped region into ram block mappings. To be used by device assignment driver. T

[Qemu-devel] Re: [PATCH 10/10] introduce qemu_ram_map

2010-04-26 Thread Anthony Liguori
On 04/26/2010 01:49 PM, Marcelo Tosatti wrote: On Mon, Apr 26, 2010 at 01:27:30PM -0500, Anthony Liguori wrote: On 04/26/2010 12:59 PM, Marcelo Tosatti wrote: Which allows drivers to register an mmaped region into ram block mappings. To be used by device assignment driver. Th

[Qemu-devel] Re: [PATCH 10/10] introduce qemu_ram_map

2010-04-26 Thread Marcelo Tosatti
On Mon, Apr 26, 2010 at 01:29:06PM -0500, Anthony Liguori wrote: > On 04/26/2010 12:59 PM, Marcelo Tosatti wrote: > >Which allows drivers to register an mmaped region into ram block mappings. > >To be used by device assignment driver. > > > > This is not kvm specific and not required by this pull

[Qemu-devel] Re: [PATCH 10/10] introduce qemu_ram_map

2010-04-26 Thread Marcelo Tosatti
On Mon, Apr 26, 2010 at 01:27:30PM -0500, Anthony Liguori wrote: > On 04/26/2010 12:59 PM, Marcelo Tosatti wrote: > >Which allows drivers to register an mmaped region into ram block mappings. > >To be used by device assignment driver. > > This doesn't make much sense to me. > > Do you use this li

[Qemu-devel] Re: [PATCH 10/10] introduce qemu_ram_map

2010-04-26 Thread Anthony Liguori
On 04/26/2010 12:59 PM, Marcelo Tosatti wrote: Which allows drivers to register an mmaped region into ram block mappings. To be used by device assignment driver. This is not kvm specific and not required by this pull request so it shouldn't really be part of the pull. Something like this

[Qemu-devel] Re: [PATCH 10/10] introduce qemu_ram_map

2010-04-26 Thread Anthony Liguori
On 04/26/2010 12:59 PM, Marcelo Tosatti wrote: Which allows drivers to register an mmaped region into ram block mappings. To be used by device assignment driver. This doesn't make much sense to me. Do you use this like: qemu_ram_map(64k, ptr); assert(qemu_ram_alloc(64k) == ptr); If so, I