Re: CVS commit: src/tests/bin/sh

2023-12-29 Thread Andrius V
On Thu, Dec 28, 2023 at 11:08 PM Robert Elz wrote: > > [I could claim that the typo was deliberate, as part of > the test but that would be kind of absurd, sh does > no spell checking to test.] > > kre > Thanks for clarification. I definitely had few instances when I needed to revert

Re: CVS commit: src/tests/bin/sh

2023-12-28 Thread Robert Elz
Date:Thu, 28 Dec 2023 20:04:11 + From:"Andrius Varanavicius" Message-ID: <20231228200411.283ccf...@cvs.netbsd.org> | Modified Files: | src/tests/bin/sh: t_syntax.sh | Log Message: | s/synax/syntax/ in test description. Thanks for the correction, but,

Re: CVS commit: src/tests/sbin/ifconfig

2023-10-18 Thread Rin Okuyama
On 2023/10/18 17:25, Rin Okuyama wrote: Module Name:src Committed By: rin Date: Wed Oct 18 08:25:14 UTC 2023 Modified Files: src/tests/sbin/ifconfig: t_capabilities.sh Log Message: ifconfig/t_capabilities: Skip unless run_unsafe is configured to yes The test modifies

Re: CVS commit: src/tests/lib/libc/sys

2022-08-01 Thread Robert Elz
Date:Mon, 1 Aug 2022 18:55:15 +0300 From:Valery Ushakov Message-ID: | The test uses __clone(), not clone() and a followup fix made __clone() | visible under plain _NETBSD_SOURCE again - which is the right thing, | IMO. So this change is not necessary (the

Re: CVS commit: src/tests/lib/libc/sys

2022-08-01 Thread Valery Ushakov
On Mon, Aug 01, 2022 at 15:48:40 +, Robert Elz wrote: > Module Name: src > Committed By: kre > Date: Mon Aug 1 15:48:40 UTC 2022 > > Modified Files: > src/tests/lib/libc/sys: Makefile > > Log Message: > Provide _GNU_SOURCE for t_clone now that is required to make clone() >

Re: CVS commit: src/tests/libexec/ld.elf_so

2022-06-21 Thread Nick Hudson
On 21/06/2022 17:24, Christos Zoulas wrote: Module Name:src Committed By: christos Date: Tue Jun 21 16:24:37 UTC 2022 Modified Files: src/tests/libexec/ld.elf_so: t_ifunc.c Log Message: sort; it is the same list as in h_ifunc_static.c; perhaps it should be a HAVE_

Re: CVS commit: src/tests/fs/vfs

2022-02-05 Thread Robert Elz
Date:Sat, 5 Feb 2022 22:20:16 + From:David Brownlee Message-ID: | Oops, my earliest unix experience was on a BSD4.3 variant, so I was | spoiled by ffs and didn't realise the (in this context) helpful v7fs | behaviour with overlong filename components. To

Re: CVS commit: src/tests/fs/vfs

2022-02-05 Thread David Brownlee
On Wed, 2 Feb 2022 at 17:24, Robert Elz wrote: > > Date:Wed, 2 Feb 2022 15:26:21 + > From:David Brownlee > Message-ID: > > > | So, we just need an optional flag when mounting v7fs to truncate any > | looked up filename component to 14 characters > > That's

Re: CVS commit: src/tests/fs/vfs

2022-02-02 Thread Robert Elz
Date:Wed, 2 Feb 2022 15:26:21 + From:David Brownlee Message-ID: | So, we just need an optional flag when mounting v7fs to truncate any | looked up filename component to 14 characters That's not, or shouldn't be, necessary - that always happened, the limit

Re: CVS commit: src/tests/fs/vfs

2022-02-02 Thread Jason Thorpe
> On Feb 2, 2022, at 6:47 AM, Robert Elz wrote: > >Date:Wed, 2 Feb 2022 07:11:45 + >From:David Holland >Message-ID: > > | v7fs isn't a compat interface for old users, > > That's sad, I could do with something just for me! > > | it's a compat interface

Re: CVS commit: src/tests/fs/vfs

2022-02-02 Thread Michael
Hello, On Wed, 02 Feb 2022 21:47:25 +0700 Robert Elz wrote: > So maybe it is for us truly old fogies (can we have v6fs as well? Well, there is this thing... https://github.com/jaylogue/retro-fuse A user-space filesystem (FUSE) for accessing ancient Unix filesystems. retro-fuse provides a way

Re: CVS commit: src/tests/fs/vfs

2022-02-02 Thread David Brownlee
On Wed, 2 Feb 2022 at 14:47, Robert Elz wrote: > > Date:Wed, 2 Feb 2022 07:11:45 + > From:David Holland > Message-ID: > > | v7fs isn't a compat interface for old users, > > That's sad, I could do with something just for me! Sounds like we need a compat_kre(8)

Re: CVS commit: src/tests/fs/vfs

2022-02-02 Thread Robert Elz
Date:Wed, 2 Feb 2022 07:11:45 + From:David Holland Message-ID: | v7fs isn't a compat interface for old users, That's sad, I could do with something just for me! | it's a compat interface for old disk images :-) And makefs -t v7fs fits into that purpose

Re: CVS commit: src/tests/fs/vfs

2022-02-01 Thread David Holland
On Wed, Feb 02, 2022 at 05:43:45AM +0700, Robert Elz wrote: > | Test mkdir(2) with one or more trailing slashes - this currently fails > | for v7fs. > > As it should I think, trailing slashes are not simply deleted in v7fs. > > [...] > > If this was ever changed, it would not truly

Re: CVS commit: src/tests/fs/vfs

2022-02-01 Thread Robert Elz
Date:Tue, 1 Feb 2022 18:27:24 + From:"Martin Husemann" Message-ID: <20220201182724.90f82f...@cvs.netbsd.org> | Test mkdir(2) with one or more trailing slashes - this currently fails | for v7fs. As it should I think, trailing slashes are not simply deleted in

Re: CVS commit: src/tests/lib/libcurses

2022-01-25 Thread Martin Husemann
On Tue, Jan 25, 2022 at 03:23:05AM +, Brett Lymn wrote: > Module Name: src > Committed By: blymn > Date: Tue Jan 25 03:23:05 UTC 2022 > > Modified Files: > src/tests/lib/libcurses: debug_test t_curses.sh > src/tests/lib/libcurses/check_files: add_wch3.chk get_wstr.chk >

re: CVS commit: src/tests/lib/csu

2021-06-07 Thread matthew green
Christos Zoulas writes: > In article , Joerg Sonnenberger > wrote: > >On Fri, Jun 04, 2021 at 10:20:24PM -, Christos Zoulas wrote: > >> In article , Joerg Sonnenberger > > wrote: > >> >On Thu, Jun 03, 2021 at 04:17:37PM -0400, Christos Zoulas wrote: > >> >> Module Name:src > >> >>

Re: CVS commit: src/tests/lib/csu

2021-06-06 Thread Christos Zoulas
In article , Joerg Sonnenberger wrote: >On Fri, Jun 04, 2021 at 10:20:24PM -, Christos Zoulas wrote: >> In article , Joerg Sonnenberger > wrote: >> >On Thu, Jun 03, 2021 at 04:17:37PM -0400, Christos Zoulas wrote: >> >> Module Name: src >> >> Committed By: christos >> >> Date:

Re: CVS commit: src/tests/lib/csu

2021-06-05 Thread Joerg Sonnenberger
On Fri, Jun 04, 2021 at 10:20:24PM -, Christos Zoulas wrote: > In article , Joerg Sonnenberger > wrote: > >On Thu, Jun 03, 2021 at 04:17:37PM -0400, Christos Zoulas wrote: > >> Module Name: src > >> Committed By: christos > >> Date: Thu Jun 3 20:17:37 UTC 2021 > >>

Re: CVS commit: src/tests/lib/csu

2021-06-04 Thread Christos Zoulas
In article , Joerg Sonnenberger wrote: >On Thu, Jun 03, 2021 at 04:17:37PM -0400, Christos Zoulas wrote: >> Module Name: src >> Committed By:christos >> Date:Thu Jun 3 20:17:37 UTC 2021 >> >> Modified Files: >> src/tests/lib/csu: h_ifunc_static.c >> >> Log

Re: CVS commit: src/tests/lib/csu

2021-06-03 Thread Joerg Sonnenberger
On Thu, Jun 03, 2021 at 04:17:37PM -0400, Christos Zoulas wrote: > Module Name: src > Committed By: christos > Date: Thu Jun 3 20:17:37 UTC 2021 > > Modified Files: > src/tests/lib/csu: h_ifunc_static.c > > Log Message: > PR/56230: Jan-Benedict Glaw: arm oabi does not and will

Re: CVS commit: src/tests/lib/libcurses/director

2021-04-05 Thread Valery Ushakov
On Tue, Apr 06, 2021 at 00:47:00 +, Roland Illig wrote: > The previous "table" was an insult to any reader. It was unsorted, > listed the functions shuffled, and was not even formatted consistently. That's pretty rude. Please, tone down your commit "messages". -uwe

Re: GCC TSan (Re: CVS commit: src/tests/usr.bin)

2020-09-15 Thread Martin Husemann
On Tue, Sep 15, 2020 at 03:32:25PM +0200, Kamil Rytarowski wrote: > I've tried to mark the TSan parts that need porting as explicit failure, > soo we can reduce the risk of shipping unported runtime. That risc is quite low as currently the runtime is apparently not buildable on anything but amd64

GCC TSan (Re: CVS commit: src/tests/usr.bin)

2020-09-15 Thread Kamil Rytarowski
On 15.09.2020 07:03, Martin Husemann wrote: > On Mon, Sep 14, 2020 at 03:17:53PM +, Kamil Rytarowski wrote: >> Enable TSan tests for GCC and >32bit address space environments > > Since tsan does not work on all architectures, this is not a good idea. > It would be better to code it with an

Re: CVS commit: src/tests/usr.bin

2020-09-14 Thread Martin Husemann
On Mon, Sep 14, 2020 at 03:17:53PM +, Kamil Rytarowski wrote: > Enable TSan tests for GCC and >32bit address space environments Since tsan does not work on all architectures, this is not a good idea. It would be better to code it with an explicit list of architectures supported. Martin

Re: CVS commit: src/tests/lib/libc/stdlib

2020-06-27 Thread Jukka Ruohonen
On Sat, Jun 27, 2020 at 10:39:08AM +, m...@netbsd.org wrote: > > Add the default TNF copyright (2005), cf. PR misc/55419. > > I don't think we can generally do this. Can you clarify if you discussed > this with the author in commit messages? Well, the committer is christos, and I doubt he

Re: CVS commit: src/tests/lib/libc/stdlib

2020-06-27 Thread maya
On Sat, Jun 27, 2020 at 10:19:43AM +, Jukka Ruohonen wrote: > Module Name: src > Committed By: jruoho > Date: Sat Jun 27 10:19:43 UTC 2020 > > Modified Files: > src/tests/lib/libc/stdlib: t_mbtowc.c > > Log Message: > Add the default TNF copyright (2005), cf. PR misc/55419. >

Re: CVS commit: src/tests/usr.sbin

2020-06-24 Thread Jukka Ruohonen
On Wed, Jun 24, 2020 at 03:05:45PM +, Martin Husemann wrote: > Modified Files: > src/tests/usr.sbin/stdethers: Makefile > src/tests/usr.sbin/stdhosts: Makefile > > Log Message: > Add input files Ups. I try to be more careful, now that I again remember the nitty-gritties. - Jukka

Re: CVS commit: src/tests/lib/libc/sys

2020-06-22 Thread Rin Okuyama
Committed. Thank you for your comment! rin On 2020/06/22 20:09, Kamil Rytarowski wrote: On 22.06.2020 04:51, Rin Okuyama wrote: Module Name:src Committed By: rin Date: Mon Jun 22 02:51:07 UTC 2020 Modified Files: src/tests/lib/libc/sys: t_ptrace_signal_wait.h

Re: CVS commit: src/tests/lib/libc/sys

2020-06-22 Thread Kamil Rytarowski
On 22.06.2020 04:51, Rin Okuyama wrote: > Module Name: src > Committed By: rin > Date: Mon Jun 22 02:51:07 UTC 2020 > > Modified Files: > src/tests/lib/libc/sys: t_ptrace_signal_wait.h t_ptrace_wait.h > > Log Message: > Turn trigger_fpe() back to integer division by zero for a

Re: CVS commit: src/tests/lib/libc/sys

2020-06-17 Thread Rin Okuyama
On 2020/06/17 17:42, Rin Okuyama wrote: Now, all *_crash_fpe tests pass for powerpc, and nothing changes for amd64 at least. Here, powerpc means powerpc/oea, more specifically Mac mini G4. At the moment, fenv.h doesn't work correctly on booke and ibm4xx, where FPU is emulated in software. For

Re: CVS commit: src/tests/lib/libarchive

2020-06-16 Thread Paul Goyette
On Tue, 16 Jun 2020, Martin Husemann wrote: On Tue, Jun 16, 2020 at 09:12:40AM -0700, Paul Goyette wrote: It might be better to run the test in a rump-kernel rather than in a "live" environment The test this is about is a plain userland test: it extracts/compresses/decompresses various

Re: CVS commit: src/tests/lib/libarchive

2020-06-16 Thread Martin Husemann
On Tue, Jun 16, 2020 at 09:12:40AM -0700, Paul Goyette wrote: > It might be better to run the test in a rump-kernel rather than in a > "live" environment The test this is about is a plain userland test: it extracts/compresses/decompresses various archive formats and compares results. Only

Re: CVS commit: src/tests/lib/libarchive

2020-06-16 Thread Paul Goyette
On Tue, 16 Jun 2020, Greg Troxel wrote: Jason Thorpe writes: On Jun 16, 2020, at 8:43 AM, Martin Husemann wrote: No, that is definitively not OK, which is what the PR is about. It is not OK for a regular atf run to cause a reboot of the test machine though, so this is a temporary hack

Re: CVS commit: src/tests/lib/libarchive

2020-06-16 Thread Greg Troxel
Jason Thorpe writes: >> On Jun 16, 2020, at 8:43 AM, Martin Husemann wrote: >> >> No, that is definitively not OK, which is what the PR is about. >> >> It is not OK for a regular atf run to cause a reboot of the test machine >> though, so this is a temporary hack around the issue (and

Re: CVS commit: src/tests/lib/libarchive

2020-06-16 Thread Jason Thorpe
> On Jun 16, 2020, at 8:43 AM, Martin Husemann wrote: > > No, that is definitively not OK, which is what the PR is about. > > It is not OK for a regular atf run to cause a reboot of the test machine > though, so this is a temporary hack around the issue (and admitedly a very > ugly hack).

Re: CVS commit: src/tests/lib/libarchive

2020-06-16 Thread Martin Husemann
On Tue, Jun 16, 2020 at 03:38:26PM -, Christos Zoulas wrote: > So we are saying that it is ok for process running with regular priority, > to be able to starve another process at the same priority from getting > any runtime for 21 seconds in a uniprocessor kernel, and this does not > indicate

Re: CVS commit: src/tests/lib/libarchive

2020-06-16 Thread Christos Zoulas
In article <20200616075907.ecafcf...@cvs.netbsd.org>, Martin Husemann wrote: >-=-=-=-=-=- > >Module Name: src >Committed By: martin >Date: Tue Jun 16 07:59:07 UTC 2020 > >Modified Files: > src/tests/lib/libarchive: t_libarchive.sh > >Log Message: >PR kern/55272: skip this test

Re: CVS commit: src/tests/lib/libc/sys

2020-06-06 Thread Andrew Doran
On Sat, Jun 06, 2020 at 06:11:21PM +, Jason R Thorpe wrote: > Module Name: src > Committed By: thorpej > Date: Sat Jun 6 18:11:21 UTC 2020 > > Modified Files: > src/tests/lib/libc/sys: t_lwp_create.c > > Log Message: > Add a test case to ensure that _lwp_create() fails with

Re: CVS commit: src/tests/rump/modautoload

2020-05-16 Thread Kamil Rytarowski
I will test this with ASan and report back! On 16.05.2020 16:15, Christos Zoulas wrote: > That is a completely different issue here. There are no linker tricks. > We want the module loader to include all the symbols any module > can require, this is why we load all the libraries. > > While it is

Re: CVS commit: src/tests/rump/modautoload

2020-05-16 Thread Christos Zoulas
That is a completely different issue here. There are no linker tricks. We want the module loader to include all the symbols any module can require, this is why we load all the libraries. While it is questionable if nofifofs is part of the base system or not, this is the way it was before. Anyway

Re: CVS commit: src/tests/rump/modautoload

2020-05-16 Thread Kamil Rytarowski
On 16.05.2020 14:54, Christos Zoulas wrote: > Module Name: src > Committed By: christos > Date: Sat May 16 12:54:27 UTC 2020 > > Modified Files: > src/tests/rump/modautoload: Makefile > > Log Message: > Do the same thing with linker flags instead of directly specifying the >

Re: CVS commit: src/tests/lib/libc/sys

2020-05-15 Thread Kamil Rytarowski
On 15.05.2020 00:43, Taylor R Campbell wrote: >> Date: Thu, 14 May 2020 23:36:28 +0200 >> From: Kamil Rytarowski >> >> If a signal would not reach the child process (as it is ignored or >> masked+SA_IGNOREd) and it is not a crash signal, it is dropped. As I >> checked, it's the design in UNIX to

Re: CVS commit: src/tests/lib/libc/sys

2020-05-14 Thread Taylor R Campbell
> Date: Thu, 14 May 2020 23:36:28 +0200 > From: Kamil Rytarowski > > If a signal would not reach the child process (as it is ignored or > masked+SA_IGNOREd) and it is not a crash signal, it is dropped. As I > checked, it's the design in UNIX to overlook SIGCHLD signals in UNIX. > Race free

Re: CVS commit: src/tests/lib/libc/sys

2020-05-14 Thread Joerg Sonnenberger
On Thu, May 14, 2020 at 11:36:28PM +0200, Kamil Rytarowski wrote: > On 14.05.2020 23:02, Joerg Sonnenberger wrote: > > On Thu, May 14, 2020 at 07:21:35PM +, Kamil Rytarowski wrote: > >> Module Name: src > >> Committed By: kamil > >> Date: Thu May 14 19:21:35 UTC 2020 >

Re: CVS commit: src/tests/lib/libc/sys

2020-05-14 Thread Kamil Rytarowski
On 14.05.2020 23:02, Joerg Sonnenberger wrote: > On Thu, May 14, 2020 at 07:21:35PM +, Kamil Rytarowski wrote: >> Module Name: src >> Committed By:kamil >> Date:Thu May 14 19:21:35 UTC 2020 >> >> Modified Files: >> src/tests/lib/libc/sys: t_ptrace_fork_wait.h >> >>

Re: CVS commit: src/tests/lib/libc/sys

2020-05-14 Thread Joerg Sonnenberger
On Thu, May 14, 2020 at 07:21:35PM +, Kamil Rytarowski wrote: > Module Name: src > Committed By: kamil > Date: Thu May 14 19:21:35 UTC 2020 > > Modified Files: > src/tests/lib/libc/sys: t_ptrace_fork_wait.h > > Log Message: > Ignore interception of the SIGCHLD signals. > >

Re: CVS commit: src/tests/lib/libc/sys

2020-05-11 Thread Robert Elz
Date:Mon, 11 May 2020 13:47:45 +0200 From:Kamil Rytarowski Message-ID: <54178983-82d1-df3d-fd54-549a6c73f...@gmx.com> | The only purpose of the test is to check whether misaligned program | counter can crash the kernel (it can for NetBSD/sparc). Later, if a |

Re: CVS commit: src/tests/lib/libc/sys

2020-05-11 Thread Kamil Rytarowski
On 11.05.2020 13:35, Robert Elz wrote: > Date:Mon, 11 May 2020 11:03:15 + > From:"Kamil Rytarowski" > Message-ID: <2020050315.54b13f...@cvs.netbsd.org> > > | Do not fail when trying to kill a dying process > | > | A dying process can disappear for a

Re: CVS commit: src/tests/lib/libc/sys

2020-05-11 Thread Robert Elz
Date:Mon, 11 May 2020 11:03:15 + From:"Kamil Rytarowski" Message-ID: <2020050315.54b13f...@cvs.netbsd.org> | Do not fail when trying to kill a dying process | | A dying process can disappear for a while. Rather than aborting, retry | sending SIGKILL

Re: CVS commit: src/tests/lib/libc/sys

2020-04-24 Thread Kamil Rytarowski
On 24.04.2020 05:25, Jason R Thorpe wrote: > Module Name: src > Committed By: thorpej > Date: Fri Apr 24 03:25:20 UTC 2020 > > Modified Files: > src/tests/lib/libc/sys: t_ptrace_wait.c t_ptrace_x86_wait.h > > Log Message: > Update for new LWP behavior -- as of 9.99.59, the LWP ID

Re: CVS commit: src/tests/lib/libc/sys

2020-03-07 Thread Christos Zoulas
In article <5e528f7a-147a-23e7-46da-6b02d76e5...@gmx.com>, Kamil Rytarowski wrote: >-=-=-=-=-=- >-=-=-=-=-=- > >On 07.03.2020 15:53, Christos Zoulas wrote: >> Module Name: src >> Committed By:christos >> Date:Sat Mar 7 14:53:14 UTC 2020 >> >> Modified Files: >>

Re: CVS commit: src/tests/lib/libc/sys

2020-03-07 Thread Kamil Rytarowski
On 07.03.2020 15:53, Christos Zoulas wrote: > Module Name: src > Committed By: christos > Date: Sat Mar 7 14:53:14 UTC 2020 > > Modified Files: > src/tests/lib/libc/sys: t_ptrace_wait.c t_ptrace_wait.h > > Log Message: > Try to fix the build. This is why all those inlines should

Re: CVS commit: src/tests/lib/libc/gen

2020-02-24 Thread Kamil Rytarowski
On 24.02.2020 20:32, Christos Zoulas wrote: > In article <20200221222550.325a6f...@cvs.netbsd.org>, > Kamil Rytarowski wrote: >> -=-=-=-=-=- >> >> Module Name: src >> Committed By:kamil >> Date:Fri Feb 21 22:25:50 UTC 2020 >> >> Modified Files: >>

Re: CVS commit: src/tests/lib/libc/gen

2020-02-24 Thread Christos Zoulas
In article <20200221222550.325a6f...@cvs.netbsd.org>, Kamil Rytarowski wrote: >-=-=-=-=-=- > >Module Name: src >Committed By: kamil >Date: Fri Feb 21 22:25:50 UTC 2020 > >Modified Files: > src/tests/lib/libc/gen: t_siginfo.c > >Log Message: >Mark division by 0 as expected in

Re: CVS commit: src/tests/lib/libc/gen

2020-02-24 Thread Kamil Rytarowski
On 24.02.2020 20:29, Christos Zoulas wrote: > In article <20200222191457.87687f...@cvs.netbsd.org>, > Kamil Rytarowski wrote: >> -=-=-=-=-=- >> >> Module Name: src >> Committed By:kamil >> Date:Sat Feb 22 19:14:57 UTC 2020 >> >> Modified Files: >>

Re: CVS commit: src/tests/lib/libc/gen

2020-02-24 Thread Christos Zoulas
In article <20200222191457.87687f...@cvs.netbsd.org>, Kamil Rytarowski wrote: >-=-=-=-=-=- > >Module Name: src >Committed By: kamil >Date: Sat Feb 22 19:14:57 UTC 2020 > >Modified Files: > src/tests/lib/libc/gen: Makefile > >Log Message: >Update t_siginfo.c build rules > >Add

Re: CVS commit: src/tests/modules

2020-02-22 Thread Kamil Rytarowski
On 22.02.2020 15:54, Paul Goyette wrote: > On Sat, 22 Feb 2020, Kamil Rytarowski wrote: > > While there, it would be good to implement modctl(MODCTL_MODSTAT, > ) to check whether a specific module is loaded into the kernel > and retrieve modstat_t describing it. > > modstat_t

Re: CVS commit: src/tests/modules

2020-02-22 Thread Paul Goyette
On Sat, 22 Feb 2020, Kamil Rytarowski wrote: While there, it would be good to implement modctl(MODCTL_MODSTAT, ) to check whether a specific module is loaded into the kernel and retrieve modstat_t describing it. modstat_t m; strlcpy(_name, "haxm", MAXMODNAME); if (modctl(MODCTL_MODSTAT, ) ==

Re: CVS commit: src/tests/modules

2020-02-22 Thread Kamil Rytarowski
On 22.02.2020 15:32, Paul Goyette wrote: > On Sat, 22 Feb 2020, Paul Goyette wrote: > >>> While there, it would be good to implement modctl(MODCTL_MODSTAT, >>> ) to check whether a specific module is loaded into the kernel >>> and retrieve modstat_t describing it. >>> >>> modstat_t m; >>>

Re: CVS commit: src/tests/modules

2020-02-22 Thread Paul Goyette
On Sat, 22 Feb 2020, Paul Goyette wrote: While there, it would be good to implement modctl(MODCTL_MODSTAT, ) to check whether a specific module is loaded into the kernel and retrieve modstat_t describing it. modstat_t m; strlcpy(_name, "haxm", MAXMODNAME); if (modctl(MODCTL_MODSTAT, ) == -1)

Re: CVS commit: src/tests/modules

2020-02-22 Thread Paul Goyette
On Sat, 22 Feb 2020, Kamil Rytarowski wrote: I have got no opinion. Please rearrange the directories as needed. It's too much bother for now to move things around. But for future changes it would be good to put new "helper" modules in the same area as the tests being helped. While there,

Re: CVS commit: src/tests/modules

2020-02-22 Thread Kamil Rytarowski
I have got no opinion. Please rearrange the directories as needed. While there, it would be good to implement modctl(MODCTL_MODSTAT, ) to check whether a specific module is loaded into the kernel and retrieve modstat_t describing it. modstat_t m; strlcpy(_name, "haxm", MAXMODNAME); if

Re: CVS commit: src/tests/modules

2020-02-21 Thread Paul Goyette
OK, I over-reacted and didn't completely read the original commit message. The t_builtin.c stuff is indeed a test-of-module-functionality so it does belong here. But some of the other stuff here does not belong, such as the threadpool, fetchstore, and kcov stuff. As far as I can see, those

Re: CVS commit: src/tests/modules

2020-02-21 Thread Paul Goyette
Really, the tests/modules directory should be only used for tests-that- relate-to-module-functionality. It should NOT be used for modules- that-support-tests-of-other-functionality. In the future, please do not put support modules here; put them in the samae place as the tests that they

Re: CVS commit: src/tests/lib/libc/gen

2020-02-21 Thread Kamil Rytarowski
On 21.02.2020 23:53, matthew green wrote: >> Disable ubsan instrumentation on the operation. > > +#if defined(__clang__) > +__attribute__((no_sanitize("undefined"))) > +#else > +__attribute__((no_sanitize_undefined)) > +#endif > > can we get a __disable_sanitizer or something i cdefs.h? > > >

re: CVS commit: src/tests/lib/libc/gen

2020-02-21 Thread matthew green
> Disable ubsan instrumentation on the operation. +#if defined(__clang__) +__attribute__((no_sanitize("undefined"))) +#else +__attribute__((no_sanitize_undefined)) +#endif can we get a __disable_sanitizer or something i cdefs.h? .mrg.

Re: CVS commit: src/tests/lib/libc/sys

2020-02-18 Thread Kamil Rytarowski
On 18.02.2020 16:57, Christos Zoulas wrote: > In article <20200213152742.081a9f...@cvs.netbsd.org>, > MichaŠGórny wrote: >> -=-=-=-=-=- >> >> Module Name: src >> Committed By:mgorny >> Date:Thu Feb 13 15:27:41 UTC 2020 >> >> Modified Files: >>

Re: CVS commit: src/tests/lib/libc/sys

2020-02-18 Thread Christos Zoulas
In article <20200213152742.081a9f...@cvs.netbsd.org>, MichaŠGórny wrote: >-=-=-=-=-=- > >Module Name: src >Committed By: mgorny >Date: Thu Feb 13 15:27:41 UTC 2020 > >Modified Files: > src/tests/lib/libc/sys: t_ptrace_wait.c > >Log Message: >Enable combined breakpoint,

Re: CVS commit: src/tests/lib/libc/sys

2020-02-13 Thread Christos Zoulas
In article <20200213114904.ga30...@bec.de>, Joerg Sonnenberger wrote: >On Thu, Feb 13, 2020 at 10:50:19AM +0100, Joerg Sonnenberger wrote: >> On Wed, Feb 12, 2020 at 09:53:46PM -0500, Christos Zoulas wrote: >> > Module Name: src >> > Committed By: christos >> > Date: Thu

Re: CVS commit: src/tests/lib/libc/sys

2020-02-13 Thread Christos Zoulas
In article <20200213095019.ga28...@bec.de>, Joerg Sonnenberger wrote: >On Wed, Feb 12, 2020 at 09:53:46PM -0500, Christos Zoulas wrote: >> Module Name: src >> Committed By:christos >> Date:Thu Feb 13 02:53:46 UTC 2020 >> >> Modified Files: >> src/tests/lib/libc/sys:

Re: CVS commit: src/tests/lib/libc/sys

2020-02-13 Thread Joerg Sonnenberger
On Thu, Feb 13, 2020 at 10:50:19AM +0100, Joerg Sonnenberger wrote: > On Wed, Feb 12, 2020 at 09:53:46PM -0500, Christos Zoulas wrote: > > Module Name:src > > Committed By: christos > > Date: Thu Feb 13 02:53:46 UTC 2020 > > > > Modified Files: > >

Re: CVS commit: src/tests/lib/libc/sys

2020-02-13 Thread Joerg Sonnenberger
On Wed, Feb 12, 2020 at 09:53:46PM -0500, Christos Zoulas wrote: > Module Name: src > Committed By: christos > Date: Thu Feb 13 02:53:46 UTC 2020 > > Modified Files: > src/tests/lib/libc/sys: t_ptrace_x86_wait.h > > Log Message: > Turn off optimization on a function which contains

re: CVS commit: src/tests/lib/libarchive

2020-01-28 Thread matthew green
"Martin Husemann" writes: > Module Name: src > Committed By: martin > Date: Tue Jan 28 18:18:32 UTC 2020 > > Modified Files: > src/tests/lib/libarchive: t_libarchive.sh > > Log Message: > Bump timeout to 3600 - the libarchive tests take quite a while to > complete (on a nearly 1

Re: CVS commit: src/tests/usr.bin

2019-10-14 Thread Kamil Rytarowski
On 14.10.2019 05:47, Jason High wrote: Module Name:src Committed By: jhigh Date: Mon Oct 14 03:47:20 UTC 2019 Modified Files: src/tests/usr.bin: Makefile Added Files: src/tests/usr.bin/argon2: Atffile Makefile t_argon2.sh Log Message: adding argon2 tests

Re: CVS commit: src/tests/kernel

2019-09-15 Thread Kamil Rytarowski
While there, it does not build for me: /usr/src/tests/kernel/h_fexecve.c: In function 'main': /usr/src/tests/kernel/h_fexecve.c:48:6: error: implicit declaration of function 'fexecve'; did you mean 'execve'? [-Werror=implicit-function-declaration] if (fexecve(fd, args, NULL) == -1)

Re: CVS commit: src/tests/kernel

2019-09-15 Thread Kamil Rytarowski
I have got no opinion, but merging them is good. Personally I prefer src/libc/* path as fexecve(2) is a libc public symbol. On 15.09.2019 20:06, Christos Zoulas wrote: > The tests are a different. Should we keep them both, or try to merge them? > I think that merging them is probably better. It

Re: CVS commit: src/tests/kernel

2019-09-15 Thread Christos Zoulas
The tests are a different. Should we keep them both, or try to merge them? I think that merging them is probably better. It is also the case that perhaps we need to get rid of the kernel tests directory and move them to the respective bin and lib directories to avoid confusion? christos > On Sep

Re: CVS commit: src/tests/kernel

2019-09-15 Thread Kamil Rytarowski
On 15.09.2019 18:53, Christos Zoulas wrote: > Module Name: src > Committed By: christos > Date: Sun Sep 15 16:53:58 UTC 2019 > > Modified Files: > src/tests/kernel: Makefile > Added Files: > src/tests/kernel: h_fexecve.c t_fexecve.sh > > Log Message: > Add tests for

Re: CVS commit: src/tests/net

2019-08-23 Thread Ryota Ozaki
On Fri, Aug 23, 2019 at 3:51 PM Martin Husemann wrote: > > On Fri, Aug 23, 2019 at 01:41:58PM +0900, Ryota Ozaki wrote: > > A workaround for the issue is: > > cp /usr/bin/vmstat ./vmstat > > $HIJACKING ./vmstat > > rm -f ./vmstat > > > > It's awkward but it's reasonable for now. A proper

Re: CVS commit: src/tests/net

2019-08-23 Thread Ryota Ozaki
On Fri, Aug 23, 2019 at 1:53 PM matthew green wrote: > > > It's awkward but it's reasonable for now. A proper fix would > > be to stop using kvm(3) for vmstat and drop the sgid bit from > > the binary. > > please finish this work :-) it's been ongoing for a very > long time now... That task

Re: CVS commit: src/tests/net

2019-08-23 Thread Martin Husemann
On Fri, Aug 23, 2019 at 01:41:58PM +0900, Ryota Ozaki wrote: > A workaround for the issue is: > cp /usr/bin/vmstat ./vmstat > $HIJACKING ./vmstat > rm -f ./vmstat > > It's awkward but it's reasonable for now. A proper fix would > be to stop using kvm(3) for vmstat and drop the sgid bit

re: CVS commit: src/tests/net

2019-08-22 Thread matthew green
> It's awkward but it's reasonable for now. A proper fix would > be to stop using kvm(3) for vmstat and drop the sgid bit from > the binary. please finish this work :-) it's been ongoing for a very long time now... .mrg.

Re: CVS commit: src/tests/net

2019-08-22 Thread Ryota Ozaki
On Thu, Aug 22, 2019 at 5:45 PM Ryota Ozaki wrote: > > On Tue, Aug 20, 2019 at 7:06 PM Ryota Ozaki wrote: > > > > On Tue, Aug 20, 2019 at 6:58 PM Martin Husemann wrote: > > > > > > On Tue, Aug 20, 2019 at 06:50:31PM +0900, Ryota Ozaki wrote: > > > > Hmm, okay, I'm going to disable the feature

Re: CVS commit: src/tests/net

2019-08-22 Thread Ryota Ozaki
On Tue, Aug 20, 2019 at 7:06 PM Ryota Ozaki wrote: > > On Tue, Aug 20, 2019 at 6:58 PM Martin Husemann wrote: > > > > On Tue, Aug 20, 2019 at 06:50:31PM +0900, Ryota Ozaki wrote: > > > Hmm, okay, I'm going to disable the feature until the issue is addressed. > > > > Could it be page size

Re: CVS commit: src/tests/net

2019-08-20 Thread Ryota Ozaki
On Tue, Aug 20, 2019 at 6:58 PM Martin Husemann wrote: > > On Tue, Aug 20, 2019 at 06:50:31PM +0900, Ryota Ozaki wrote: > > Hmm, okay, I'm going to disable the feature until the issue is addressed. > > Could it be page size related? I'm not sure. My fresh chroot environment for ATF tests on

Re: CVS commit: src/tests/net

2019-08-20 Thread Martin Husemann
On Tue, Aug 20, 2019 at 06:50:31PM +0900, Ryota Ozaki wrote: > Hmm, okay, I'm going to disable the feature until the issue is addressed. Could it be page size related? Martin

Re: CVS commit: src/tests/net

2019-08-20 Thread Ryota Ozaki
On Tue, Aug 20, 2019 at 6:35 PM Martin Husemann wrote: > > On Tue, Aug 20, 2019 at 07:53:28AM +0200, Martin Husemann wrote: > > On Tue, Aug 20, 2019 at 11:21:08AM +0900, Ryota Ozaki wrote: > > > Is it an issue specific to sparc64? > > > > No, same happens on (big endian) arm. Endianess issue? > >

Re: CVS commit: src/tests/net

2019-08-20 Thread Martin Husemann
On Tue, Aug 20, 2019 at 07:53:28AM +0200, Martin Husemann wrote: > On Tue, Aug 20, 2019 at 11:21:08AM +0900, Ryota Ozaki wrote: > > Is it an issue specific to sparc64? > > No, same happens on (big endian) arm. Endianess issue? > I'll do a little endian arm run today. Nope, evbarm fails the same:

Re: CVS commit: src/tests/net

2019-08-19 Thread Martin Husemann
On Tue, Aug 20, 2019 at 11:21:08AM +0900, Ryota Ozaki wrote: > Is it an issue specific to sparc64? No, same happens on (big endian) arm. Endianess issue? I'll do a little endian arm run today. Martin

Re: CVS commit: src/tests/net

2019-08-19 Thread Ryota Ozaki
On Mon, Aug 19, 2019 at 10:18 PM Martin Husemann wrote: > > On Mon, Aug 19, 2019 at 03:22:47AM +, Ryota Ozaki wrote: > > Module Name: src > > Committed By: ozaki-r > > Date: Mon Aug 19 03:22:47 UTC 2019 > > > > Modified Files: > > src/tests/net: net_common.sh > > > > Log

Re: CVS commit: src/tests/net

2019-08-19 Thread Martin Husemann
On Mon, Aug 19, 2019 at 03:22:47AM +, Ryota Ozaki wrote: > Module Name: src > Committed By: ozaki-r > Date: Mon Aug 19 03:22:47 UTC 2019 > > Modified Files: > src/tests/net: net_common.sh > > Log Message: > tests: check pool object leaks > > Currently only llentpl leaks can

Re: CVS commit: src/tests/lib/libc/sys

2019-07-01 Thread Kamil Rytarowski
On 01.07.2019 14:59, Robert Elz wrote: > Date:Mon, 1 Jul 2019 11:55:57 +0200 > From:Joerg Sonnenberger > Message-ID: <20190701095557.ga55...@bec.de> > > | On Mon, Jul 01, 2019 at 02:04:38AM +, Kamil Rytarowski wrote: > > | > Cast note_hdr.n_namesz to ssize_t

Re: CVS commit: src/tests/lib/libc/sys

2019-07-01 Thread Robert Elz
Date:Mon, 1 Jul 2019 11:55:57 +0200 From:Joerg Sonnenberger Message-ID: <20190701095557.ga55...@bec.de> | On Mon, Jul 01, 2019 at 02:04:38AM +, Kamil Rytarowski wrote: | > Cast note_hdr.n_namesz to ssize_t through size_t to avoid potential | > signedness

Re: CVS commit: src/tests/lib/libc/sys

2019-07-01 Thread Joerg Sonnenberger
On Mon, Jul 01, 2019 at 02:04:38AM +, Kamil Rytarowski wrote: > Module Name: src > Committed By: kamil > Date: Mon Jul 1 02:04:38 UTC 2019 > > Modified Files: > src/tests/lib/libc/sys: t_ptrace_wait.c > > Log Message: > Avoid GCC warning on NetBSD/i386 > > Cast

Re: CVS commit: src/tests/lib/libcurses

2019-06-26 Thread Roy Marples
On 25/06/2019 23:19, Brett Lymn wrote: Module Name:src Committed By: blymn Date: Tue Jun 25 22:19:29 UTC 2019 Modified Files: src/tests/lib/libcurses: t_curses.sh src/tests/lib/libcurses/tests: mvscanw Log Message: Fixed mvscanw test but leave disabled for the

Re: CVS commit: src/tests/lib/libc/atomic

2019-02-27 Thread Christos Zoulas
On Feb 28, 1:05am, is...@pastel-flower.jp (Tetsuya Isaki) wrote: -- Subject: Re: CVS commit: src/tests/lib/libc/atomic | At Wed, 27 Feb 2019 10:32:11 -0500, | Christos Zoulas wrote: | > Module Name:src | > Committed By: christos | > Date: Wed Feb 27 15:32:11

Re: CVS commit: src/tests/lib/libc/atomic

2019-02-27 Thread Tetsuya Isaki
At Wed, 27 Feb 2019 10:32:11 -0500, Christos Zoulas wrote: > Module Name: src > Committed By: christos > Date: Wed Feb 27 15:32:11 UTC 2019 > > Modified Files: > src/tests/lib/libc/atomic: t___sync_and.c > > Log Message: > Make the _and_and_ have-nots compile. Sorry for build

Re: CVS commit: src/tests/lib/libc/misc

2019-02-04 Thread Kamil Rytarowski
On 04.02.2019 09:50, Robert Elz wrote: > Date:Mon, 4 Feb 2019 05:02:46 +0100 > From:Kamil Rytarowski > Message-ID: <2eadaf71-d7d7-c285-bdec-78ddcd3a5...@gmx.com> > > > | If GCC is fine with it, we could try raise(!!(a * b) ? SIGSEGV : SIGBUS);= > > That's a kind

Re: CVS commit: src/tests/lib/libc/misc

2019-02-04 Thread Robert Elz
Date:Mon, 4 Feb 2019 05:02:46 +0100 From:Kamil Rytarowski Message-ID: <2eadaf71-d7d7-c285-bdec-78ddcd3a5...@gmx.com> | If GCC is fine with it, we could try raise(!!(a * b) ? SIGSEGV : SIGBUS);= That's a kind of odd way of saying (a * b) != 0 ? ... kre

  1   2   3   4   5   >