public inbox for kvm@vger.kernel.org
 help / color / mirror / Atom feed
* Stop the clock!
@ 2008-03-04 10:47 Avi Kivity
  2008-03-04 12:41 ` Glauber Costa
  2008-03-04 15:49 ` Joerg Roedel
  0 siblings, 2 replies; 3+ messages in thread
From: Avi Kivity @ 2008-03-04 10:47 UTC (permalink / raw)
  To: Glauber de Oliveira Costa; +Cc: kvm-devel

With paravirt clocksource, reboot and kexec are broken: the clock keeps 
updating after the reboot, and the new kernel will have a random memory 
location trampled occasionally.

So we need to stop the clock on kexec (in the guest) and reboot (in the 
host).  On the host side, this can be done either in the kernel, or in 
userspace via new ioctls.

Joerg, I think you mentioned you were working on a vm-wide reset 
ioctl()?  If so, that would be the place to stop the clock on reboot.

Glauber, can you extend the interface to support stopping the clock?  It 
needs to be done even outside kexec, for example if the the user decides 
to stop using your clock.

-- 
error compiling committee.c: too many arguments to function


-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: Stop the clock!
  2008-03-04 10:47 Stop the clock! Avi Kivity
@ 2008-03-04 12:41 ` Glauber Costa
  2008-03-04 15:49 ` Joerg Roedel
  1 sibling, 0 replies; 3+ messages in thread
From: Glauber Costa @ 2008-03-04 12:41 UTC (permalink / raw)
  To: Avi Kivity; +Cc: kvm-devel

Avi Kivity wrote:
> With paravirt clocksource, reboot and kexec are broken: the clock keeps 
> updating after the reboot, and the new kernel will have a random memory 
> location trampled occasionally.
> 
> So we need to stop the clock on kexec (in the guest) and reboot (in the 
> host).  On the host side, this can be done either in the kernel, or in 
> userspace via new ioctls.
> 
> Joerg, I think you mentioned you were working on a vm-wide reset 
> ioctl()?  If so, that would be the place to stop the clock on reboot.
> 
> Glauber, can you extend the interface to support stopping the clock?  It 
> needs to be done even outside kexec, for example if the the user decides 
> to stop using your clock.
> 
Sure, I'll work on it.

-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: Stop the clock!
  2008-03-04 10:47 Stop the clock! Avi Kivity
  2008-03-04 12:41 ` Glauber Costa
@ 2008-03-04 15:49 ` Joerg Roedel
  1 sibling, 0 replies; 3+ messages in thread
From: Joerg Roedel @ 2008-03-04 15:49 UTC (permalink / raw)
  To: Avi Kivity; +Cc: kvm-devel, Glauber de Oliveira Costa

On Tue, Mar 04, 2008 at 12:47:58PM +0200, Avi Kivity wrote:
> With paravirt clocksource, reboot and kexec are broken: the clock keeps updating after the reboot, and the new kernel will have a random memory location trampled occasionally.
> 
> So we need to stop the clock on kexec (in the guest) and reboot (in the host).  On the host side, this can be done either in the kernel, or in userspace via new ioctls.
> 
> Joerg, I think you mentioned you were working on a vm-wide reset ioctl()?  If so, that would be the place to stop the clock on reboot.
> 
> Glauber, can you extend the interface to support stopping the clock?  It needs to be done even outside kexec, for example if the the user decides to stop using your clock.

Yes, I am working on that ioctl(), but don't expect a patch this week
because I am busy with an other task temporarily.

Joerg

-- 
           |           AMD Saxony Limited Liability Company & Co. KG
 Operating |         Wilschdorfer Landstr. 101, 01109 Dresden, Germany
 System    |                  Register Court Dresden: HRA 4896
 Research  |              General Partner authorized to represent:
 Center    |             AMD Saxony LLC (Wilmington, Delaware, US)
           | General Manager of AMD Saxony LLC: Dr. Hans-R. Deppe, Thomas McCoy


-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2008-03-04 15:49 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2008-03-04 10:47 Stop the clock! Avi Kivity
2008-03-04 12:41 ` Glauber Costa
2008-03-04 15:49 ` Joerg Roedel

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox