Re: [PATCH] powerpc/time: use get_tb instead of get_vtb in running_clock

2017-07-13 Thread Benjamin Herrenschmidt
On Thu, 2017-07-13 at 14:55 +0800, hejianet wrote: > Hi Ben > I add some printk logs in watchdog_timer_fn in the guest > [   16.025222] get_vtb=8236291881, get_tb=13756711357, get_timestamp=4 > [   20.025624] get_vtb=9745285807, get_tb=15804711283, get_timestamp=7 > [   24.025042]

Re: [PATCH] powerpc/time: use get_tb instead of get_vtb in running_clock

2017-07-13 Thread hejianet
Hi Ben I add some printk logs in watchdog_timer_fn in the guest [ 16.025222] get_vtb=8236291881, get_tb=13756711357, get_timestamp=4 [ 20.025624] get_vtb=9745285807, get_tb=15804711283, get_timestamp=7 [ 24.025042] get_vtb=11518119641, get_tb=17852711085, get_timestamp=10 [ 28.024074]

Re: [PATCH] powerpc/time: use get_tb instead of get_vtb in running_clock

2017-07-12 Thread Benjamin Herrenschmidt
On Wed, 2017-07-12 at 23:01 +0800, Jia He wrote: > Virtual time base(vtb) is a register which increases only in guest. > Any exit from guest to host will stop the vtb(saved and restored by kvm). > But if there is an IO causes guest exits to host, the guest's watchdog > (watchdog_timer_fn ->

[PATCH] powerpc/time: use get_tb instead of get_vtb in running_clock

2017-07-12 Thread Jia He
Virtual time base(vtb) is a register which increases only in guest. Any exit from guest to host will stop the vtb(saved and restored by kvm). But if there is an IO causes guest exits to host, the guest's watchdog (watchdog_timer_fn -> is_softlockup -> get_timestamp -> running_clock) needs to also