Re: CVS commit: src/sys/arch/xen

2012-06-24 Thread Jean-Yves Migeon
On 24.06.2012 23:44, Christoph Egger wrote:
> On 24.06.12 20:31, Jean-Yves Migeon wrote:
> 
>> Module Name:src
>> Committed By:jym
>> Date:Sun Jun 24 18:31:53 UTC 2012
>>
>> Modified Files:
>> src/sys/arch/xen/include: xenpmap.h
>> src/sys/arch/xen/x86: xen_pmap.c
>>
>> Log Message:
>> Enable the map/unmap recursive mapping functions for all Xen ports for
>> save/restore.
>>
>> For an unknown reason (to me) Xen refuses to update VM translations
>> when the entry is pointing back to itself (which is precisely
>> what our recursive VM model does). So enable the functions that take
>> care of this, which will avoid all sort of memory corruption upon restore
>> leading domU to trample upon itself.
> 
> Please report this Xen behaviour to xen-devel.

It has always been a tricky step anyway; it already happened with PAE.

I am not even sure that Xen is at fault. IIRC the translations fixup
step is done by the low level tools upon restore, not the hypervisor
directly.

IMHO correcting the fixup step will never be possible nor perfect
anyway; save operation is like a snapshot: it loses all the history of
how the mappings were created. It can get accepted by hypervisor when
you validate it in a given order, and not when done it is done in another.

-- 
Jean-Yves Migeon
jeanyves.mig...@free.fr


Re: CVS commit: src/sys/arch/xen

2012-06-24 Thread Christoph Egger

On 24.06.12 20:31, Jean-Yves Migeon wrote:


Module Name:src
Committed By:   jym
Date:   Sun Jun 24 18:31:53 UTC 2012

Modified Files:
src/sys/arch/xen/include: xenpmap.h
src/sys/arch/xen/x86: xen_pmap.c

Log Message:
Enable the map/unmap recursive mapping functions for all Xen ports for
save/restore.

For an unknown reason (to me) Xen refuses to update VM translations
when the entry is pointing back to itself (which is precisely
what our recursive VM model does). So enable the functions that take
care of this, which will avoid all sort of memory corruption upon restore
leading domU to trample upon itself.



Please report this Xen behaviour to xen-devel.

Christoph

 

Save/restore works again for amd64. The occasional domU frontend corruption is
still present, but is harmless to dom0. Now we have a working shell and
ddb inside domU, that helps debugging a tiny bit.

XXX pull-up to -6.





Re: CVS commit: src/share/man/man4

2012-06-24 Thread David Holland
On Sat, Jun 23, 2012 at 07:14:35PM +, David Brownlee wrote:
 > Modified Files:
 >  src/share/man/man4: cd.4
 > 
 > Log Message:
 > Document DIOCTUR (test unit ready)

So, if your CD's unwell, what do you do? Call the DIOCTUR!

-- 
David A. Holland
dholl...@netbsd.org