Re: [SeaBIOS] [Qemu-devel] [PATCH v3 3/3] tests: add testcase for TCO watchdog emulation

[Paulo Alcantara]

On Wed, 17 Jun 2015 15:37:49 +0200
"Michael S. Tsirkin"  wrote:
> On Mon, Jun 01, 2015 at 08:48:41PM -0300, Paulo Alcantara wrote:
> > v1 -> v2:
> > some cleanup
> > add test for TCO_LOCK bit
> >
> > v2 -> v3:
> > add tests for TCO control & status bits
> > fix check of SECOND_TO_STS bit (it's set in TCO2_STS reg)
>
> Changelog after -- please.
> Pls add a bit of description here.

Ok.
> > Signed-off-by: Paulo Alcantara
> >
> > tests/Makefile   |   2 +
> >  tests/tco-test.c | 475
> > +++++++++++++++++++++++++++++++++++++++++++++++++++++++ 2 files
> > changed, 477 insertions(+) create mode 100644 tests/tco-test.c
> >
> > diff --git a/tests/Makefile b/tests/Makefile
> > index 729b969..43950d0 100644
> > --- a/tests/Makefile
> > +++ b/tests/Makefile
> > @@ -150,6 +150,7 @@ check-qtest-i386-y += tests/i440fx-test$(EXESUF)
> >  check-qtest-i386-y += tests/fw_cfg-test$(EXESUF)
> >  check-qtest-i386-y += tests/drive_del-test$(EXESUF)
> >  check-qtest-i386-y += tests/wdt_ib700-test$(EXESUF)
> > +check-qtest-i386-y += tests/tco-test$(EXESUF)
> >  gcov-files-i386-y += hw/watchdog/watchdog.c hw/watchdog/wdt_ib700.c
> >  check-qtest-i386-y += $(check-qtest-pci-y)
> >  gcov-files-i386-y += $(gcov-files-pci-y)
> > @@ -363,6 +364,7 @@ tests/eepro100-test$(EXESUF):
> > tests/eepro100-test.o tests/vmxnet3-test$(EXESUF):
> > tests/vmxnet3-test.o tests/ne2000-test$(EXESUF): tests/ne2000-test.o
> >  tests/wdt_ib700-test$(EXESUF): tests/wdt_ib700-test.o
> > +tests/tco-test$(EXESUF): tests/tco-test.o $(libqos-pc-obj-y)
> >  tests/virtio-balloon-test$(EXESUF): tests/virtio-balloon-test.o
> >  tests/virtio-blk-test$(EXESUF): tests/virtio-blk-test.o
> > $(libqos-virtio-obj-y) tests/virtio-net-test$(EXESUF):
> > tests/virtio-net-test.o $(libqos-pc-obj-y) diff --git
> > a/tests/tco-test.c b/tests/tco-test.c new file mode 100644
> > index 0000000..b9acf43
> > --- /dev/null
> > +++ b/tests/tco-test.c
> > @@ -0,0 +1,475 @@
> > +/*
> > QEMU ICH9 TCO emulation tests
> > Copyright (c) 2015 Paulo Alcantara
> > Permission is hereby granted, free of charge, to any person
> >
> > obtaining a copy
> > of this software and associated documentation files (the
> >
> > "Software"), to deal
> > in the Software without restriction, including without
> >
> > limitation the rights
> > to use, copy, modify, merge, publish, distribute, sublicense,
> >
> > and/or sell
> > copies of the Software, and to permit persons to whom the
> >
> > Software is
> > furnished to do so, subject to the following conditions:
> > The above copyright notice and this permission notice shall be
> >
> > included in
> > all copies or substantial portions of the Software.
> > THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
> >
> > EXPRESS OR
> > IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
> >
> > MERCHANTABILITY,
> > FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO
> >
> > EVENT SHALL
> > THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM,
> >
> > DAMAGES OR OTHER
> > LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE,
> >
> > ARISING FROM,
> > OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER
> >
> > DEALINGS IN
> > THE SOFTWARE.
> > */
> >
> > +#include <glib.h>
> > +#include <string.h>
> > +#include <stdio.h>
> > +#include <stdlib.h>
> >
> > +#include "libqtest.h"
> > +#include "libqos/pci.h"
> > +#include "libqos/pci-pc.h"
> > +#include "hw/pci/pci_regs.h"
> > +#include "hw/i386/ich9.h"
> > +#include "hw/acpi/ich9.h"
> > +#include "hw/acpi/tco.h"
> >
> > +#define PM_IO_BASE_ADDR    0xb000
> > +#define RCBA_BASE_ADDR     0xfed1c000
>
> This is duplicated in DSL. Add a macro?

Ok. I will add a macro for RCBA address.

Thanks,

Paulo