[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 02/14] add basic user-space tests with qemu
From: |
Samuel Thibault |
Subject: |
Re: [PATCH 02/14] add basic user-space tests with qemu |
Date: |
Fri, 29 Dec 2023 14:57:08 +0100 |
User-agent: |
NeoMutt/20170609 (1.8.3) |
Luca Dariz, le ven. 29 déc. 2023 14:51:31 +0100, a ecrit:
> Il 29/12/23 14:37, Samuel Thibault ha scritto:
> > Luca Dariz, le jeu. 28 déc. 2023 20:42:49 +0100, a ecrit:
> > > new file mode 100644
> > > index 00000000..4cf25891
> > > --- /dev/null
> > > +++ b/tests/README
> > > @@ -0,0 +1,37 @@
> > > +
> > > +There are some basic tests that can be run qith qemu. You can run all
> > > the tests with
> > > +
> > > + $ make check
> > > +
> > > +or selectively with:
> > > +
> > > + $ make run-hello
> > > +
> > > +Also, you can debug the existing tests, or a new one, by starting on one
> > > shell
> > > +
> > > + $ make debug-hello
> > > +
> > > +and on another shell you can attach with gdb, load the symbols of the
> > > +bootstrap module and break on its _start():
> > > +
> > > + $ gdb gnuamch
> >
> > Typo ;)
>
> What would be a better command?
The command is fine, I just mean the typo gnuamch -> gnumach :)
> > > +++ b/tests/run-qemu.sh.template
> > > +++ b/tests/test-hello.c
> > > +++ b/tests/user-qemu.mk
> >
> > These are non-trivial, and so definitely need a copyright header.
>
> Right, I forgot to add it to all files... I'll resend the patch set
Perhaps wait for my reviews on the other patchs.
> > > diff --git a/tests/syscalls.S b/tests/syscalls.S
> > > new file mode 100644
> > > index 00000000..df9c9bc0
> > > --- /dev/null
> > > +++ b/tests/syscalls.S
> > > @@ -0,0 +1,4 @@
> > > +
> >
> > Spurious line?
>
> I'll add the copyright header here also
>
> > > + #include <mach/syscall_sw.h>
> > > +
> > > + kernel_trap(invalid_syscall,-31,0)
For such a trivial content it's not really useful.
Samuel
[PATCH 08/14] add thread creation helper to tests, Luca Dariz, 2023/12/28
[PATCH 03/14] add mach_host tests, Luca Dariz, 2023/12/28
[PATCH 13/14] add basic thread tests, Luca Dariz, 2023/12/28
[PATCH 01/14] USER32: change default to disabled and make it a general option, Luca Dariz, 2023/12/28