Re: CVS commit: src/share/man/man7

2023-11-26 Thread Valery Ushakov
On Sat, Nov 25, 2023 at 11:36:34 -0800, Alistair Crooks wrote: > I find it better to have > > MAN+= bar.7 > MAN+= foo.7 > > Since a grep for 'MAN.*foo' will produce meaningful results Also good for diffs in the future, e.g. MAN += f.3 -MAN += g.3 MAN += h.3 vs. f.3 \ -g.3 \

Re: CVS commit: src/share/man/man7

2023-11-26 Thread Alistair Crooks
On Fri, Nov 24, 2023 at 18:09 Taylor R Campbell < campbell+netbsd-source-change...@mumble.net> wrote: > > Date: Sat, 25 Nov 2023 02:05:25 + > > From: Taylor R Campbell > > > > > Date: Sat, 25 Nov 2023 00:23:33 - (UTC) > > > From: chris...@astron.com (Christos Zoulas) > > > > > > Yes,

Re: CVS commit: src/share/man/man7

2023-11-24 Thread Taylor R Campbell
> Date: Sat, 25 Nov 2023 02:05:25 + > From: Taylor R Campbell > > > Date: Sat, 25 Nov 2023 00:23:33 - (UTC) > > From: chris...@astron.com (Christos Zoulas) > > > > Yes, this is indeed a lot better. I prefer though: > > > > MAN+= \ > > bar.7 \ > > foo.7 > > > > It is faster to parse,

Re: CVS commit: src/share/man/man7

2023-11-24 Thread Taylor R Campbell
> Date: Sat, 25 Nov 2023 00:23:33 - (UTC) > From: chris...@astron.com (Christos Zoulas) > > Yes, this is indeed a lot better. I prefer though: > > MAN+= \ > bar.7 \ > foo.7 > > It is faster to parse, involves less typing, whitespace is cleaner. This one doesn't have the same pattern for

Re: CVS commit: src/share/man/man7

2023-11-24 Thread Christos Zoulas
In article <20231123211614.011a1f...@cvs.netbsd.org>, Taylor R Campbell wrote: >-=-=-=-=-=- > >Module Name: src >Committed By: riastradh >Date: Thu Nov 23 21:16:13 UTC 2023 > >Modified Files: > src/share/man/man7: Makefile > >Log Message: >share/man/man7/Makefile: Split MAN on

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

2023-11-14 Thread Jörg Sonnenberger
On Wednesday, November 15, 2023 4:15:28 AM CET you wrote: > Module Name: src > Committed By: christos > Date: Wed Nov 15 03:15:28 UTC 2023 > > Modified Files: > src/lib/libc/ssp: Makefile.inc > Added Files: > src/lib/libc/ssp: ssp_redirect.c > > Log Message: > provide

Re: CVS commit: src/sys/arch/mips/mips

2023-10-25 Thread Andrius V
Thank you, I should pay attention to that. On Wed, Oct 25, 2023 at 9:02 AM Nick Hudson wrote: > > Module Name:src > Committed By: skrll > Date: Wed Oct 25 06:02:14 UTC 2023 > > Modified Files: > src/sys/arch/mips/mips: kgdb_machdep.c > > Log Message: > -> > > > To

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/sys/dev/pci/ixgbe

2023-10-17 Thread SAITOH Masanobu
On 2023/10/12 14:50, SAITOH Masanobu wrote: > Module Name: src > Committed By: msaitoh > Date: Thu Oct 12 05:50:56 UTC 2023 > > Modified Files: > src/sys/dev/pci/ixgbe: ixgbe.c > > Log Message: > ixg(4): Don't print wrong error message about ixgbe_num_queues. > > Don't override

Re: CVS commit: src/sys/dev/pci/igc

2023-10-16 Thread Greg Oster
On 2023-10-15 17.06, Joerg Sonnenberger wrote: On Sun, Oct 15, 2023 at 10:36:53PM +, Greg Oster wrote: Module Name:src Committed By: oster Date: Sun Oct 15 22:36:53 UTC 2023 Modified Files: src/sys/dev/pci/igc: if_igc.c Log Message: Fix build of the MODULAR

Re: CVS commit: src/sys/dev/pci/igc

2023-10-15 Thread Joerg Sonnenberger
On Sun, Oct 15, 2023 at 10:36:53PM +, Greg Oster wrote: > Module Name: src > Committed By: oster > Date: Sun Oct 15 22:36:53 UTC 2023 > > Modified Files: > src/sys/dev/pci/igc: if_igc.c > > Log Message: > Fix build of the MODULAR kernel, which explicitly excludes vlans.

Re: CVS commit: src/sys

2023-10-14 Thread Taylor R Campbell
> Date: Fri, 13 Oct 2023 19:03:35 + > From: Andrew Doran > > On Thu, Oct 12, 2023 at 11:55:46AM +0200, J. Hannken-Illjes wrote: > > This is not true for RUMP. Hero you added sleepq_remove() as > > "sleepq_unsleep(l, true)". This will unlock l_mutex without changing. > > > > Just poking

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

2023-10-13 Thread matthew green
> Minor changes to jemalloc100 (the old one that only vax etc currently uses). thanks. i'm still using this version on a bunch of modern machines. new jemalloc was problematic for a few things for me a number of years ago and i keep meaning to test again, but for now i'm still mostly using this

Re: CVS commit: src/sys

2023-10-13 Thread Andrew Doran
On Thu, Oct 12, 2023 at 11:55:46AM +0200, J. Hannken-Illjes wrote: > > On 10. Oct 2023, at 20:58, Andrew Doran wrote: > > > > On Tue, Oct 10, 2023 at 06:00:57PM +0200, J. Hannken-Illjes wrote: > > > >>> cvs rdiff -u -r1.63 -r1.64 src/sys/kern/sys_select.c > >> > >> -sleepq_unsleep(l,

Re: CVS commit: src/sys/kern

2023-10-13 Thread Taylor R Campbell
> Date: Fri, 13 Oct 2023 17:52:07 +0900 > From: Rin Okuyama > > It would be really nice if we can find some systematical/reliable methods to > figure out files that really depends on struct syncobj, e.g.. I tried > ctfdump(1) to > *.o for kernel modules, but I cannot extract information better

Re: CVS commit: src/sys

2023-10-13 Thread Taylor R Campbell
> Date: Tue, 10 Oct 2023 18:58:29 + > From: Andrew Doran > > On Tue, Oct 10, 2023 at 06:00:57PM +0200, J. Hannken-Illjes wrote: > > > > cvs rdiff -u -r1.63 -r1.64 src/sys/kern/sys_select.c > > > > -sleepq_unsleep(l, false); > > +sleepq_remove(l->l_sleepq, l, true); > > } > >

Re: CVS commit: src/sys/kern

2023-10-13 Thread Rin Okuyama
On Thu, Oct 12, 2023 at 8:23 PM Taylor R Campbell wrote: > > > Date: Thu, 12 Oct 2023 17:06:02 +0900 > > From: Rin Okuyama > > > > On Thu, Oct 5, 2023 at 5:39 AM Andrew Doran wrote: > > > > > > Module Name:src > > > Committed By: ad > > > Date: Wed Oct 4 20:39:35 UTC 2023 > >

Re: CVS commit: src/sys/kern

2023-10-12 Thread Taylor R Campbell
> Date: Thu, 12 Oct 2023 17:06:02 +0900 > From: Rin Okuyama > > On Thu, Oct 5, 2023 at 5:39 AM Andrew Doran wrote: > > > > Module Name:src > > Committed By: ad > > Date: Wed Oct 4 20:39:35 UTC 2023 > > > > Modified Files: > > src/sys/kern: kern_rwlock.c

Re: CVS commit: src/sys

2023-10-12 Thread J. Hannken-Illjes
> On 10. Oct 2023, at 20:58, Andrew Doran wrote: > > On Tue, Oct 10, 2023 at 06:00:57PM +0200, J. Hannken-Illjes wrote: > >>> cvs rdiff -u -r1.63 -r1.64 src/sys/kern/sys_select.c >> >> -sleepq_unsleep(l, false); >> +sleepq_remove(l->l_sleepq, l, true); >>} >> } >>

Re: CVS commit: src/sys/kern

2023-10-12 Thread Rin Okuyama
Cool! Can I send a pull up request to netbsd-10? I've not yet observed deadlocks since this was committed, fortunately or unfortunately, although ;) Thanks, rin On Thu, Oct 5, 2023 at 5:39 AM Andrew Doran wrote: > > Module Name:src > Committed By: ad > Date: Wed Oct 4 20:39:35

Re: CVS commit: src/sys

2023-10-10 Thread Andrew Doran
On Tue, Oct 10, 2023 at 06:00:57PM +0200, J. Hannken-Illjes wrote: > > cvs rdiff -u -r1.63 -r1.64 src/sys/kern/sys_select.c > > -sleepq_unsleep(l, false); > +sleepq_remove(l->l_sleepq, l, true); > } >} >mutex_spin_exit(lock); > > Looks like sleepq_remove() unlocks l->l_mutex

Re: CVS commit: src/sys

2023-10-10 Thread J. Hannken-Illjes
> On 8. Oct 2023, at 15:23, Andrew Doran wrote: > > Module Name: src > Committed By: ad > Date: Sun Oct 8 13:23:05 UTC 2023 > > Modified Files: > src/sys/kern: kern_condvar.c kern_sleepq.c kern_timeout.c >kern_turnstile.c sys_lwp.c sys_select.c > src/sys/rump/librump/rumpkern: sleepq.c >

Re: CVS commit: src/sys/arch/news68k/conf

2023-10-08 Thread Andrius V
OK, it makes sense. I will revert the changes. Thanks for your explanations. On Sun, Oct 8, 2023 at 12:49 PM matthew green wrote: > > > I was changing news68k specific code, thus wasn't treating them as > > common. But I understand the point. > > there's a *LOT* of m68k code that is copied

re: CVS commit: src/sys/arch/news68k/conf

2023-10-08 Thread matthew green
> I was changing news68k specific code, thus wasn't treating them as > common. But I understand the point. there's a *LOT* of m68k code that is copied into all the ports that is almost identical, and should really be shared, but it not, and changes like can make this harder to share. ie, while

Re: CVS commit: src/sys/arch/news68k/conf

2023-10-08 Thread Andrius V
On Sun, Oct 8, 2023 at 6:56 AM Izumi Tsutsui wrote: > > > In this case maybe I should remove all FPSP references (vectors.S, > > locore.S, Makefile.news68k (MD_LIBS={FPSP})? > > IMO we don't have to keep strict consistencies or buildabilities of > options but rather should consider readabilities

Re: CVS commit: src/sys/arch/news68k/conf

2023-10-07 Thread Izumi Tsutsui
> In this case maybe I should remove all FPSP references (vectors.S, > locore.S, Makefile.news68k (MD_LIBS={FPSP})? IMO we don't have to keep strict consistencies or buildabilities of options but rather should consider readabilities and maintainabilities. - options FPSP in a config file is not

Re: CVS commit: src/sys

2023-10-05 Thread Andrew Doran
On Thu, Oct 05, 2023 at 12:15:23PM +0200, Martin Husemann wrote: > On Thu, Oct 05, 2023 at 09:59:49AM +, Andrew Doran wrote: > > Yes that makes sense and is what I plan to do after work today if nobody > > beats me to it. MULTIPROCESSOR is long overdue removal from the MI kernel, > > IMO. >

Re: CVS commit: src/sys

2023-10-05 Thread Martin Husemann
On Thu, Oct 05, 2023 at 09:59:49AM +, Andrew Doran wrote: > Yes that makes sense and is what I plan to do after work today if nobody > beats me to it. MULTIPROCESSOR is long overdue removal from the MI kernel, > IMO. Hey, this is a tiny landisk kernel, do not bloat it :-) Unfortunately it

Re: CVS commit: src/sys

2023-10-05 Thread Andrew Doran
Martin, On Thu, Oct 05, 2023 at 11:42:03AM +0200, Martin Husemann wrote: > On Thu, Oct 05, 2023 at 11:36:23AM +0200, Martin Husemann wrote: > > No, I was confused by the #ifdef maze - it breaks the build for > > non-MULTIPROCESSOR kernels only, and I am not actually sure what "use" > > gcc sees

Re: CVS commit: src/sys

2023-10-05 Thread Martin Husemann
On Thu, Oct 05, 2023 at 11:36:23AM +0200, Martin Husemann wrote: > No, I was confused by the #ifdef maze - it breaks the build for > non-MULTIPROCESSOR kernels only, and I am not actually sure what "use" > gcc sees for the "nlocks" variable at all in that case. Scratch that too, I'll get coffee.

Re: CVS commit: src/sys

2023-10-05 Thread Martin Husemann
On Thu, Oct 05, 2023 at 11:28:49AM +0200, Martin Husemann wrote: > This breaks the build: > > ../../../../kern/kern_synch.c: In function 'kpause': > ../../../../kern/kern_synch.c:257:10: error: 'nlocks' may be used > uninitialized in this function [-Werror=maybe-uninitialized] > 257 | error =

Re: CVS commit: src/sys

2023-10-05 Thread Martin Husemann
On Wed, Oct 04, 2023 at 08:29:18PM +, Andrew Doran wrote: > Module Name: src > Committed By: ad > Date: Wed Oct 4 20:29:18 UTC 2023 > > Modified Files: > src/sys/kern: kern_condvar.c kern_exec.c kern_exit.c kern_sig.c > kern_sleepq.c kern_synch.c kern_timeout.c

Re: CVS commit: src/sys/arch/news68k/conf

2023-10-01 Thread Andrius V
In this case maybe I should remove all FPSP references (vectors.S, locore.S, Makefile.news68k (MD_LIBS={FPSP})? On Sun, Oct 1, 2023 at 10:08 PM Izumi Tsutsui wrote: > > > Module Name: src > > Committed By: andvar > > Date: Sun Oct 1 18:50:53 UTC 2023 > > > > Modified Files: > >

Re: CVS commit: src/sys/arch/news68k/conf

2023-10-01 Thread Izumi Tsutsui
> Module Name: src > Committed By: andvar > Date: Sun Oct 1 18:50:53 UTC 2023 > > Modified Files: > src/sys/arch/news68k/conf: Makefile.news68k > > Log Message: > include fpsp Makefile.inc in Makefile.news68k, same as other m68k ports. > > needed for FPSP option to build,

Re: CVS commit: src/sbin/gpt

2023-09-27 Thread Robert Elz
Date:Wed, 27 Sep 2023 09:44:10 + From:"Taylor R Campbell" Message-ID: <20230927094410.b9257f...@cvs.netbsd.org> | gpt(8): Make gpt type array and enum match again. Thanks, and apologies for not checking that better - I did test that it recognised the new one

Re: CVS commit: src

2023-09-23 Thread Andrew Doran
On Sun, Sep 24, 2023 at 03:51:07AM +0900, Rin Okuyama wrote: > Hi, > > On 2023/09/24 3:21, Andrew Doran wrote: > > Index: src/common/lib/libc/gen/radixtree.c > > diff -u src/common/lib/libc/gen/radixtree.c:1.31 > > src/common/lib/libc/gen/radixtree.c:1.32 > > ---

Re: CVS commit: src

2023-09-23 Thread Rin Okuyama
Hi, On 2023/09/24 3:21, Andrew Doran wrote: Index: src/common/lib/libc/gen/radixtree.c diff -u src/common/lib/libc/gen/radixtree.c:1.31 src/common/lib/libc/gen/radixtree.c:1.32 --- src/common/lib/libc/gen/radixtree.c:1.31Tue Sep 12 16:17:21 2023 +++ src/common/lib/libc/gen/radixtree.c Sat

Re: CVS commit: src/external/gpl3/binutils/dist/ld/emultempl

2023-09-11 Thread Joerg Sonnenberger
On Mon, Sep 11, 2023 at 12:27:25PM +1000, matthew green wrote: > "Rin Okuyama" writes: > > Module Name:src > > Committed By: rin > > Date: Mon Sep 11 01:54:18 UTC 2023 > > > > Modified Files: > > src/external/gpl3/binutils/dist/ld/emultempl: aarch64elf.em armelf.em

Re: CVS commit: src/external/gpl3/binutils/dist/ld/emultempl

2023-09-10 Thread Rin Okuyama
On 2023/09/11 11:27, matthew green wrote: "Rin Okuyama" writes: Module Name:src Committed By: rin Date: Mon Sep 11 01:54:18 UTC 2023 Modified Files: src/external/gpl3/binutils/dist/ld/emultempl: aarch64elf.em armelf.em elf.em Log Message: ld: Enable

re: CVS commit: src/external/gpl3/binutils/dist/ld/emultempl

2023-09-10 Thread matthew green
"Rin Okuyama" writes: > Module Name: src > Committed By: rin > Date: Mon Sep 11 01:54:18 UTC 2023 > > Modified Files: > src/external/gpl3/binutils/dist/ld/emultempl: aarch64elf.em armelf.em > elf.em > > Log Message: > ld: Enable --copy-dt-needed-entries by default again

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

2023-09-10 Thread bch
On Fri, Sep 8, 2023 at 21:38 matthew green wrote: > Module Name:src > Committed By: mrg > Date: Sat Sep 9 04:38:49 UTC 2023 > > Modified Files: > src/usr.bin/make: main.c > > Log Message: > add explicit cast for long -> int truncation warning-as-error. > > as this is

Re: CVS commit: src/sys/arch/amiga/conf

2023-09-05 Thread Rin Okuyama
Hi, Oops, I didn't notice it is not generated. Sorry for bothering you! Thanks, rin On Mon, Sep 4, 2023 at 10:46 PM Andrius V wrote: > > Hi, > > Thanks for the note. I honestly wasn't aware that we have generated > configs and will be more careful in changing them in the future to > account

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

2023-09-05 Thread Rin Okuyama
On Tue, Sep 5, 2023 at 4:46 AM matthew green wrote: > > > I did similar verification for gdb/dist/bfd also. I'd like to > > sync {binutils,gdb}/dist/bfd, but there are huge diffs between > > them. Most of them seem like binutils or gdb specific fixes, > > but I may overlook something... > > > >

re: CVS commit: src/external/gpl3/binutils/dist/bfd

2023-09-04 Thread matthew green
> I did similar verification for gdb/dist/bfd also. I'd like to > sync {binutils,gdb}/dist/bfd, but there are huge diffs between > them. Most of them seem like binutils or gdb specific fixes, > but I may overlook something... > > It must be nice if we could unify two libbfd's. The upstream > uses

Re: CVS commit: src/sys/arch/amiga/conf

2023-09-04 Thread Andrius V
Hi, Thanks for the note. I honestly wasn't aware that we have generated configs and will be more careful in changing them in the future to account that. However, MDINSTALL config seems to be some older/legacy config, which was not generated from GENERIC.in, I believe. I may try to transform into

Re: CVS commit: src/sys/arch/amiga/conf

2023-09-04 Thread Rin Okuyama
Hi, On 2023/08/31 5:17, Andrius Varanavicius wrote: Module Name:src Committed By: andvar Date: Wed Aug 30 20:17:06 UTC 2023 Modified Files: src/sys/arch/amiga/conf: MDINSTALL Log Message: s/Piccalo/Piccolo/ in device description. Some ports including amiga use

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

2023-09-04 Thread Rin Okuyama
On 2023/08/28 19:55, Valery Ushakov wrote: On Mon, Aug 28, 2023 at 00:02:50 +, Rin Okuyama wrote: Log Message: binutils/bfd: Adjust blank line to reduce diff from upstream Thanks a lot for these cleanups! Do we need to apply similar cleanups to the bfd version in gdb?

Re: CVS commit: src/sys/net

2023-09-04 Thread Rin Okuyama
Thanks for kind words! Apparently, we need some test to detect kernel and userland mismatch for bpf(4) header... I'd also like to fix problems by which ATF does not complete on ERL3. Some CPU/memory consuming tests, e.g., lib/libc/regex/t_exhaust, trigger watchdog. I guess something wrong in

re: CVS commit: src/external/gpl3/gcc.old/dist/gcc

2023-09-03 Thread matthew green
"Christos Zoulas" writes: > Module Name: src > Committed By: christos > Date: Sun Sep 3 18:45:26 UTC 2023 > > Modified Files: > src/external/gpl3/gcc.old/dist/gcc: ubsan.c > > Log Message: > remap generated ubsan filename string labels. fixes reproducible builds. > reported by wiz@

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

2023-08-28 Thread Valery Ushakov
On Mon, Aug 28, 2023 at 00:02:50 +, Rin Okuyama wrote: > Log Message: > binutils/bfd: Adjust blank line to reduce diff from upstream Thanks a lot for these cleanups! Do we need to apply similar cleanups to the bfd version in gdb? (external/gpl3/gdb/dist/bfd) -uwe

Re: CVS commit: src/sys/net

2023-08-25 Thread David H. Gutteridge
Module Name:src Committed By: rin Date: Wed Aug 23 13:21:17 UTC 2023 Modified Files: src/sys/net: bpf.h Log Message: bpf: Fix SIZEOF_BPF_HDR (for LP64 userland) on mips64 It cannot fit within 18 bytes, of course ;) As we had never provided working bpf(4) implementation

Re: CVS commit: src/sys/compat/linux/common

2023-08-25 Thread Theodore Preduta
On 2023-08-25 13:30, Taylor R Campbell wrote: > Since VOP_READDIR requires vp to be locked, I can infer that the > handle_write caller must already hold vp locked. But that means that > we have ifd_lock -> vnode lock in one path, and vnode lock -> ifd_lock > in another path, which is forbidden

Re: CVS commit: src/sys/compat/linux/common

2023-08-25 Thread Taylor R Campbell
> Date: Fri, 25 Aug 2023 13:38:02 -0400 > From: Theodore Preduta > > On 2023-08-25 13:13, Taylor R Campbell wrote: > > This can't be right, and it's a little unsettling that the problem > > isn't caught by any automatic tests. > > > > As I understand it, the `ie_name' member is supposed to

Re: CVS commit: src/sys/compat/linux/common

2023-08-25 Thread Theodore Preduta
On 2023-08-25 13:13, Taylor R Campbell wrote: >> Module Name:src >> Committed By: christos >> Date: Wed Aug 23 19:17:59 UTC 2023 >> >> Modified Files: >> src/sys/compat/linux/common: linux_inotify.c >> >> Log Message: >> put variable length structure at the end, so that

Re: CVS commit: src/sys/compat/linux/common

2023-08-25 Thread Taylor R Campbell
> Module Name:src > Committed By: christos > Date: Thu Aug 24 19:51:24 UTC 2023 > > Modified Files: > src/sys/compat/linux/common: linux_inotify.c > > Log Message: > fix a locking bug (Theodore Preduta) > > if (needs_lock) > vn_lock(vp, LK_SHARED

Re: CVS commit: src/sys/compat/linux/common

2023-08-25 Thread Taylor R Campbell
> Module Name:src > Committed By: christos > Date: Wed Aug 23 19:17:59 UTC 2023 > > Modified Files: > src/sys/compat/linux/common: linux_inotify.c > > Log Message: > put variable length structure at the end, so that clang does not complain. > > struct inotify_entry { >

Re: CVS commit: src/sys/compat/linux

2023-08-21 Thread Christos Zoulas
Yes, I committed it (I had the change in my tree). Best, christos > On Aug 21, 2023, at 3:40 PM, Ryo ONODERA wrote: > > Hi, > > "Christos Zoulas" writes: > >> Module Name: src >> Committed By:christos >> Date:Sun Aug 20 18:08:57 UTC 2023 >> >> Modified Files: >>

Re: CVS commit: src/sys/compat/linux

2023-08-21 Thread Ryo ONODERA
Hi, "Christos Zoulas" writes: > Module Name: src > Committed By: christos > Date: Sun Aug 20 18:08:57 UTC 2023 > > Modified Files: > src/sys/compat/linux: files.linux > > Log Message: > add inotify (forgot to commit this) > > > To generate a diff of this commit: > cvs rdiff -u

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

2023-08-19 Thread Rin Okuyama
Ah, it must be better. Thank you for useful comment as always :) I've confirmed the same files are generated. I will commit soon, but commit mail seems to be delayed for some reasons... Thanks, rin On Sun, Aug 20, 2023 at 11:17 AM Valery Ushakov wrote: > > On Sun, Aug 20, 2023 at 02:02:40

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

2023-08-19 Thread Valery Ushakov
On Sun, Aug 20, 2023 at 02:02:40 +, Rin Okuyama wrote: > Modified Files: > src/external/gpl3/gdb/dist/gnulib: configure > src/external/gpl3/gdb/dist/gnulib/import/m4: rename.m4 > > Log Message: > gdb/gnulib: ``guess yes'' to rename(2) checks for NetBSD Can't we just set

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

2023-08-17 Thread Rin Okuyama
> For vax, something still wrong. This was due to my test environment built by GCC 12. For system built by GCC 10, GDB 13 works just fine (except for C++ exception handling), as far as I can see. Thanks, rin On Thu, Aug 17, 2023 at 4:45 PM Rin Okuyama wrote: > > Module Name:src > Committed

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

2023-08-17 Thread Rin Okuyama
This was not correct. Core file support for NetBSD/riscv was present for our local version of GDB 11, and lost during merge. Thanks, rin On Thu, Aug 17, 2023 at 4:37 PM Rin Okuyama wrote: > > Module Name:src > Committed By: rin > Date: Thu Aug 17 07:37:36 UTC 2023 > > Modified

Re: CVS commit: src/external/gpl3/binutils.old/dist/bfd

2023-08-17 Thread Rin Okuyama
> binutils/bfd: Correct auxv offset for NetBSD, from gdb/bfd binutils.old/bfd for this commit... rin On Thu, Aug 17, 2023 at 3:49 PM Rin Okuyama wrote: > > Module Name:src > Committed By: rin > Date: Thu Aug 17 06:49:01 UTC 2023 > > Modified Files: >

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

2023-08-16 Thread Rin Okuyama
> gcc/ppc: Register NetBSD OSABI for rs6000, lost during merge oops, apparently "gdb/ppc", not "gcc". Thanks, rin On Thu, Aug 17, 2023 at 2:53 PM Rin Okuyama wrote: > > Module Name:src > Committed By: rin > Date: Thu Aug 17 05:53:45 UTC 2023 > > Modified Files: >

Re: CVS commit: src

2023-08-14 Thread Andrius V
I would agree with Taylor, it is not tested well enough to be enabled by default and mainly unmaintained by now. I guess the better solution would be to attempt to integrate openchrome-drm instead at some point in the future. I think making viadrmums as kernel module is quite a good trade off

Re: CVS commit: src

2023-08-13 Thread Taylor R Campbell
> Date: Thu, 10 Aug 2023 07:42:53 +1000 > from: matthew green > > > Log Message: > > viadrmums(4): build legacy VIA DRM UMS driver module for amd64. > > > > This driver is not built-in by default, thus loadable module can help > > (un)lucky > > if it works, why isn't it in GENERIC as well as a

Re: CVS commit: src

2023-08-13 Thread David Brownlee
On Sat, 12 Aug 2023 at 16:16, Andrius V wrote: > > Hi, > > Sorry, didn't notice the email until today. To answer your question > why it is not enabled in GENERIC by default, I am not sure > (Taylor(?)), it was commented since the day one both on i386 and > amd64. > > However, I have some doubts

Re: CVS commit: src

2023-08-12 Thread Andrius V
Hi, Sorry, didn't notice the email until today. To answer your question why it is not enabled in GENERIC by default, I am not sure (Taylor(?)), it was commented since the day one both on i386 and amd64. However, I have some doubts if it is very relevant driver to enable it. From the context

Re: CVS commit: src

2023-08-11 Thread Christos Zoulas
In article <20230803122447.d5e18f...@cvs.netbsd.org>, Nia Alarie wrote: >-=-=-=-=-=- > >Module Name: src >Committed By: nia >Date: Thu Aug 3 12:24:47 UTC 2023 > >Modified Files: > src/distrib/sets/lists/comp: mi > src/distrib/sets/lists/debug: mi >

Re: CVS commit: src/sys/dev/pci

2023-08-10 Thread Taylor R Campbell
> Date: Thu, 10 Aug 2023 17:42:35 +0900 > From: Kengo NAKAHARA > > Could you tell me how you test this fix for future reference? I asked skrll@ to boot a VM with vmxnet3 and hammer on it with a combination of: 1. dhcp 2. host$ nc -l 54321 /dev/null guest$ nc host 54321 /dev/null 3. for i in

Re: CVS commit: src/sys/dev/pci

2023-08-10 Thread Kengo NAKAHARA
Hi, On 2023/08/10 18:07, Nick Hudson wrote: On 10/08/2023 09:42, Kengo NAKAHARA wrote: Hi, Could you tell me how you test this fix for future reference? He didn't - I did. :) Taylor suggested running with network traffic and doing ifconfig down/up. To generate network traffic Taylor

Re: CVS commit: src/sys/dev/pci

2023-08-10 Thread Nick Hudson
On 10/08/2023 09:42, Kengo NAKAHARA wrote: Hi, Could you tell me how you test this fix for future reference? He didn't - I did. :) Taylor suggested running with network traffic and doing ifconfig down/up. To generate network traffic Taylor suggested host$ nc -l 54321 /dev/null guest$ nc

Re: CVS commit: src/sys/dev/pci

2023-08-10 Thread Kengo NAKAHARA
Hi, Could you tell me how you test this fix for future reference? Thanks, On 2023/08/10 17:24, Taylor R Campbell wrote: Module Name:src Committed By: riastradh Date: Thu Aug 10 08:24:45 UTC 2023 Modified Files: src/sys/dev/pci: if_vmx.c Log Message: vmxnet(4): Fix

re: CVS commit: src

2023-08-09 Thread matthew green
> Log Message: > viadrmums(4): build legacy VIA DRM UMS driver module for amd64. > > This driver is not built-in by default, thus loadable module can help > (un)lucky if it works, why isn't it in GENERIC as well as a module? thanks. .mrg.

re: CVS commit: src/share/mk

2023-08-08 Thread matthew green
please review this. i'll try to figure out tests for everything, though it seems annoying :) https://www.netbsd.org/~mrg/gcc12-use-after-free.diff it should handle all the open use-after-free problems. .mrg. ps: you'll notice no new headers needed for ptrdiff_t usage ;)

re: CVS commit: src/share/mk

2023-08-08 Thread matthew green
matthew green writes: > > > - used = dst - conv->wbuff; > > > + size_t sused = (uintptr_t)dst - (uintptr_t)conv->wbuff; > > > > Any particular reason why there is a cast to uintptr_t here? I don't > > think there is a guarantee that you can calculate an offset by > > subtracting

re: CVS commit: src/share/mk

2023-08-08 Thread matthew green
> > - used = dst - conv->wbuff; > > + size_t sused = (uintptr_t)dst - (uintptr_t)conv->wbuff; > > Any particular reason why there is a cast to uintptr_t here? I don't > think there is a guarantee that you can calculate an offset by > subtracting uintptr_ts calculated from

Re: CVS commit: src/share/mk

2023-08-08 Thread Rhialto
On Tue 08 Aug 2023 at 14:10:41 +0200, Joerg Sonnenberger wrote: > On Tue, Aug 08, 2023 at 01:42:39PM +0200, Rhialto wrote: > > On Tue 08 Aug 2023 at 09:44:41 +1000, matthew green wrote: > > > Index: lib/libedit/chartype.c > > > === >

Re: CVS commit: src/share/mk

2023-08-08 Thread Joerg Sonnenberger
On Tue, Aug 08, 2023 at 01:42:39PM +0200, Rhialto wrote: > On Tue 08 Aug 2023 at 09:44:41 +1000, matthew green wrote: > > Index: lib/libedit/chartype.c > > === > > RCS file: /cvsroot/src/lib/libedit/chartype.c,v > > retrieving

Re: CVS commit: src/share/mk

2023-08-08 Thread Rhialto
On Tue 08 Aug 2023 at 09:44:41 +1000, matthew green wrote: > Index: lib/libedit/chartype.c > === > RCS file: /cvsroot/src/lib/libedit/chartype.c,v > retrieving revision 1.36 > diff -p -u -r1.36 chartype.c > --- lib/libedit/chartype.c

Re: CVS commit: src/sys/arch

2023-08-08 Thread Joerg Sonnenberger
On Tue, Aug 08, 2023 at 04:01:19PM +1000, matthew green wrote: > Joerg Sonnenberger writes: > > On Thu, Aug 03, 2023 at 08:16:31AM +, matthew green wrote: > > > Module Name: src > > > Committed By: mrg > > > Date: Thu Aug 3 08:16:31 UTC 2023 > > > > > > Modified Files: >

Re: CVS commit: src/share/mk

2023-08-08 Thread Taylor R Campbell
> Date: Mon, 7 Aug 2023 23:58:50 +0200 > From: Tobias Nygren > > Is this sort of fix acceptable for the above cases? > > + ptrdiff_t offset = pos - buf; > new_buf = realloc(buf, buf_size); > if (!new_buf) >

re: CVS commit: src/sys/arch

2023-08-08 Thread matthew green
Joerg Sonnenberger writes: > On Thu, Aug 03, 2023 at 08:16:31AM +, matthew green wrote: > > Module Name:src > > Committed By: mrg > > Date: Thu Aug 3 08:16:31 UTC 2023 > > > > Modified Files: > > src/sys/arch/evbarm/gumstix: gumstix_machdep.c > >

re: CVS commit: src/share/mk

2023-08-07 Thread matthew green
Valery Ushakov writes: > On Mon, Aug 07, 2023 at 23:58:50 +0200, Tobias Nygren wrote: > > > Is this sort of fix acceptable for the above cases? > [...] > > + ptrdiff_t offset = pos - buf; > [...] > > - pos = new_buf + (pos - buf); > > + pos =

Re: Restoring pointers after realloc (was: Re: CVS commit: src/share/mk)

2023-08-07 Thread Valery Ushakov
On Tue, Aug 08, 2023 at 00:44:41 +0200, Roland Illig wrote: > Am 07.08.2023 um 23:58 schrieb Tobias Nygren: > > Is this sort of fix acceptable for the above cases? > > > > + ptrdiff_t offset = pos - buf; > > new_buf = realloc(buf, buf_size); > > - pos = new_buf + (pos - buf); > > + pos

Re: CVS commit: src/share/mk

2023-08-07 Thread Valery Ushakov
On Mon, Aug 07, 2023 at 23:58:50 +0200, Tobias Nygren wrote: > Is this sort of fix acceptable for the above cases? [...] > + ptrdiff_t offset = pos - buf; [...] > - pos = new_buf + (pos - buf); > + pos = new_buf + offset; I think so.

Restoring pointers after realloc (was: Re: CVS commit: src/share/mk)

2023-08-07 Thread Roland Illig
Am 07.08.2023 um 23:58 schrieb Tobias Nygren: > Is this sort of fix acceptable for the above cases? > > + ptrdiff_t offset = pos - buf; > new_buf = realloc(buf, buf_size); > - pos = new_buf + (pos - buf); > + pos = new_buf + offset; Yes. Instead of ptrdiff_t, I prefer to use

Re: CVS commit: src/share/mk

2023-08-07 Thread Tobias Nygren
On Thu, 3 Aug 2023 23:30:31 +0900 Rin Okuyama wrote: > On 2023/08/03 23:23, Valery Ushakov wrote: > > On Thu, Aug 03, 2023 at 13:33:27 +, Rin Okuyama wrote: > > > >> -Wuse-after-free for GCC 12 is premature. It fires on a common idiom: > >> > >>newbuf = realloc(buf, size); > >>p =

Re: CVS commit: src/sys/arch/sun2/dev

2023-08-04 Thread Rin Okuyama
On 2023/08/04 20:18, Rin Okuyama wrote: Module Name:src Committed By: rin Date: Fri Aug 4 11:18:18 UTC 2023 Modified Files: src/sys/arch/sun2/dev: sc_mbmem.c Log Message: sun2/sc(4): Fix panic due to wrong ENOMEM for DMA buffer Use kmem_zalloc(9) for

re: CVS commit: src/share/mk

2023-08-03 Thread matthew green
> > Ah, I only thought about "obvious" impl. Thank you for kind > > explanation! I will revert them for now. > > We should fix those cases that gcc12 found. i was able to fix some of them with some code to apply offsets generated before realloc() or free(), but not in all case. a couple of them

Re: CVS commit: src/sys/arch

2023-08-03 Thread Joerg Sonnenberger
On Thu, Aug 03, 2023 at 08:16:31AM +, matthew green wrote: > Module Name: src > Committed By: mrg > Date: Thu Aug 3 08:16:31 UTC 2023 > > Modified Files: > src/sys/arch/evbarm/gumstix: gumstix_machdep.c > src/sys/arch/evbarm/ixm1200: ixm1200_machdep.c >

Re: CVS commit: src/share/mk

2023-08-03 Thread Valery Ushakov
On Thu, Aug 03, 2023 at 23:30:31 +0900, Rin Okuyama wrote: > On 2023/08/03 23:23, Valery Ushakov wrote: > > On Thu, Aug 03, 2023 at 13:33:27 +, Rin Okuyama wrote: > > > > > -Wuse-after-free for GCC 12 is premature. It fires on a common idiom: > > > > > > newbuf = realloc(buf, size); > > >

Re: CVS commit: src/share/mk

2023-08-03 Thread Rin Okuyama
On 2023/08/03 23:23, Valery Ushakov wrote: On Thu, Aug 03, 2023 at 13:33:27 +, Rin Okuyama wrote: -Wuse-after-free for GCC 12 is premature. It fires on a common idiom: newbuf = realloc(buf, size); p = newbuf + (p - buf); Let shut this up for GCC 12 (with hoping it gets

Re: CVS commit: src/share/mk

2023-08-03 Thread Valery Ushakov
On Thu, Aug 03, 2023 at 13:33:27 +, Rin Okuyama wrote: > -Wuse-after-free for GCC 12 is premature. It fires on a common idiom: > > newbuf = realloc(buf, size); > p = newbuf + (p - buf); > > Let shut this up for GCC 12 (with hoping it gets improved for 13!). C99 says J.2

Re: CVS commit: src

2023-08-02 Thread Christos Zoulas
Let's not make a mountain out of a molehill. Thanks Ryo for looking into it and the changes are good. In fact I had forgotten to commit the: M dist/libsanitizer/sanitizer_common/sanitizer_platform_limits_netbsd.h And the others worked for me because of the compat name stuff that I added as

Re: CVS commit: src

2023-08-02 Thread Ryo ONODERA
Hi, I have no strong opinion about this change. I just want to unbreak the build to test the latest epoll(2) related side effects for pkgsrc packages. Thank you. On Wed, Aug 2, 2023 at 4:20 PM Taylor R Campbell wrote: > > Let's please just revert all these ioctl renames now, and go back to >

Re: CVS commit: src

2023-08-02 Thread Taylor R Campbell
Let's please just revert all these ioctl renames now, and go back to the drawing board, rather than continue to dig ourselves into a deeper hole with unnecessary incremental cosmetic changes that are breaking all the builds. I generally agree that calling something plain `old' is bad and the

re: CVS commit: src

2023-08-02 Thread matthew green
Ryo ONODERA writes: > I have the same build failure. > I think that there are some typos and inconsistencies. > > With the attached patch, I can finish build.sh build successfully. this looks like what i expected. the before / after showed fewer of the new names. please commit. we'll need

Re: CVS commit: src

2023-08-02 Thread Ryo ONODERA
Hi, I have the same build failure. I think that there are some typos and inconsistencies. With the attached patch, I can finish build.sh build successfully. Thank you. On Wed, Aug 2, 2023 at 2:32 AM Christos Zoulas wrote: > > I am looking into it. With new code that restores the old names it

Re: CVS commit: src

2023-08-01 Thread Christos Zoulas
Bumped. christos > On Aug 1, 2023, at 2:17 AM, matthew green wrote: > > "Christos Zoulas" writes: >> Module Name: src >> Committed By:christos >> Date:Mon Jul 31 17:38:28 UTC 2023 >> >> Modified Files: >> src/distrib/sets/lists/comp: mi >> src/include:

  1   2   3   4   5   6   7   8   9   10   >