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

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

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 +++

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:

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

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

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

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

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

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

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

[PATCH 10/10] introduce qemu_ram_map

2010-04-26 Thread Marcelo Tosatti
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 2 files changed, 29 insertions(+), 0 deletions(-) dif