On 04.03.2012, at 22:19, Benjamin Herrenschmidt
wrote:
> On Sun, 2012-03-04 at 21:59 +0100, Alexander Graf wrote:
> g_assert(TARGET_PAGE_SIZE <= getpagesize())
>
> Just declare the above case as unsupported and abort if we
>> encounter it.
What I'm trying to tell you is
Am 04.03.2012 21:59, schrieb Alexander Graf:
>
>
> On 04.03.2012, at 21:31, Andreas Färber wrote:
>
>> Am 04.03.2012 21:25, schrieb Alexander Graf:
>>>
>>>
>>> On 04.03.2012, at 21:21, Andreas Färber wrote:
>>>
Am 04.03.2012 19:46, schrieb Alexander Graf:
>
>
> On 04.03.2012,
On Sun, 2012-03-04 at 21:59 +0100, Alexander Graf wrote:
> >>> g_assert(TARGET_PAGE_SIZE <= getpagesize())
> >>>
> >>> Just declare the above case as unsupported and abort if we
> encounter it.
> >>
> >> What I'm trying to tell you is that it's the default case on book3s
> ppc! ;)
> >
> > Exactl
On Sun, 2012-03-04 at 17:46 +0100, Andreas Färber wrote:
> Except for ppcemb-softmmu (1k), which is irrelevant for KVM AFAIU.
>
> Maybe just add an assert and be done with it?
Well, my patch should work anyway, as long as getpagesize() returns a
higher power of two.
The case that wouldn't work (
On 04.03.2012, at 21:31, Andreas Färber wrote:
> Am 04.03.2012 21:25, schrieb Alexander Graf:
>>
>>
>> On 04.03.2012, at 21:21, Andreas Färber wrote:
>>
>>> Am 04.03.2012 19:46, schrieb Alexander Graf:
On 04.03.2012, at 17:46, Andreas Färber wrote:
> Am 04.03.201
Am 04.03.2012 21:25, schrieb Alexander Graf:
>
>
> On 04.03.2012, at 21:21, Andreas Färber wrote:
>
>> Am 04.03.2012 19:46, schrieb Alexander Graf:
>>>
>>>
>>> On 04.03.2012, at 17:46, Andreas Färber wrote:
>>>
Am 04.03.2012 12:53, schrieb Benjamin Herrenschmidt:
> On Sun, 2012-03-04
On 04.03.2012, at 21:21, Andreas Färber wrote:
> Am 04.03.2012 19:46, schrieb Alexander Graf:
>>
>>
>> On 04.03.2012, at 17:46, Andreas Färber wrote:
>>
>>> Am 04.03.2012 12:53, schrieb Benjamin Herrenschmidt:
On Sun, 2012-03-04 at 12:49 +0200, Avi Kivity wrote:
> On 02/28/2012 11:
Am 04.03.2012 19:46, schrieb Alexander Graf:
>
>
> On 04.03.2012, at 17:46, Andreas Färber wrote:
>
>> Am 04.03.2012 12:53, schrieb Benjamin Herrenschmidt:
>>> On Sun, 2012-03-04 at 12:49 +0200, Avi Kivity wrote:
On 02/28/2012 11:48 PM, Benjamin Herrenschmidt wrote:
> On Tue, 2012-02-2
On 04.03.2012, at 17:46, Andreas Färber wrote:
> Am 04.03.2012 12:53, schrieb Benjamin Herrenschmidt:
>> On Sun, 2012-03-04 at 12:49 +0200, Avi Kivity wrote:
>>> On 02/28/2012 11:48 PM, Benjamin Herrenschmidt wrote:
On Tue, 2012-02-28 at 14:32 +0200, Avi Kivity wrote:
> What if T
Am 04.03.2012 12:53, schrieb Benjamin Herrenschmidt:
> On Sun, 2012-03-04 at 12:49 +0200, Avi Kivity wrote:
>> On 02/28/2012 11:48 PM, Benjamin Herrenschmidt wrote:
>>> On Tue, 2012-02-28 at 14:32 +0200, Avi Kivity wrote:
>>>
What if TARGET_PAGE_SIZE > getpagesize()? Or is that impossible?
>>
On 03/04/2012 01:53 PM, Benjamin Herrenschmidt wrote:
> On Sun, 2012-03-04 at 12:49 +0200, Avi Kivity wrote:
> > On 02/28/2012 11:48 PM, Benjamin Herrenschmidt wrote:
> > > On Tue, 2012-02-28 at 14:32 +0200, Avi Kivity wrote:
> > >
> > > > What if TARGET_PAGE_SIZE > getpagesize()? Or is that impos
On Sun, 2012-03-04 at 12:49 +0200, Avi Kivity wrote:
> On 02/28/2012 11:48 PM, Benjamin Herrenschmidt wrote:
> > On Tue, 2012-02-28 at 14:32 +0200, Avi Kivity wrote:
> >
> > > What if TARGET_PAGE_SIZE > getpagesize()? Or is that impossible?
> >
> > We have yet to encounter such a case. It's not cu
On 02/28/2012 11:48 PM, Benjamin Herrenschmidt wrote:
> On Tue, 2012-02-28 at 14:32 +0200, Avi Kivity wrote:
>
> > What if TARGET_PAGE_SIZE > getpagesize()? Or is that impossible?
>
> We have yet to encounter such a case. It's not currently possible on
> power (some old embedded chips could do 1K
On Mon, Feb 27, 2012 at 00:36, Alexander Graf wrote:
>
> On 26.02.2012, at 22:41, Blue Swirl wrote:
>
>> On Fri, Feb 24, 2012 at 00:23, David Gibson
>> wrote:
>>> From: Benjamin Herrenschmidt
>>>
>>> If the kernel page size is larger than TARGET_PAGE_SIZE, which
>>> happens for example on ppc64
On Wed, Feb 29, 2012 at 12:32:51AM +0100, Alexander Graf wrote:
>
> On 24.02.2012, at 01:23, David Gibson wrote:
>
> > From: Benjamin Herrenschmidt
> >
> > If the kernel page size is larger than TARGET_PAGE_SIZE, which
> > happens for example on ppc64 with kernels compiled for 64K pages,
> > th
On 24.02.2012, at 01:23, David Gibson wrote:
> From: Benjamin Herrenschmidt
>
> If the kernel page size is larger than TARGET_PAGE_SIZE, which
> happens for example on ppc64 with kernels compiled for 64K pages,
> the dirty tracking doesn't work.
>
> Cc: Avi Kivity
> Cc: Marcelo Tossatti
>
>
On Tue, 2012-02-28 at 14:32 +0200, Avi Kivity wrote:
> What if TARGET_PAGE_SIZE > getpagesize()? Or is that impossible?
We have yet to encounter such a case. It's not currently possible on
power (some old embedded chips could do 1K and 2K page sizes in the TLB
iirc but we never supported that in
On 02/24/2012 02:23 AM, David Gibson wrote:
> From: Benjamin Herrenschmidt
>
> If the kernel page size is larger than TARGET_PAGE_SIZE, which
> happens for example on ppc64 with kernels compiled for 64K pages,
> the dirty tracking doesn't work.
>
> Cc: Avi Kivity
> Cc: Marcelo Tossatti
>
> Signe
On 26.02.2012, at 22:41, Blue Swirl wrote:
> On Fri, Feb 24, 2012 at 00:23, David Gibson
> wrote:
>> From: Benjamin Herrenschmidt
>>
>> If the kernel page size is larger than TARGET_PAGE_SIZE, which
>> happens for example on ppc64 with kernels compiled for 64K pages,
>> the dirty tracking doe
On Mon, 2012-02-27 at 11:16 +1100, David Gibson wrote:
> > > If the kernel page size is larger than TARGET_PAGE_SIZE, which
> > > happens for example on ppc64 with kernels compiled for 64K pages,
> > > the dirty tracking doesn't work.
> >
> > I think a better solution would be to push this to memo
On Sun, Feb 26, 2012 at 09:41:17PM +, Blue Swirl wrote:
> On Fri, Feb 24, 2012 at 00:23, David Gibson
> wrote:
> > From: Benjamin Herrenschmidt
> >
> > If the kernel page size is larger than TARGET_PAGE_SIZE, which
> > happens for example on ppc64 with kernels compiled for 64K pages,
> > the
On Fri, Feb 24, 2012 at 00:23, David Gibson wrote:
> From: Benjamin Herrenschmidt
>
> If the kernel page size is larger than TARGET_PAGE_SIZE, which
> happens for example on ppc64 with kernels compiled for 64K pages,
> the dirty tracking doesn't work.
I think a better solution would be to push t
> diff --git a/kvm-all.c b/kvm-all.c
> index 5e188bf..3f8cfd9 100644
> --- a/kvm-all.c
> +++ b/kvm-all.c
> @@ -348,10 +348,11 @@ static int kvm_set_migration_log(int enable)
> static int kvm_get_dirty_pages_log_range(MemoryRegionSection *section,
> unsigne
23 matches
Mail list logo