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

-BEGIN PGP SIGNED MESSAGE- Hash: SHA256 On 09.08.2015 15:29, Joerg Sonnenberger wrote: On Tue, Jul 28, 2015 at 05:13:34PM +, Kamil Rytarowski wrote: Module Name: src Committed By: kamil Date: Tue Jul 28 17:13:34 UTC 2015 Modified Files: src/lib/libc/stdlib

Re: libutil shlib_version lossage

-BEGIN PGP SIGNED MESSAGE- Hash: SHA256 On 14.08.2015 08:53, Christos Zoulas wrote: In article 20150814032236.1da9260...@jupiter.mumble.net, Taylor R Campbell campbell+netbsd-source-change...@mumble.net wrote: Whisky tango foxtrot? Did someone botch a cvs admin? Looks that way,

Re: CVS commit: src

-BEGIN PGP SIGNED MESSAGE- Hash: SHA256 On 26.07.2015 09:45, matthew green wrote: matthew green writes: Kamil Rytarowski writes: Module Name:src Committed By: kamil Date: Sun Jul 26 02:20:30 UTC 2015 Modified Files: src/distrib/sets/lists/comp: mi src

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

-BEGIN PGP SIGNED MESSAGE- Hash: SHA256 On 16.07.2015 10:25, Pierre Pronchery wrote: Hi, On 07/16/15 02:04, Kamil Rytarowski wrote: Module Name: src Committed By: kamil Date: Thu Jul 16 00:04:00 UTC 2015 Modified Files: src/lib/libc/stdlib: reallocarr.c Log

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

-BEGIN PGP SIGNED MESSAGE- Hash: SHA256 On 19.07.2015 20:56, Joerg Sonnenberger wrote: On Thu, Jul 16, 2015 at 12:12:27PM +0200, Kamil Rytarowski wrote: The C standard permits memcpy(3) to affect errno(2). More like it hasn't explicitly ruled it out. That might or might not be seen

Re: CVS commit: src/lib/libpanel

-BEGIN PGP SIGNED MESSAGE- Hash: SHA256 On 27.10.2015 00:09, Valeriy E. Ushakov wrote: > Module Name: src Committed By: uwe Date: Mon Oct 26 > 23:09:50 UTC > 2015 > > Added Files: src/lib/libpanel: Makefile _deck.c above.c below.c > bottom.c del.c getuser.c hidden.c

Re: CVS commit: src/sys/kern

-BEGIN PGP SIGNED MESSAGE- Hash: SHA256 On 11.09.2015 03:23, Christos Zoulas wrote: > Module Name: src Committed By: christos Date: Fri Sep 11 > 01:23:37 > UTC 2015 > > Modified Files: src/sys/kern: kern_exec.c > > Log Message: On non absolute exec pathnames, prepend

Re: CVS commit: src/lib/libpanel

-BEGIN PGP SIGNED MESSAGE- Hash: SHA256 On 16.11.2015 12:25, Valery Ushakov wrote: > On Mon, Nov 16, 2015 at 00:00:48 +0100, Kamil Rytarowski wrote: > >> On 27.10.2015 00:09, Valeriy E. Ushakov wrote: >>> Module Name:src Committed By: uwe Date:

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

-BEGIN PGP SIGNED MESSAGE- Hash: SHA256 On 12.01.2016 22:43, David Laight wrote: > On Sat, Nov 14, 2015 at 06:40:21AM +1100, matthew green wrote: >> Christos Zoulas writes: >>> In article <2015111344.ga13...@britannica.bec.de>, Joerg >>> Sonnenberger wrote:

Re: CVS import: othersrc/external/mit/micropython

-BEGIN PGP SIGNED MESSAGE- Hash: SHA256 On 14.01.2016 02:38, Alistair G. Crooks wrote: > Module Name: othersrc Committed By: agc Date: Thu Jan 14 > 01:38:53 > UTC 2016 > > Update of /cvsroot/othersrc/external/mit/micropython In directory >

Re: CVS commit: src

On 04.08.2016 08:43, Christos Zoulas wrote: > Module Name: src > Committed By: christos > Date: Thu Aug 4 06:43:43 UTC 2016 > > Modified Files: > src/include: limits.h > src/lib/libc/gen: sysconf.c > src/sys/kern: kern_sig.c sys_sig.c > src/sys/sys: signal.h

Re: CVS commit: src

On 17.02.2017 15:03, Robert Elz wrote: > Date:Fri, 17 Feb 2017 01:42:59 + > From: "Kamil Rytarowski" <ka...@netbsd.org> > Message-ID: <20170217014259.e68fef...@cvs.netbsd.org> > > | Modified Files: > | src: UPDAT

Re: CVS commit: src

On 17.02.2017 22:47, Robert Elz wrote: > Date:Fri, 17 Feb 2017 22:28:32 +0100 > From: Kamil Rytarowski <n...@gmx.com> > Message-ID: <1f5043f2-b004-f558-bd66-cc96675ab...@gmx.com> > > > | Thank you for your feedback. I was require

Re: CVS commit: src/distrib/sets/lists/tests

On 25.02.2017 06:31, matthew green wrote: > "Kamil Rytarowski" writes: >> Module Name: src >> Committed By:kamil >> Date:Wed Feb 22 09:09:49 UTC 2017 >> >> Modified Files: >> src/distrib/sets/lists/tests: md.amd64 m

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

On 08.02.2017 19:01, Maya Rashish wrote: > Module Name: src > Committed By: maya > Date: Wed Feb 8 18:01:24 UTC 2017 > > Modified Files: > src/lib/libc/sys: accept.2 > > Log Message: > Document accept4 in accept(2) > In general we don't put POSIX/standard and nonstandard

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

On 23.02.2017 12:33, Robert Elz wrote: > Date:Thu, 23 Feb 2017 13:48:10 +0300 > From:Valery Ushakov <u...@stderr.spb.ru> > Message-ID: <20170223104810.gw20...@pony.stderr.spb.ru> > > | > From: Kamil Rytarowski <n...@gmx.com> &

Re: CVS commit: src/sys

On 07.11.2016 22:39, J. Hannken-Illjes wrote: > My patch contains corruption issues only and it passes ATF and > it passes my stress test which is a bit more than just some fsx. > > As -current currently corrupts file systems we should either fix it very > soon or revert your changes completely.

Re: CVS commit: src/tests/kernel

On 03.11.2016 18:08, Kamil Rytarowski wrote: > > For now I'm focusing on functional tests, which are equivalent to > FreeBSD capabilities. This is why I will reschedule combination of > wait(2) usage functions for later and move to on other ptrace(2) use-cases. > With recent repo

Re: CVS commit: src/tests/kernel

On 06.11.2016 17:56, Robert Elz wrote: > Date:Sun, 6 Nov 2016 16:24:16 + > From: "Kamil Rytarowski" <ka...@netbsd.org> > Message-ID: <20161106162416.95d77f...@cvs.netbsd.org> > > | assert_pid1 asserts that

Re: CVS commit: src

On 06.11.2016 17:38, Robert Elz wrote: > Date:Sun, 6 Nov 2016 15:03:31 + > From: "Kamil Rytarowski" <ka...@netbsd.org> > Message-ID: <20161106150331.25fb2f...@cvs.netbsd.org> > > | The t_wait_noproc_wnohang adds to option

Re: CVS commit: src/tests/kernel

On 06.11.2016 18:26, Nicolas Joly wrote: > On Sun, Nov 06, 2016 at 11:56:31PM +0700, Robert Elz wrote: >> Date:Sun, 6 Nov 2016 16:24:16 + >> From:"Kamil Rytarowski" <ka...@netbsd.org> >> Message-ID: <20161106162416.95d77f.

Re: CVS commit: src/tests/kernel

On 10.11.2016 03:28, Paul Goyette wrote: > On Thu, 10 Nov 2016, matthew green wrote: > also, root can't attach to pid1 if securelevel is >= 0. >>> >>> To adjust securelevel this test would need to be modified to run under >>> rump ... We wouldn't want the test to manipulate securelevel of

Re: CVS commit: src/tests/kernel

On 10.11.2016 03:44, matthew green wrote: > it would actually be useful to have a testcase that ran iff > root *and* securelevel >= 0 and tests it is unable to attach > to pid 1. > > thanks. > > > .mrg. > OK, I will have a look at it. signature.asc Description: OpenPGP digital signature

Re: CVS commit: src/sys

On 11.11.2016 11:52, J. Hannken-Illjes wrote: > >> On 10 Nov 2016, at 21:56, Jaromir Dolecek wrote: >> >> Module Name: src >> Committed By:jdolecek >> Date:Thu Nov 10 20:56:32 UTC 2016 >> >> Modified Files: >> src/sys/kern: vfs_wapbl.c >>

Re: CVS commit: src/sys/kern

On 12.11.2016 15:47, Valery Ushakov wrote: > On Fri, Nov 11, 2016 at 12:10:04 -0500, Christos Zoulas wrote: > >> Date:Fri Nov 11 17:10:04 UTC 2016 >> >> Modified Files: >> src/sys/kern: sys_ptrace_common.c >> >> Log Message: >> kern/51621: When attaching to a child send it a

Re: CVS commit: src/tests/kernel

On 25.11.2016 23:41, Christos Zoulas wrote: > In article <o1aegv$210$1...@blaine.gmane.org>, > Christos Zoulas <chris...@astron.com> wrote: >> In article <20161125200105.5dbb4f...@cvs.netbsd.org>, >> Kamil Rytarowski <source-changes-d@NetBSD.org>

Re: CVS commit: src/sbin/mount_puffs

On 23.11.2016 15:33, Masatake Daimon wrote: > Module Name: src > Committed By: pho > Date: Wed Nov 23 14:33:29 UTC 2016 > > Modified Files: > src/sbin/mount_puffs: mount_puffs.8 mount_puffs.c > > Log Message: > Major rework on mount_puffs(8) so that it can actually start file

Re: CVS commit: src/sys/kern

On 13.11.2016 02:44, Kamil Rytarowski wrote: > > > On 13.11.2016 02:39, Robert Elz wrote: >> Date:Sat, 12 Nov 2016 14:42:47 -0500 >> From:"Christos Zoulas" <chris...@netbsd.org> >> Message-ID: <20161112194247.37910f

Re: CVS commit: src/sys/kern

On 13.11.2016 02:39, Robert Elz wrote: > Date:Sat, 12 Nov 2016 14:42:47 -0500 > From:"Christos Zoulas" > Message-ID: <20161112194247.37910f...@cvs.netbsd.org> > > | PR/51624: Return the original parent for a traced process. > > Maybe the

Re: CVS commit: src/sys/kern

On 11.11.2016 18:10, Christos Zoulas wrote: > Module Name: src > Committed By: christos > Date: Fri Nov 11 17:10:04 UTC 2016 > > Modified Files: > src/sys/kern: sys_ptrace_common.c > > Log Message: > kern/51621: When attaching to a child send it a SIGTRAP not a SIGSTOP like >

Re: CVS commit: src/sys/kern

On 13.11.2016 03:39, Robert Elz wrote: > Date:Sun, 13 Nov 2016 02:44:03 +0100 > From: Kamil Rytarowski <n...@gmx.com> > Message-ID: <332a57da-1ac6-38ed-4fc3-947e2e6ca...@gmx.com> > > | I can add a test for it, comparing old parent identi

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

On 31.10.2016 21:23, Taylor R Campbell wrote: >Date: Mon, 31 Oct 2016 18:29:56 +0100 >From: Kamil Rytarowski <n...@gmx.com> > >pthread_mutex_timedlock(3) is broken and it does not work at all for me, >not as a standard mutex (like pthread_mutex_lock(3), suff

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

On 31.10.2016 17:21, Christos Zoulas wrote: > Module Name: src > Committed By: christos > Date: Mon Oct 31 16:21:23 UTC 2016 > > Modified Files: > src/tests/lib/libpthread: t_mutex.c t_timedmutex.c > > Log Message: > Merge and fix the timed mutex tests to use absolute time. >

Re: CVS commit: src/tests/kernel

On 03.11.2016 15:24, Christos Zoulas wrote: > In article <19801.1478175...@andromeda.noi.kre.to>, > Robert Elz wrote: >> >> Which is actually correct? (That is, which makes more sense, if it is >> not actually specified somewhere.) >> >> Please make the tests test correct

Re: CVS commit: src

On 08.12.2016 12:31, Nathanial Sloss wrote: > Module Name: src > Committed By: nat > Date: Thu Dec 8 11:31:15 UTC 2016 > > Modified Files: > src/etc/etc.aarch64: MAKEDEV.conf > src/etc/etc.algor: MAKEDEV.conf > src/etc/etc.amiga: MAKEDEV.conf >

Re: CVS commit: src/tests/kernel

On 06.12.2016 19:59, Christos Zoulas wrote: > Module Name: src > Committed By: christos > Date: Tue Dec 6 18:59:00 UTC 2016 > > Modified Files: > src/tests/kernel: t_ptrace_wait.c > > Log Message: > switch to using fork so we can see the child output. > atf_utils_fork() still

Re: CVS commit: src/doc

On 02.01.2017 04:44, David Holland wrote: > On Sat, Dec 31, 2016 at 08:57:16PM +0000, Kamil Rytarowski wrote: > > Update TODO.ptrace > > > > Mark exect(3) for removal, there is no use-case for it. exec() is already > > monitored and emits SIGTRAP when traced. >

Re: CVS commit: src/doc

On 02.01.2017 18:54, David Holland wrote: > On Mon, Jan 02, 2017 at 05:18:49AM +0100, Kamil Rytarowski wrote: > > > Historically exect() is used by debuggers to start debuggees. While > > > it's equivalent to using PT_TRACE_ME followed by execve(), I think the > &g

Re: CVS commit: src/sys/kern

On 09.01.2017 04:36, Christos Zoulas wrote: > In article <20170109003130.33807f...@cvs.netbsd.org>, > Kamil Rytarowski <source-changes-d@NetBSD.org> wrote: >> -=-=-=-=-=- >> >> Module Name: src >> Committed By:kamil >> Date:

Re: CVS commit: src/sys/kern

On 09.01.2017 05:06, Christos Zoulas wrote: > On Jan 9, 4:40am, n...@gmx.com (Kamil Rytarowski) wrote: > -- Subject: Re: CVS commit: src/sys/kern > > | It's almost 5 years since commenting out - I assumed that it will rot to > | the point that it might need to be rewritten. A

Re: CVS commit: src/sys/kern

> Sent: Wednesday, March 29, 2017 at 10:09 PM > From: "Christos Zoulas" <chris...@astron.com> > To: source-changes-d@NetBSD.org > Subject: Re: CVS commit: src/sys/kern > > In article <20170329195230.d895af...@cvs.netbsd.org>, > Kamil Ryt

Re: CVS commit: src/sys/kern

> Sent: Wednesday, March 29, 2017 at 11:27 PM > From: "Kamil Rytarowski" <n...@gmx.com> > To: "Christos Zoulas" <chris...@astron.com> > Cc: source-changes-d@NetBSD.org > Subject: Re: CVS commit: src/sys/kern > > > > > Sent: We

Re: CVS commit: src/sys/kern

On 12.02.2017 22:52, Valeriy E. Ushakov wrote: > Module Name: src > Committed By: uwe > Date: Sun Feb 12 21:52:46 UTC 2017 > > Modified Files: > src/sys/kern: exec_elf.c > > Log Message: > netbsd_elf_signature - look at note segments (phdrs) not note > sections. They point to the

Re: CVS commit: src/sys/kern

On 21.04.2017 18:09, Martin Husemann wrote: > On Fri, Apr 21, 2017 at 05:50:41PM +0200, Kamil Rytarowski wrote: > >> .section ".note.netbsd.ident", "", @note > > That is missing an "a" flag: > > .section ".note.netbsd.ident&quo

Re: CVS commit: src/sbin/fsck_ext2fs

On 21.04.2017 19:47, co...@sdf.org wrote: > On Fri, Apr 21, 2017 at 01:33:05PM -0400, Christos Zoulas wrote: >> e2di_block is an array; can't be NULL, (clang) > > I'm guessing this is from > http://releng.netbsd.org/builds/HEAD-llvm/201704191240Z/amd64.build.failed > > >

Re: CVS commit: src/sbin/fsck_ext2fs

On 21.04.2017 20:08, co...@sdf.org wrote: > On Fri, Apr 21, 2017 at 07:46:27PM +0200, Kamil Rytarowski wrote: >> >> It's a valid code. >> > > Other things fail: > http://releng.netbsd.org/builds/HEAD/201704202350Z/evbarm64-aarch64.build.failed > > /home/

Re: CVS commit: src

On 01.01.2019 04:13, Joerg Sonnenberger wrote: >> Log Message: >> Install most of the llvm headers. >> >> This is missing some generated files, llvm/include/IR/*.inc > Can you please not do such changes without proper discussion? I'm *very* > strongly against this. At the very least it is not

Re: CVS commit: src

On 02.01.2019 02:52, Roy Marples wrote: > On 01/01/2019 21:30, Kamil Rytarowski wrote: >> Joerg seems to just want to play with Clang out of the LLVM projects, >> deteriorating experience with the rest blocking this patch. Other people >> need more than that. > >

Re: CVS commit: src

On 01.01.2019 14:33, Martin Husemann wrote: > On Tue, Jan 01, 2019 at 05:45:31AM +0100, Kamil Rytarowski wrote: >> LLVM has no API/ABI stability contract and we are down to just keep >> syncing the headers and libraries with LLVM releases. > > Can you explain this in mor

Re: CVS commit: src

On 01.01.2019 15:07, Joerg Sonnenberger wrote: > On Tue, Jan 01, 2019 at 05:45:31AM +0100, Kamil Rytarowski wrote: >> Regarding usefulness, it's a step forward to reusing builtin toolchain >> for dependencies (xsrc, pkgsrc). > > Installing headers alone is absolutely

Re: CVS commit: src

On 02.01.2019 08:42, Joerg Sonnenberger wrote: > On Tue, Jan 01, 2019 at 10:30:38PM +0100, Kamil Rytarowski wrote: >> Joerg seems to just want to play with Clang out of the LLVM projects, >> deteriorating experience with the rest blocking this patch. Other people >> need mor

Re: Unaligned access in kernel on ARMv6+ (Re: CVS commit: src/sys/dev/usb)

kUBSan detected a number of unaligned accesses in USB code: http://netbsd.org/~kamil/kubsan/0007-boot-real-hardware.txt On 06.01.2019 09:46, Rin Okuyama wrote: > (CC added to port-...@netbsd.org) > > Let me summarize the problem briefly. In axe(4), there is a code where > memcpy() is carried

Re: CVS commit: src/external/mpl/dhcp

On 21.06.2018 13:09, Paul Goyette wrote: > On Thu, 21 Jun 2018, Kamil Rytarowski wrote: > >> Module Name:    src >> Committed By:    kamil >> Date:    Thu Jun 21 11:02:48 UTC 2018 >> >> Modified Files: >> src/external/mpl/dhcp: Makefile.inc &

Re: Pre-branch commits

On 12.01.2019 13:54, Martin Husemann wrote: > On Sat, Jan 12, 2019 at 05:11:31AM +0700, Robert Elz wrote: >> I'd like to suggest a possible solution: Go back to the old way, >> and announce the branch date in advance (with a reasonable >> lead time, not just a day or so, which would change

Re: CVS commit: src/bin/sleep

On 26.01.2019 02:30, Robert Elz wrote: > Date:Fri, 25 Jan 2019 15:51:05 +0100 > From: Kamil Rytarowski > Message-ID: > > | sort(1) > | stat(1) > > Those take no floating point input that I can see. For sort, > its only use of fl

Re: CVS commit: src/bin/sleep

On 26.01.2019 16:05, Martin Husemann wrote: > On Sat, Jan 26, 2019 at 12:28:08PM +0100, Kamil Rytarowski wrote: >>> Those take no floating point input that I can see. For sort, >>> its only use of floats would be sorting files containing them, >>> for which (I ass

Re: CVS commit: src/bin/sleep

On 27.01.2019 05:42, Robert Elz wrote: > Yes, like English... I wasn't previously aware that '.' was ever used > as the grouping char, though I did believe that some locales use a > space for that purpose. I don't know whether there is formality that is followed, but in practice people use no

Re: CVS commit: src/bin/sleep

On 27.01.2019 06:54, Kamil Rytarowski wrote: > On 27.01.2019 05:42, Robert Elz wrote: >> Yes, like English... I wasn't previously aware that '.' was ever used >> as the grouping char, though I did believe that some locales use a >> space for that purpose. >

Re: CVS commit: src/bin/sleep

On 28.01.2019 13:45, Robert Elz wrote: > Date:Mon, 28 Jan 2019 11:12:07 +0100 > From: Kamil Rytarowski > Message-ID: > > | In my perception yes. Such calls won't be portable and can stop to be > | functional with tool upgrade/change. > >

Re: CVS commit: src/sys

On 03.04.2019 14:04, Christos Zoulas wrote: > Yes, but this md copy-pasted code should be handled with an MI function. > If it is all the same don't copy it 13 times! > Nothing to add. This is an intermediate step. Next planned steps in this domain: - assert expected behavior in posix_spawn

Re: CVS commit: src/sys

On 04.04.2019 02:42, Christos Zoulas wrote: > Intermediate or not quality counts It would have been simple enough to > write the function once and call it from 13 places, perhaps even less work! > It does. I'm waiting now on releng test results (the service seems to be returning 503) and I

Re: CVS commit: src

On 04.04.2019 21:35, matthew green wrote: > thanks for updating indent. > >> Log Message: >> Upgrade indent(1) >> >> Merge all the changes from the recent FreeBSD HEAD snapshot >> into our local copy. > > can you use "cvs import" on the original sources so we know what > we changed, etc, making

Re: CVS commit: src/sys

On 04.04.2019 21:32, Christos Zoulas wrote: > In article <86734bad-b0e6-57dc-3e0f-5d7c124fa...@gmx.com>, > Kamil Rytarowski wrote: >> -=-=-=-=-=- >> -=-=-=-=-=- >> >> On 04.04.2019 02:42, Christos Zoulas wrote: >>> Intermediate or not quality co

Re: CVS commit: src

On 06.04.2019 05:06, Jason R Thorpe wrote: > Module Name: src > Committed By: thorpej > Date: Sat Apr 6 03:06:29 UTC 2019 > > Modified Files: > src/distrib/sets/lists/comp: mi > src/distrib/sets/lists/tests: module.mi > src/share/man/man9: Makefile ipi.9 ucas.9 >

Re: CVS commit: src/sys/netinet

On 24.02.2019 19:15, David Holland wrote: > On Sun, Feb 24, 2019 at 05:01:52PM +0000, Kamil Rytarowski wrote: > > Modified Files: > >src/sys/netinet: sctp_asconf.h > > > > Log Message: > > Appease GCC7 in sctp_asconf.h > > > > Do not

Re: CVS commit: src/sys/netinet

On 25.02.2019 11:23, Martin Husemann wrote: > On Mon, Feb 25, 2019 at 01:19:02PM +0300, Valery Ushakov wrote: >> On Mon, Feb 25, 2019 at 06:23:33 +0100, Martin Husemann wrote: >> >>> On Sun, Feb 24, 2019 at 09:43:52PM +0100, Kamil Rytarowski wrote: >>>> I con

Re: CVS commit: src/sys/netinet

On 24.02.2019 20:39, Martin Husemann wrote: > On Sun, Feb 24, 2019 at 07:20:10PM +0100, Kamil Rytarowski wrote: >> On 24.02.2019 19:15, David Holland wrote: >>> On Sun, Feb 24, 2019 at 05:01:52PM +, Kamil Rytarowski wrote: >>> > Modified Files: >>

Re: CVS commit: src/sys/netinet

On 24.02.2019 21:38, Martin Husemann wrote: > On Sun, Feb 24, 2019 at 09:36:55PM +0100, Kamil Rytarowski wrote: >> My only specific change was NetBSD/i386 kernel=GENERIC with kUBSan and >> KCOV enabled. > > This does not answer the question. What does enabling kUBSan

Re: CVS commit: src/sys/netinet

On 24.02.2019 21:43, Kamil Rytarowski wrote: > On 24.02.2019 21:38, Martin Husemann wrote: >> On Sun, Feb 24, 2019 at 09:36:55PM +0100, Kamil Rytarowski wrote: >>> My only specific change was NetBSD/i386 kernel=GENERIC with kUBSan and >>> KCOV enabled. >> >>

Re: CVS commit: src/sys/netinet

On 24.02.2019 23:55, Robert Swindells wrote: > > Kamil Rytarowski wrote: >> Module Name:src >> Committed By: kamil >> Date: Sun Feb 24 17:01:52 UTC 2019 >> >> Modified Files: >>src/sys/netinet: sctp_asconf.h >> >>

Re: CVS commit: src

On 23.02.2019 08:35, Martin Husemann wrote: > On Sat, Feb 23, 2019 at 03:10:07AM +0000, Kamil Rytarowski wrote: >> Module Name: src >> Committed By:kamil >> Date:Sat Feb 23 03:10:06 UTC 2019 >> >> Modified Files: >> src/distrib/

Re: CVS commit: src/external/bsd/jemalloc

On 10.03.2019 10:00, Tom Ivar Helbekkmo wrote: > Christos Zoulas writes: > >> Module Name: src >> Committed By:christos >> Date:Sun Mar 10 02:49:52 UTC 2019 >> >> Modified Files: >> src/external/bsd/jemalloc/dist/src: jemalloc.c >>

Re: CVS commit: src/external/bsd/jemalloc

On 10.03.2019 10:21, Kamil Rytarowski wrote: > On 10.03.2019 10:00, Tom Ivar Helbekkmo wrote: >> Christos Zoulas writes: >> >>> Module Name:src >>> Committed By: christos >>> Date: Sun Mar 10 02:49:52 UTC 2019 >>> &g

Re: CVS commit: src/external/gpl3/gcc/lib

On 08.02.2019 23:06, matthew green wrote: > Module Name: src > Committed By: mrg > Date: Fri Feb 8 22:06:12 UTC 2019 > > Modified Files: > src/external/gpl3/gcc/lib/libasan: Makefile > src/external/gpl3/gcc/lib/libtsan: Makefile > src/external/gpl3/gcc/lib/libubsan:

Re: CVS commit: src/bin/sleep

On 25.01.2019 11:38, Robert Elz wrote: > Date:Fri, 25 Jan 2019 12:30:23 +0300 > From:Valery Ushakov > Message-ID: <20190125093023.gc18...@pony.stderr.spb.ru> > > | As someone who actually have to ecnoutner locales in daily life and > | not just think about them

Re: CVS commit: src/bin/sleep

On 25.01.2019 15:35, Christos Zoulas wrote: > On Jan 25, 8:59pm, k...@munnari.oz.au (Robert Elz) wrote: > -- Subject: Re: CVS commit: src/bin/sleep > > | Sounds entirely reasonable, except the standards won't say anything > | relevant about sleep in particular, as only integers are supported

Re: CVS commit: src/bin/sleep

On 25.01.2019 14:08, Robert Elz wrote: > Date:Fri, 25 Jan 2019 12:38:00 +0100 > From: Kamil Rytarowski > Message-ID: <9790a289-8b3e-e57a-e4d8-c021aa7b1...@gmx.com> > > | I think it's better to keep an uniform way of handling separator in >

Re: CVS commit: src

On 03.02.2019 11:45, matthew green wrote: > Module Name: src > Committed By: mrg > Date: Sun Feb 3 10:45:58 UTC 2019 > > Modified Files: > src/games/backgammon/backgammon: main.c > src/tests/lib/libc/misc: t_ubsan.c > > Log Message: > - avoid multipling a boolean value, use

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

On 04.02.2019 04:10, matthew green wrote: > Module Name: src > Committed By: mrg > Date: Mon Feb 4 03:10:33 UTC 2019 > > Modified Files: > src/tests/lib/libc/misc: Makefile t_ubsan.c > > Log Message: > - revert previous to t_ubsan.c, it is desired behaviour. from kamil. > - use

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

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

Re: CVS commit: src/bin/sleep

On 28.01.2019 01:52, Robert Elz wrote: > The only question is whether there > is any harm in also accepting the 0,2 form when ',' is the > "decimal point". In my perception yes. Such calls won't be portable and can stop to be functional with tool upgrade/change. I have checked that some countries

Re: CVS commit: src

On 04.11.2019 13:45, Joerg Sonnenberger wrote: > Module Name: src > Committed By: joerg > Date: Mon Nov 4 12:45:10 UTC 2019 > > Modified Files: > src/lib/libc/tls: Makefile.inc tls.c > src/libexec/ld.elf_so: Makefile tls.c > src/usr.bin/ldd: Makefile.elf > > Log

CVS commit: src/sys/kern

Module Name:src Committed By: kamil Date: Wed Nov 6 13:07:32 UTC 2019 Modified Files: src/sys/kern: subr_disk_mbr.c Log Message: Avoid unaligned pointer arithmetic in check_label_magic() Replace the logic of calculating the address with with base + offset. Reported by

CVS commit: src/sys/kern

Module Name:src Committed By: kamil Date: Wed Nov 6 13:07:32 UTC 2019 Modified Files: src/sys/kern: subr_disk_mbr.c Log Message: Avoid unaligned pointer arithmetic in check_label_magic() Replace the logic of calculating the address with with base + offset. Reported by

Re: CVS commit: src/sys/kern

On 07.11.2019 14:25, Valery Ushakov wrote: > If the sanitizer does complain about other uses, there is little point > in fixing one instance and not the others. We already agreed with Christos that this is appeasing of GCC. If you want to scan the whole kernel (or whole C) file for more

Re: CVS commit: src/sys/kern

On 07.11.2019 16:49, Martin Husemann wrote: > On Thu, Nov 07, 2019 at 04:45:31PM +0100, Kamil Rytarowski wrote: >> Unfortunately the C committee went into the opposite direction here and >> specified a potential dereference. > > Where? > > Martin > 6.3.2.1 C9

Re: CVS commit: src/sys/kern

On 07.11.2019 16:45, Kamil Rytarowski wrote: > On 07.11.2019 16:26, Martin Husemann wrote: >> On Thu, Nov 07, 2019 at 02:53:08PM +0100, Kamil Rytarowski wrote: >>> On 07.11.2019 14:25, Valery Ushakov wrote: >>>> If the sanitizer does complain about other uses, there

CVS commit: src/sys/kern

Module Name:src Committed By: kamil Date: Thu Nov 7 18:35:41 UTC 2019 Modified Files: src/sys/kern: subr_disk_mbr.c Log Message: Revert src/sys/kern/subr_disk_mbr.c r.1.52 Addressed in a better way in r. 1.53. To generate a diff of this commit: cvs rdiff -u -r1.53

Re: CVS commit: src/sys/kern

On 07.11.2019 17:20, Kamil Rytarowski wrote: > On 07.11.2019 17:08, Martin Husemann wrote: >> On Thu, Nov 07, 2019 at 04:56:16PM +0100, Kamil Rytarowski wrote: >>> 6.3.2.1 C11 >>> >>> 'An lvalue is an expression (with an object type other than void) that

CVS commit: src/sys/kern

Module Name:src Committed By: kamil Date: Thu Nov 7 18:35:41 UTC 2019 Modified Files: src/sys/kern: subr_disk_mbr.c Log Message: Revert src/sys/kern/subr_disk_mbr.c r.1.52 Addressed in a better way in r. 1.53. To generate a diff of this commit: cvs rdiff -u -r1.53

Re: CVS commit: src/sys/kern

On 07.11.2019 17:09, Martin Husemann wrote: > On Thu, Nov 07, 2019 at 09:58:06AM -0600, David Young wrote: >> I think the problem is that if you have the series of statements, >> >> element_t *e = >element; >> >> if (s == NULL) >> return; > > Note that this example

CVS commit: src/sys/kern

Module Name:src Committed By: kamil Date: Thu Nov 7 18:30:27 UTC 2019 Modified Files: src/sys/kern: subr_disk_mbr.c Log Message: Ensure in validate_label() that struct disklabel pointer is 8-byte aligned The label is searched each 4 bytes and can be detected in an

CVS commit: src/sys/kern

Module Name:src Committed By: kamil Date: Thu Nov 7 18:30:27 UTC 2019 Modified Files: src/sys/kern: subr_disk_mbr.c Log Message: Ensure in validate_label() that struct disklabel pointer is 8-byte aligned The label is searched each 4 bytes and can be detected in an

Re: CVS commit: src/sys/kern

On 07.11.2019 17:08, Martin Husemann wrote: > On Thu, Nov 07, 2019 at 04:56:16PM +0100, Kamil Rytarowski wrote: >> 6.3.2.1 C11 >> >> 'An lvalue is an expression (with an object type other than void) that >> potentially designates an object' >> >> This me

Re: CVS commit: src/sys/kern

On 07.11.2019 18:20, Martin Husemann wrote: > On Thu, Nov 07, 2019 at 06:08:40PM +0100, Kamil Rytarowski wrote: >> Please review: >> >> http://netbsd.org/~kamil/patch-00194-disklabel-alignment.txt >> >> This patch works for me. > > Yes, I believe that it

Re: CVS commit: src/sys/kern

On 07.11.2019 19:32, Valery Ushakov wrote: > On Thu, Nov 07, 2019 at 18:08:40 +0100, Kamil Rytarowski wrote: > >> On 07.11.2019 16:45, Kamil Rytarowski wrote: >>> On 07.11.2019 16:26, Martin Husemann wrote: >>>> On Thu, Nov 07, 2019 at 02:53:08PM +0100, Kamil Ryta

Re: CVS commit: src/sys/kern

On 07.11.2019 07:25, Martin Husemann wrote: > On Wed, Nov 06, 2019 at 11:17:23PM +0100, Kamil Rytarowski wrote: >> Technically, I think that this is a real UB. >> >> 6.3.2.3/7 >> A pointer to an object type may be converted to a pointer to a >> different obje

Re: CVS commit: src/sys/kern

On 07.11.2019 11:53, Martin Husemann wrote: > On Thu, Nov 07, 2019 at 11:46:47AM +0100, Kamil Rytarowski wrote: >> Please see my newer mail with rationale and another one with a >> confirmation that this was real UB. > > Confirmation? The dereference in this case happens i

Re: CVS commit: src/sys/kern

On 07.11.2019 13:17, Valery Ushakov wrote: > On Thu, Nov 07, 2019 at 06:02:39 +0100, Kamil Rytarowski wrote: > >> I have checked received the following patch and received a feedback from >> a LLVM developer. >> >> On 07.11.2019 05:47, 'Dmitry Vyukov' via syzkall

Re: CVS commit: src/sys/kern

On 07.11.2019 13:48, Valery Ushakov wrote: > On Thu, Nov 07, 2019 at 13:37:21 +0100, Kamil Rytarowski wrote: > >> On 07.11.2019 13:17, Valery Ushakov wrote: >>> On Thu, Nov 07, 2019 at 06:02:39 +0100, Kamil Rytarowski wrote: >>> As a side note - the C99 standard c

  1   2   3   4   5   6   7   8   9   >