Re: [PATCH qemu] timer/i8254: Fix one shot PIT mode

2023-10-05 Thread Michael S. Tsirkin
On Fri, Oct 06, 2023 at 02:36:52AM +, Damien Zammit wrote: > >From: Michael Tokarev > >26.02.2023 04:58, Damien Zammit wrote: > >> Currently, the one-shot (mode 1) PIT expires far too quickly, > >> due to the output being set under the wrong logic. > >> This change fixes the one-shot PIT mode

Re: [PATCH qemu] timer/i8254: Fix one shot PIT mode

2023-10-05 Thread Damien Zammit
>From: Michael Tokarev >26.02.2023 04:58, Damien Zammit wrote: >> Currently, the one-shot (mode 1) PIT expires far too quickly, >> due to the output being set under the wrong logic. >> This change fixes the one-shot PIT mode to behave similarly to mode 0. >> >> TESTED: using the one-shot PIT mode

Re: [PATCH qemu] timer/i8254: Fix one shot PIT mode

2023-05-15 Thread Michael Tokarev
26.02.2023 04:58, Damien Zammit wrote: Currently, the one-shot (mode 1) PIT expires far too quickly, due to the output being set under the wrong logic. This change fixes the one-shot PIT mode to behave similarly to mode 0. TESTED: using the one-shot PIT mode to calibrate a local apic timer. Ha

Re: [PATCH qemu] timer/i8254: Fix one shot PIT mode

2023-02-26 Thread Michael S. Tsirkin
On Sun, Feb 26, 2023 at 01:11:19PM +0100, BALATON Zoltan wrote: > On Sun, 26 Feb 2023, Max Filippov wrote: > > On Sun, Feb 26, 2023 at 1:18 AM Damien Zammit wrote: > > > > > > Hi Michael, > > > > > > Thanks for reviewing this on a weekend! > > > > > > On 26/2/23 19:51, Michael S. Tsirkin wrote:

Re: [PATCH qemu] timer/i8254: Fix one shot PIT mode

2023-02-26 Thread BALATON Zoltan
On Sun, 26 Feb 2023, Max Filippov wrote: On Sun, Feb 26, 2023 at 1:18 AM Damien Zammit wrote: Hi Michael, Thanks for reviewing this on a weekend! On 26/2/23 19:51, Michael S. Tsirkin wrote: On Sun, Feb 26, 2023 at 01:58:10AM +, Damien Zammit wrote: case 0: -out = (d >= s-

Re: [PATCH qemu] timer/i8254: Fix one shot PIT mode

2023-02-26 Thread Max Filippov
On Sun, Feb 26, 2023 at 1:18 AM Damien Zammit wrote: > > Hi Michael, > > Thanks for reviewing this on a weekend! > > On 26/2/23 19:51, Michael S. Tsirkin wrote: > > On Sun, Feb 26, 2023 at 01:58:10AM +, Damien Zammit wrote: > >> case 0: > >> -out = (d >= s->count); > >> -

Re: [PATCH qemu] timer/i8254: Fix one shot PIT mode

2023-02-26 Thread Damien Zammit
Hi Michael, Thanks for reviewing this on a weekend! On 26/2/23 19:51, Michael S. Tsirkin wrote: > On Sun, Feb 26, 2023 at 01:58:10AM +, Damien Zammit wrote: >> case 0: >> -out = (d >= s->count); >> -break; > > > I think you need something like > /* FALLTHRU */ > he

Re: [PATCH qemu] timer/i8254: Fix one shot PIT mode

2023-02-26 Thread Michael S. Tsirkin
On Sun, Feb 26, 2023 at 01:58:10AM +, Damien Zammit wrote: > Currently, the one-shot (mode 1) PIT expires far too quickly, > due to the output being set under the wrong logic. > This change fixes the one-shot PIT mode to behave similarly to mode 0. > > TESTED: using the one-shot PIT mode to ca

[PATCH qemu] timer/i8254: Fix one shot PIT mode

2023-02-25 Thread Damien Zammit
Currently, the one-shot (mode 1) PIT expires far too quickly, due to the output being set under the wrong logic. This change fixes the one-shot PIT mode to behave similarly to mode 0. TESTED: using the one-shot PIT mode to calibrate a local apic timer. Signed-off-by: Damien Zammit --- hw/timer