On 25/04/2016 13:55, Dominik Dingel wrote:
> While in the anonymous ram case we already take care of the right alignment
> such an alignment gurantee does not exist for file backed ram allocation.
>
> Instead, pagesize is used for alignment. On s390 this is not enough for gmap,
> as we need to
On Fri, 04/29 10:26, Dominik Dingel wrote:
> On Fri, 29 Apr 2016 15:32:22 +0800
> Fam Zheng wrote:
>
> > On Mon, 04/25 13:55, Dominik Dingel wrote:
> > > While in the anonymous ram case we already take care of the right
> > > alignment
> > > such an alignment gurantee does not
On Mon, 04/25 13:55, Dominik Dingel wrote:
> While in the anonymous ram case we already take care of the right alignment
> such an alignment gurantee does not exist for file backed ram allocation.
s/gurantee/guarantee/
Otherwise looks good to me,
Reviewed-by: Fam Zheng
>
>
On Fri, 29 Apr 2016 15:32:22 +0800
Fam Zheng wrote:
> On Mon, 04/25 13:55, Dominik Dingel wrote:
> > While in the anonymous ram case we already take care of the right alignment
> > such an alignment gurantee does not exist for file backed ram allocation.
>
>
While in the anonymous ram case we already take care of the right alignment
such an alignment gurantee does not exist for file backed ram allocation.
Instead, pagesize is used for alignment. On s390 this is not enough for gmap,
as we need to satisfy an alignment up to segments.
Reported-by:
On 29/03/2016 11:29, Dominik Dingel wrote:
>> > Why is this part necessary? On x86 you can have 1 megabyte of RAM,
>> > but QEMU_VMALLOC_ALIGN is 2MB.
> You are right, I changed this to keep the change consistent and thought
> the use case of x86 guests with 1 MB RAM might be a little bit
On Thu, 24 Mar 2016 12:35:10 +0100
Paolo Bonzini wrote:
>
>
> On 23/03/2016 22:32, Dominik Dingel wrote:
> > -page_size = qemu_fd_getpagesize(fd);
> > -block->mr->align = page_size;
> > +alignment = MAX(qemu_fd_getpagesize(fd), QEMU_VMALLOC_ALIGN);
> > +
On 23/03/2016 22:32, Dominik Dingel wrote:
> -page_size = qemu_fd_getpagesize(fd);
> -block->mr->align = page_size;
> +alignment = MAX(qemu_fd_getpagesize(fd), QEMU_VMALLOC_ALIGN);
> +block->mr->align = alignment;
>
> -if (memory < page_size) {
> +if (memory <
While in the anonymous ram case we already take care of the right alignment
such an alignment gurantee does not exist for file backed ram allocation.
Instead, pagesize is used for alignment. On s390 this is not enough for gmap,
as we need to satisfy an alignment up to segments.
Reported-by: