Re: CVS commit: src/lib/libpthread

2020-01-31 Thread Kamil Rytarowski
On 31.01.2020 19:55, Christos Zoulas wrote: > In article <724af477-010b-9ddf-6ece-e23d7cf59...@gmx.com>, > Kamil Rytarowski wrote: >> -=-=-=-=-=- >> -=-=-=-=-=- >> >> On 31.01.2020 03:38, Christos Zoulas wrote: >>> And it is fixed now. >>>

Re: CVS commit: src/lib/libpthread

2020-02-01 Thread Kamil Rytarowski
On 31.01.2020 22:10, Andrew Doran wrote: > On Fri, Jan 31, 2020 at 06:55:00PM -, Christos Zoulas wrote: > >> In article <724af477-010b-9ddf-6ece-e23d7cf59...@gmx.com>, >> Kamil Rytarowski wrote: >>> -=-=-=-=-=- >>> -=-=-=-=-=- >>> >>

Re: CVS commit: src/lib/libpthread

2020-02-01 Thread Kamil Rytarowski
Actually it happened that modifiying pthread_atfork() to stop malloc()ing is enough to address the problem. I have landed the changes and removed '#if 0' kludge. Thanks! On 01.02.2020 13:59, Kamil Rytarowski wrote: > On 31.01.2020 22:10, Andrew Doran wrote: >> On Fri, Jan 31,

Re: CVS commit: src/lib/libpthread

2020-02-01 Thread Kamil Rytarowski
@@ > { > char buf[1024]; > size_t len; > - > + > if (pthread__diagassert == 0) > return; > > > > "Kamil Rytarowski" writes: > >> Module Name: src >> Committed By:kamil >> Date:

Re: CVS commit: src/lib/libpthread

2020-02-01 Thread Kamil Rytarowski
read", >> t1->pt_magic == PT_MAGIC); >> >> pthread__error(EINVAL, "Invalid thread", >> t2->pt_magic == PT_MAGIC); >> +#endif >> >> /* Nothing special here. */ >> return (t1 == t2); >> @@ -1108,7

Re: CVS commit: src/lib/libpthread

2020-02-02 Thread Kamil Rytarowski
ith a core(5) file. On 01.02.2020 22:20, Kamil Rytarowski wrote: > Good idea. It could be checked quicker... however I presume that > t1->pt_magic + t1->pt_magic already crash on invalid t1/t2 pointers as > the argument with condition is evaluated. > > Ryo, you might check: &

Re: CVS commit: src/lib/libpthread

2020-02-03 Thread Kamil Rytarowski
Please check this workaround: http://netbsd.org/~kamil/patch-00224-firefox-pthread_equal.txt It has to be applied on firefox's package. The problem has to be reported upstream as a real bug. On 03.02.2020 16:01, Ryo ONODERA wrote: > Hi, > > Ryo ONODERA writes: > >> Hi, >> >> I had tested wit

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

2020-02-03 Thread Kamil Rytarowski
On 03.02.2020 17:21, Joerg Sonnenberger wrote: > On Sat, Feb 01, 2020 at 03:38:46PM +0000, Kamil Rytarowski wrote: >> Module Name: src >> Committed By:kamil >> Date:Sat Feb 1 15:38:46 UTC 2020 >> >> Modified Files: >> src/l

Re: CVS commit: src/lib/libpthread

2020-02-03 Thread Kamil Rytarowski
On 03.02.2020 17:10, Ryo ONODERA wrote: > Hi, > > Kamil Rytarowski writes: > >> Please check this workaround: >> >> http://netbsd.org/~kamil/patch-00224-firefox-pthread_equal.txt >> >> It has to be applied on firefox's package. > > Thank

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

2020-02-03 Thread Kamil Rytarowski
On 03.02.2020 17:24, Kamil Rytarowski wrote: > On 03.02.2020 17:21, Joerg Sonnenberger wrote: >> On Sat, Feb 01, 2020 at 03:38:46PM +0000, Kamil Rytarowski wrote: >>> Module Name:src >>> Committed By: kamil >>> Date: Sat Feb 1 15:3

Re: CVS commit: src/lib/libpthread

2020-02-12 Thread Kamil Rytarowski
> The invalid thread pointer is not 0x. > pthread_equal t1: 0x7b066d4d7800 > pthread_equal t2: 0x60f5f000 > > Of course, it is desirable to fix every misuses and bugs in pkgsrc. > However it is impossible for now (at least for me). > > "Kamil Rytarowski"

Re: CVS commit: src/lib/libpthread

2020-02-12 Thread Kamil Rytarowski
Hello, I will have a look at them. On 12.02.2020 14:02, Ryo ONODERA wrote: > Hi, > > Kamil Rytarowski writes: > >> Please apple workaround (same like in NSPR) for now if fixing is difficult. >> >> Such bugs can have security implications. > > Adding w

Re: CVS commit: src/external/bsd/dhcpcd/dist/src

2020-02-13 Thread Kamil Rytarowski
On 13.02.2020 00:58, Joerg Sonnenberger wrote: > On Mon, Feb 10, 2020 at 04:45:35PM +, Roy Marples wrote: >> On 09/02/2020 19:21, Joerg Sonnenberger wrote: >>> On Sat, Feb 08, 2020 at 12:17:16PM +, Santhosh Raju wrote: Module Name: src Committed By: fox Date:

Re: CVS commit: src/external/bsd/dhcpcd/dist/src

2020-02-13 Thread Kamil Rytarowski
On 13.02.2020 14:13, Joerg Sonnenberger wrote: > On Thu, Feb 13, 2020 at 02:05:12PM +0100, Kamil Rytarowski wrote: >> On 13.02.2020 00:58, Joerg Sonnenberger wrote: >>> On Mon, Feb 10, 2020 at 04:45:35PM +, Roy Marples wrote: >>>> On 09/02/2020 19:21, Joerg So

Re: CVS commit: src/external/bsd/dhcpcd/dist/src

2020-02-13 Thread Kamil Rytarowski
On 13.02.2020 14:50, Valery Ushakov wrote: > On Thu, Feb 13, 2020 at 14:18:43 +0100, Kamil Rytarowski wrote: > >> > Can you show the original problem that you are trying to fix here? > When and how did you get warning? > GCC has a property (as called by Joerg bug) t

Re: CVS commit: src/lib/libpthread

2020-02-13 Thread Kamil Rytarowski
On 12.02.2020 15:01, Ryo ONODERA wrote: > Hi, > > Kamil Rytarowski writes: > >> Hello, >> >> I will have a look at them. > > Thank you. > Real fix is welcome. > > And multimedia/handbrake has workaround already. > I have workaround patches

Re: CVS commit: src/external/bsd/dhcpcd/dist/src

2020-02-13 Thread Kamil Rytarowski
On 13.02.2020 18:00, Valery Ushakov wrote: > Arguably, if the tool you use is broken, you shouln't be mutilating > the source code just to shut that tool up. The introduced changes were good, even if GCC would be silent. It is far from mutilating. As an alternative option we can disable warnings

Re: CVS commit: src/external/bsd/dhcpcd/dist/src

2020-02-13 Thread Kamil Rytarowski
On 13.02.2020 22:20, Valery Ushakov wrote: > I did not propose to disable the warning. I proposed to downgrade > -Werror to -Wno-error (i.e. a warning) and only for the buggy > sanitizer build. That file will still be compiled in normal builds > with all the warnings=errors enabled, so real probl

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: >> src/tests/lib/libc/sy

Re: CVS commit: src/lib/libpthread

2020-02-19 Thread Kamil Rytarowski
On 13.02.2020 20:03, Ryo ONODERA wrote: > Hi, > > Kamil Rytarowski writes: > >> On 12.02.2020 15:01, Ryo ONODERA wrote: >>> Hi, >>> >>> Kamil Rytarowski writes: >>> >>>> Hello, >>>> >>>> I will have a

Re: CVS commit: src/external/apache2/llvm/config/llvm/Config

2020-02-19 Thread Kamil Rytarowski
On 19.02.2020 14:32, Joerg Sonnenberger wrote: > Module Name: src > Committed By: joerg > Date: Wed Feb 19 13:32:40 UTC 2020 > > Modified Files: > src/external/apache2/llvm/config/llvm/Config: config.h.in > > Log Message: > Revert last. It was objected to pre-commit, there is no ac

Re: CVS commit: src/doc

2020-02-19 Thread Kamil Rytarowski
On 18.02.2020 22:14, Christos Zoulas wrote: > Module Name: src > Committed By: christos > Date: Tue Feb 18 21:14:16 UTC 2020 > > Modified Files: > src/doc: 3RDPARTY CHANGES > > Log Message: > new awk > > > To generate a diff of this commit: > cvs rdiff -u -r1.1690 -r1.1691 src/do

Re: CVS commit: src/doc

2020-02-19 Thread Kamil Rytarowski
On 19.02.2020 16:32, Kamil Rytarowski wrote: > On 18.02.2020 22:14, Christos Zoulas wrote: >> Module Name: src >> Committed By:christos >> Date:Tue Feb 18 21:14:16 UTC 2020 >> >> Modified Files: >> src/doc: 3RDPARTY CHANGES &g

Re: CVS commit: src/external/apache2/llvm/config/llvm/Config

2020-02-19 Thread Kamil Rytarowski
On 19.02.2020 23:03, Joerg Sonnenberger wrote: > On Wed, Feb 19, 2020 at 04:10:17PM +0100, Kamil Rytarowski wrote: >> On 19.02.2020 14:32, Joerg Sonnenberger wrote: >>> Module Name:src >>> Committed By: joerg >>> Date: Wed Feb 19 13:3

Re: CVS commit: src/doc

2020-02-20 Thread Kamil Rytarowski
MESTAMP    Wed Feb 19 19:18:01 UTC 2020 >          Successful make release >          build.sh ended:      Wed Feb 19 17:48:15 EST 2020 > ===> . > > > >> On Feb 19, 2020, at 11:09 AM, Kamil Rytarowski > <mailto:n...@gmx.com>> wrote: >> >> Signed

Re: CVS commit: src/external/apache2/llvm/config/llvm/Config

2020-02-20 Thread Kamil Rytarowski
On 19.02.2020 23:50, Joerg Sonnenberger wrote: > On Wed, Feb 19, 2020 at 11:29:09PM +0100, Kamil Rytarowski wrote: >> On 19.02.2020 23:03, Joerg Sonnenberger wrote: >>> On Wed, Feb 19, 2020 at 04:10:17PM +0100, Kamil Rytarowski wrote: >>>> On 19.02.2020 1

Re: CVS commit: src/external/apache2/llvm/config/llvm/Config

2020-02-20 Thread Kamil Rytarowski
On 20.02.2020 01:09, Roy Marples wrote: > On 19/02/2020 22:29, Kamil Rytarowski wrote: >> Why do you need terminfo/termios in ./build.sh tools? > > We build the nbtic tool so we can build the terminfo database. > > Roy I will try to have a look and switch to

Re: CVS commit: src/external/apache2/llvm/config/llvm/Config

2020-02-20 Thread Kamil Rytarowski
On 21.02.2020 01:17, Christos Zoulas wrote: > In article <5f8df6c1-bb25-f24a-27fc-b3a752a6d...@gmx.com>, > Kamil Rytarowski wrote: > >> We don't have any direct reproducer (we tried) and we must figure it out >>from syzkaller bot. We don't have access to t

Re: CVS commit: src/external/apache2/llvm/config/llvm/Config

2020-02-20 Thread Kamil Rytarowski
On 21.02.2020 02:47, Christos Zoulas wrote: > So host needs to install terminfo-dev. How is that different from the host > environment needing to provide other pieces of the development environment? > For example if another tool needed openssl-dev, would we disable crypto? Or > if it needed yacc

Re: CVS commit: src/external/apache2/llvm/config/llvm/Config

2020-02-21 Thread Kamil Rytarowski
On 21.02.2020 09:17, Leonardo Taccari wrote: > Kamil Rytarowski writes: >> [...] >> ./build.sh tools for LLVM is broken when host ships with no terminfo >> devel libraries. I tried to disable this superfluous feature for tools >> (for colors in a terminal), but it got r

Re: CVS commit: src/external/apache2/llvm/config/llvm/Config

2020-02-21 Thread Kamil Rytarowski
On 21.02.2020 07:20, Martin Husemann wrote: > On Fri, Feb 21, 2020 at 03:01:45AM +0100, Kamil Rytarowski wrote: >> I consider calling ncurses-dev essential as a bug. > > My knee jerk reaction was: if a unix compiler needs terminfo (especially > when it only does that to create c

Re: CVS commit: src/sys/kern

2020-02-21 Thread Kamil Rytarowski
On 20.02.2020 22:14, Jaromir Dolecek wrote: > Module Name: src > Committed By: jdolecek > Date: Thu Feb 20 21:14:23 UTC 2020 > > Modified Files: > src/sys/kern: subr_autoconf.c > > Log Message: > protect deferred lists' manipulation by config_misc_lock, same as > config_pending sem

Re: CVS commit: src

2020-02-21 Thread Kamil Rytarowski
On 22.12.2019 20:47, Andrew Doran wrote: > Module Name: src > Committed By: ad > Date: Sun Dec 22 19:47:35 UTC 2019 > > Modified Files: > src/external/cddl/osnet/dist/uts/common/fs/zfs: zfs_ctldir.c > src/sys/kern: vfs_mount.c vfs_subr.c vfs_syscalls.c > src/sys/miscfs/g

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/modules

2020-02-22 Thread Kamil Rytarowski
t; >> 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 th

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, >>> &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 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, >>>>> &modstat) to check whether a specific module is loaded into the kernel &g

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

2020-02-22 Thread Kamil Rytarowski
On 23.02.2020 02:29, Valery Ushakov wrote: > On Sat, Feb 22, 2020 at 14:07:57 +0000, Kamil Rytarowski wrote: > >> Module Name: src >> Committed By:kamil >> Date:Sat Feb 22 14:07:57 UTC 2020 >> >> Modified Files: >> src/li

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

2020-02-22 Thread Kamil Rytarowski
On 23.02.2020 03:20, Valery Ushakov wrote: > On Sun, Feb 23, 2020 at 02:51:49 +0100, Kamil Rytarowski wrote: > >> On 23.02.2020 02:29, Valery Ushakov wrote: >>> On Sat, Feb 22, 2020 at 14:07:57 +0000, Kamil Rytarowski wrote: >>> >>>> Module Name: s

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

2020-02-23 Thread Kamil Rytarowski
On 23.02.2020 08:46, Martin Husemann wrote: > On Sun, Feb 23, 2020 at 03:35:19AM +0100, Kamil Rytarowski wrote: >> Algorithm would be changed from calculating on 32bit numbers with signed >> integer overflows to an algorithm calculating on 64bit numbers. The >> __dorand48() f

Re: CVS commit: src/external/bsd/tcpdump/dist

2020-02-24 Thread Kamil Rytarowski
On 24.02.2020 17:44, Valery Ushakov wrote: > On Mon, Feb 24, 2020 at 16:19:35 +0000, Kamil Rytarowski wrote: > >> Module Name: src >> Committed By:kamil >> Date:Mon Feb 24 16:19:35 UTC 2020 >> >> Modified Files: >> src/exte

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 >&g

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 >&g

Re: CVS commit: src/sys/kern

2020-02-25 Thread Kamil Rytarowski
On 24.02.2020 21:47, Jaromir Dolecek wrote: > Module Name: src > Committed By: jdolecek > Date: Mon Feb 24 20:47:47 UTC 2020 > > Modified Files: > src/sys/kern: init_main.c > > Log Message: > move config_init_mi() call before vfsinit(), which can trigger loading > of VFS modules >

Re: CVS commit: src/external/bsd/libarchive/dist/libarchive

2020-02-25 Thread Kamil Rytarowski
On 25.02.2020 16:24, Andreas Gustafsson wrote: > Kamil Rytarowski wrote: >> To generate a diff of this commit: >> cvs rdiff -u -r1.1.1.4 -r1.2 \ >> src/external/bsd/libarchive/dist/libarchive/archive_read.c > > What kind of sorcery is this? Why is the diff not r

Re: CVS commit: src/doc

2020-02-29 Thread Kamil Rytarowski
On 29.02.2020 19:00, Taylor R Campbell wrote: >> Module Name:src >> Committed By: kamil >> Date: Sat Feb 29 04:27:01 UTC 2020 >> >> Modified Files: >> src/doc: CHANGES >> >> Log Message: >> ld.elf_so(1): Implement DT_GNU_HASH > > Was this discussed anywhere? In the toolcha

Re: CVS commit: src/doc

2020-02-29 Thread Kamil Rytarowski
On 29.02.2020 19:29, Martin Husemann wrote: > On Sat, Feb 29, 2020 at 06:00:29PM +, Taylor R Campbell wrote: >>> Module Name:src >>> Committed By: kamil >>> Date: Sat Feb 29 04:27:01 UTC 2020 >>> >>> Modified Files: >>> src/doc: CHANGES >>> >>> Log Message: >>> ld.elf_so

Re: CVS commit: src/doc

2020-02-29 Thread Kamil Rytarowski
On 29.02.2020 21:58, Joerg Sonnenberger wrote: > On Sat, Feb 29, 2020 at 07:36:00PM +0100, Kamil Rytarowski wrote: >> On 29.02.2020 19:00, Taylor R Campbell wrote: >>>> Module Name:src >>>> Committed By: kamil >>>> Date: Sat Feb

Re: CVS commit: src/doc

2020-02-29 Thread Kamil Rytarowski
On 29.02.2020 22:29, Kamil Rytarowski wrote: > On 29.02.2020 21:58, Joerg Sonnenberger wrote: >> On Sat, Feb 29, 2020 at 07:36:00PM +0100, Kamil Rytarowski wrote: >>> On 29.02.2020 19:00, Taylor R Campbell wrote: >>>>> Module Name:src >>>>> Com

Re: CVS commit: src/doc

2020-03-01 Thread Kamil Rytarowski
On 29.02.2020 23:31, Joerg Sonnenberger wrote: > On Sat, Feb 29, 2020 at 10:29:02PM +0100, Kamil Rytarowski wrote: >> On 29.02.2020 21:58, Joerg Sonnenberger wrote: >>> On Sat, Feb 29, 2020 at 07:36:00PM +0100, Kamil Rytarowski wrote: >>>> On 29.02.2020

Re: CVS commit: src/include

2020-03-01 Thread Kamil Rytarowski
On 01.03.2020 19:31, Joerg Sonnenberger wrote: > On Sun, Mar 01, 2020 at 03:08:16PM +0000, Kamil Rytarowski wrote: >> Module Name: src >> Committed By:kamil >> Date:Sun Mar 1 15:08:16 UTC 2020 >> >> Modified Files: >> src/include

Re: CVS commit: src/include

2020-03-01 Thread Kamil Rytarowski
On 01.03.2020 22:54, Joerg Sonnenberger wrote: > On Sun, Mar 01, 2020 at 10:26:08PM +0100, Kamil Rytarowski wrote: >> On 01.03.2020 19:31, Joerg Sonnenberger wrote: >>> On Sun, Mar 01, 2020 at 03:08:16PM +, Kamil Rytarowski wrote: >>>> Module Name: src

Re: CVS commit: src/doc

2020-03-01 Thread Kamil Rytarowski
On 01.03.2020 22:36, Joerg Sonnenberger wrote: > On Sun, Mar 01, 2020 at 10:20:44PM +0100, Kamil Rytarowski wrote: >> If that will nor realize, I will request to enable GNU_HASH for NetBSD-11. > > You have so far demonstrated no justification at all beyond handwaving > fo

Re: CVS commit: src/include

2020-03-01 Thread Kamil Rytarowski
On 02.03.2020 03:58, matthew green wrote: >>> On Sun, Mar 01, 2020 at 03:08:16PM +0000, Kamil Rytarowski wrote: >>>> Module Name: src >>>> Committed By: kamil >>>> Date: Sun Mar 1 15:08:16 UTC 2020 >>>> >>

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 r

Re: CVS commit: src/sys/rump

2020-03-09 Thread Kamil Rytarowski
On 09.03.2020 15:31, Taylor R Campbell wrote: >> Module Name:src >> Committed By: kamil >> Date: Mon Mar 9 00:03:00 UTC 2020 >> >> Modified Files: >> src/sys/rump: Makefile.rump >> >> Log Message: >> Build RUMP with -fno-delete-null-pointer-checks on all compilers > > I as

Re: CVS commit: src/external/gpl3/gdb/dist/bfd

2020-03-14 Thread Kamil Rytarowski
On 26.02.2016 17:28, Christos Zoulas wrote: > Module Name: src > Committed By: christos > Date: Fri Feb 26 16:28:14 UTC 2016 > > Modified Files: > src/external/gpl3/gdb/dist/bfd: merge.c > > Log Message: > CID 420802: Avoid NULL deref. > > > To generate a diff of this commit: > c

Re: CVS commit: src/lib/librumpuser

2020-03-23 Thread Kamil Rytarowski
On 24.03.2020 05:27, Robert Elz wrote: > There was no great urgency to "fix" > this particular one, it doesn't seem to have been causing any real problems, > and could have easily waited a few days (or weeks, or even months) until > the correct solution was found. > > kre > This patch was sittin

Re: CVS commit: src/lib/librumpuser

2020-03-24 Thread Kamil Rytarowski
On 24.03.2020 06:09, Robert Elz wrote: > Date:Tue, 24 Mar 2020 05:40:13 +0100 > From: Kamil Rytarowski > Message-ID: > > > | This patch was sitting in the tree since August 2019. > > In your tree I assume you mean, it certainly hasn'

Re: CVS commit: src/lib/librumpuser

2020-03-24 Thread Kamil Rytarowski
On 24.03.2020 15:41, Robert Elz wrote: > Date:Tue, 24 Mar 2020 13:27:45 +0100 > From: Kamil Rytarowski > Message-ID: <5ec1195a-f1c8-cd46-6a14-ea29da109...@gmx.com> > > | I patched it myself only when I reproduced the problems myself. >

Re: CVS commit: src/external/gpl3

2020-03-25 Thread Kamil Rytarowski
Upstream (GCC) is strongly against this change (even under __NetBSD__ ifdef) as /var/tmp is typically larger than /tmp: > I'd strongly recommend against this as-is. > > The whole reason we prefer /var/tmp is because it's often dramatically larger > than a ram-backed /tmp. -- by Jeff Law. Do we i

Re: CVS commit: src/external/gpl3

2020-03-26 Thread Kamil Rytarowski
On 26.03.2020 09:48, Martin Husemann wrote: > On Thu, Mar 26, 2020 at 12:51:24AM +, Taylor R Campbell wrote: >> We should keep the change. There is no semantic justification for >> putting build-time temporary files in the directory for temporary >> files that are meant to persist across reboo

Re: CVS commit: src/external/gpl3

2020-03-26 Thread Kamil Rytarowski
On 26.03.2020 15:01, Martin Husemann wrote: > On Thu, Mar 26, 2020 at 02:57:40PM +0100, Kamil Rytarowski wrote: >> The build of tools could be fixed independently. > The problem is that we build the whole system with the tools gcc, and that > gcc misbehaves (or so I understood). &

Re: CVS commit: src/external/gpl3

2020-03-26 Thread Kamil Rytarowski
On 26.03.2020 15:52, Greg Troxel wrote: > What is the real problem here? I think it's great that NetBSD-specific > fixes are being upstreamed, and that we are reducing gratuitous changes > from upstream. But upstream's position that the default tmp should be > /var/tmp is at odds with our and tra

Re: CVS commit: src/external/gpl3

2020-03-26 Thread Kamil Rytarowski
On 26.03.2020 16:02, Robert Elz wrote: > EVery extra env var has a cost in every exec of absolutely everything. As this is true.. but as I have benchmarked it. For true(1) on amd64, we execute 0.05% more instructions for extra TMPDIR environment variable. $ ./singlestepper true Total count: 1286

Re: CVS commit: src/external/gpl3

2020-03-26 Thread Kamil Rytarowski
On 26.03.2020 16:17, Greg Troxel wrote: > I don't think we can go from "upstream is unwilling to do X" > to "impose pain on NetBSD" by making "divergence bad" be the > highest-weighted concern.   We have to say "what is the minimally > painful way to cope". I agree here that we want to get /tmp by

Re: CVS commit: src/external/gpl3

2020-03-26 Thread Kamil Rytarowski
On 26.03.2020 17:03, Kamil Rytarowski wrote: > On 26.03.2020 16:17, Greg Troxel wrote: >> I don't think we can go from "upstream is unwilling to do X" >> to "impose pain on NetBSD" by making "divergence bad" be the >> highest-weighted concern

Re: CVS commit: src/external/gpl3

2020-03-26 Thread Kamil Rytarowski
On 26.03.2020 17:49, Robert Elz wrote: > Date:Thu, 26 Mar 2020 16:28:18 +0100 > From: Kamil Rytarowski > Message-ID: <84460ebb-b4bf-f3ee-e51b-e27d0b6e2...@gmx.com> > > > | That is negligible cost of getting TMPDIR propagated to most programs.

Re: CVS commit: src/external/gpl3

2020-03-27 Thread Kamil Rytarowski
a0e605b50d99665de27a77f5ca870f2c8b41d (HEAD -> test) Author: ozaki-r Date: Mon Aug 10 09:32:01 2015 + Fix head and cleanup definitions for cache_expiration tests On 26.03.2020 01:43, Jonathan A. Kollasch wrote: > We (to the extent I can assert that as an individual developer)

Re: CVS commit: src/external/gpl3

2020-03-27 Thread Kamil Rytarowski
On 27.03.2020 16:01, Martin Husemann wrote: > Which compiler version did you test? > I tested on NetBSD HEAD GCC style distribution as host. $ uname -rms NetBSD 9.99.51 amd64 > I see this on netbsd-8: > >> echo $TMPDIR > TMPDIR: Undefined variable. >> ktrace -i $TOOLDIR/bin/x86_64--netbsd-gcc b

Re: CVS commit: src/external/gpl3

2020-03-27 Thread Kamil Rytarowski
On 27.03.2020 18:24, Martin Husemann wrote: > On Fri, Mar 27, 2020 at 06:14:24PM +0100, Kamil Rytarowski wrote: >> On 27.03.2020 16:01, Martin Husemann wrote: >>> Which compiler version did you test? >>> >> >> I tested on NetBSD HEAD GCC style distribution

Re: CVS commit: src/external/gpl3

2020-03-27 Thread Kamil Rytarowski
On 27.03.2020 18:42, Martin Husemann wrote: > On Fri, Mar 27, 2020 at 06:36:29PM +0100, Kamil Rytarowski wrote: >> I tested the code without our local patch as mentioned in my commit, >> even in the snapshot 1 commit before jak@'s change. >> >> In all the cases I g

Re: CVS commit: src/external/gpl3

2020-03-28 Thread Kamil Rytarowski
On 28.03.2020 05:39, matthew green wrote: >> Date:Sat, 28 Mar 2020 11:46:29 +1100 >> From:matthew green >> Message-ID: <15233.1585356...@splode.eterna.com.au> >> >> | can we just leave this as-is and let netbsd GCC people care? >> >> Only if the GCC people do care, a

Re: CVS commit: src/sys/modules/examples/fopsmapper

2020-04-01 Thread Kamil Rytarowski
On 01.04.2020 15:47, Robert Elz wrote: > Date:Wed, 1 Apr 2020 11:45:53 + > From: "Kamil Rytarowski" > Message-ID: <20200401114554.05167f...@cvs.netbsd.org> > > | Log Message: > | Avoid comparison between signed and unsigned int

Re: CVS commit: src/lib/librumpuser

2020-04-01 Thread Kamil Rytarowski
On 24.03.2020 19:37, Kamil Rytarowski wrote: > On 24.03.2020 15:41, Robert Elz wrote: >> Date:Tue, 24 Mar 2020 13:27:45 +0100 >> From:Kamil Rytarowski >> Message-ID: <5ec1195a-f1c8-cd46-6a14-ea29da109...@gmx.com> >> >> | I p

Re: CVS commit: src/sys/modules/examples/fopsmapper

2020-04-01 Thread Kamil Rytarowski
On 01.04.2020 16:10, Paul Goyette wrote: > On Wed, 1 Apr 2020, Kamil Rytarowski wrote: > >> On 01.04.2020 15:47, Robert Elz wrote: >>>     Date:    Wed, 1 Apr 2020 11:45:53 + >>>     From:    "Kamil Rytarowski" >>>     Mess

Re: CVS commit: src/sys/modules/examples/fopsmapper

2020-04-01 Thread Kamil Rytarowski
On 01.04.2020 16:17, Christos Zoulas wrote: > In article , > Paul Goyette wrote: >> On Wed, 1 Apr 2020, Kamil Rytarowski wrote: >> >>> On 01.04.2020 15:47, Robert Elz wrote: >>>> Date:Wed, 1 Apr 2020 11:45:53 + >>>> F

Re: CVS commit: src/lib/librumpuser

2020-04-01 Thread Kamil Rytarowski
On 01.04.2020 17:06, Robert Elz wrote: > Date:Wed, 1 Apr 2020 15:54:15 +0200 > From: Kamil Rytarowski > Message-ID: <969362d2-d93e-2cf4-7437-ab593ab11...@gmx.com> > > | Ping? This still breaks. > > I am still working on it.Best I can

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

2020-04-02 Thread Kamil Rytarowski
On 02.04.2020 19:40, Christos Zoulas wrote: > +set -x > +AWK=gawk > + gawk?

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

2020-04-02 Thread Kamil Rytarowski
it uses cpu > > return area; > } > > >> On Apr 2, 2020, at 5:31 PM, Kamil Rytarowski wrote: >> >> On 02.04.2020 19:40, Christos Zoulas wrote: >>> +set -x >>> +AWK=gawk >>> + >> >> gawk? >

Re: CVS commit: src/tools/binutils

2020-04-03 Thread Kamil Rytarowski
On 04.04.2020 02:47, Christos Zoulas wrote: > Module Name: src > Committed By: christos > Date: Sat Apr 4 00:47:52 UTC 2020 > > Modified Files: > src/tools/binutils: mknative-binutils > > Log Message: > Handle libctf new in binutils 2.34 > > Please note that the upstreamed libctf

Re: CVS commit: src/include

2020-04-17 Thread Kamil Rytarowski
On 17.04.2020 22:22, Joerg Sonnenberger wrote: > On Fri, Apr 17, 2020 at 03:22:35PM +0000, Kamil Rytarowski wrote: >> Module Name: src >> Committed By:kamil >> Date:Fri Apr 17 15:22:35 UTC 2020 >> >> Modified Files: >> src/include

Re: CVS commit: src

2020-04-22 Thread Kamil Rytarowski
On 04.04.2020 22:20, Jason R Thorpe wrote: > Module Name: src > Committed By: thorpej > Date: Sat Apr 4 20:20:12 UTC 2020 > > Modified Files: > src/sys/compat/netbsd32: syscalls.master > src/sys/kern: kern_exit.c kern_lwp.c sys_lwp.c syscalls.master > src/sys/sys: lwp.h

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 o

Re: CVS commit: src/sys

2020-04-24 Thread Kamil Rytarowski
On 24.04.2020 05:22, Jason R Thorpe wrote: > Module Name: src > Committed By: thorpej > Date: Fri Apr 24 03:22:06 UTC 2020 > > Modified Files: > src/sys/compat/linux/common: linux_exec.c linux_sched.c > src/sys/kern: kern_exec.c kern_exit.c kern_fork.c kern_lwp.c > k

Re: CVS commit: src/lib/librumpuser

2020-05-05 Thread Kamil Rytarowski
Ping? We are blocked by this in GSoC now. On 01.04.2020 20:19, Kamil Rytarowski wrote: > On 01.04.2020 17:06, Robert Elz wrote: >> Date:Wed, 1 Apr 2020 15:54:15 +0200 >> From:Kamil Rytarowski >> Message-ID: <969362d2-d93e-2cf4-7

Re: CVS commit: src/lib/librumpuser

2020-05-05 Thread Kamil Rytarowski
On 06.05.2020 07:17, Alistair Crooks wrote: > > > On Tue, 5 May 2020 at 20:54, Kamil Rytarowski <mailto:n...@gmx.com>> wrote: > > Ping? We are blocked by this in GSoC now. > > > I doubt that you are "blocked by this in GSoC", as the GSoC project

Re: CVS commit: src/bin/kill

2020-05-06 Thread Kamil Rytarowski
On 06.05.2020 11:07, Robert Elz wrote: > Module Name: src > Committed By: kre > Date: Wed May 6 09:07:15 UTC 2020 > > Modified Files: > src/bin/kill: kill.1 > > Log Message: > kill is built-in to more than just csh(1). > While here, add missing Xr sh 1 (which was previously needed

Re: CVS commit: src/sys/kern

2020-05-06 Thread Kamil Rytarowski
This caused regressions in t_ptrace_wait* tests and random hangs/timeouts/failures. If I revert the pipe(2) changes on top of NetBSD-current, the test is no longer racy again. http://netbsd.org/~kamil/patch-00249-pipe-revert.txt Reproducer: cd /usr/tests/lib/libc/sys ./t_ptrace_waitpid tracer_

Re: CVS commit: src/sys/kern

2020-05-07 Thread Kamil Rytarowski
On 07.05.2020 07:46, Michael van Elst wrote: > On Wed, May 06, 2020 at 12:39:21PM +0200, Kamil Rytarowski wrote: > > Hi Kamil, > >> If I revert the pipe(2) changes on top of NetBSD-current, the test is no >> longer racy again. > > I tried 9.99.60 with and with

Re: CVS commit: src/bin/kill

2020-05-08 Thread Kamil Rytarowski
On 06.05.2020 14:48, Robert Elz wrote: > Date:Wed, 6 May 2020 11:28:42 +0200 > From: Kamil Rytarowski > Message-ID: > > > | While there, we have got a long standing issue with wait.1 man page, > > Huh! I had no idea any such thing exi

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 > | >

Re: CVS commit: src

2020-05-11 Thread Kamil Rytarowski
On 19.04.2020 03:06, Joerg Sonnenberger wrote: > Module Name: src > Committed By: joerg > Date: Sun Apr 19 01:06:16 UTC 2020 > > Modified Files: > src/lib/libc/gen: pthread_atfork.c > src/libexec/ld.elf_so: rtld.c rtld.h symbols.map > > Log Message: > Rename __atomic_fork to

Re: CVS commit: src

2020-05-12 Thread Kamil Rytarowski
On 12.05.2020 02:59, Joerg Sonnenberger wrote: > On Mon, May 11, 2020 at 11:07:02PM +0200, Kamil Rytarowski wrote: >> On 19.04.2020 03:06, Joerg Sonnenberger wrote: >>> Module Name:src >>> Committed By: joerg >>> Date: Sun Apr 19 01:0

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 +0000, Kamil Rytarowski wrote: >> Module Name: src >> Committed By:kamil >> Date:Thu May 14 19:21:35 UTC 2020 >> >> Modified Files: >> src/t

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 >>

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 > archi

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

2020-05-16 Thread Kamil Rytarowski
; >> On May 16, 2020, at 9:46 AM, Kamil Rytarowski wrote: >> >> Signed PGP part >> On 16.05.2020 14:54, Christos Zoulas wrote: >>> Module Name:src >>> Committed By: christos >>> Date: Sat May 16 12:54:27 UTC 2020 >

Re: CVS commit: src/external/bsd/ntp/bin/ntpd

2020-05-29 Thread Kamil Rytarowski
On 29.05.2020 16:15, Tobias Nygren wrote: > On Fri, 29 May 2020 16:08:30 +0200 > Joerg Sonnenberger wrote: > >> On Fri, May 29, 2020 at 10:53:02AM +, Kamil Rytarowski wrote: >>> Module Name:src >>> Committed By: kamil >>> Date:

Re: CVS commit: src/sys/ddb

2020-05-31 Thread Kamil Rytarowski
I wrote a tiny malloc (libc-style) implementation over a small static storage (could be over stack or preallocated on the heap) without any external dependencies. Would it be useful for you? On 01.06.2020 01:34, Rin Okuyama wrote: > Module Name: src > Committed By: rin > Date: Sun May 31

<    1   2   3   4   5   6   7   8   9   >