Re: CVS commit: src/sys/uvm

2017-10-27 Thread Christos Zoulas
On Oct 27, 10:14pm, campbell+netbsd-source-change...@mumble.net (Taylor R Campbell) wrote: -- Subject: Re: CVS commit: src/sys/uvm | > Due to incorrect error recovery mmap requests that were denied due to | > PaX/MPROTECT ended up not cleaning up, which made processes stuck. I | > could fix the c

Re: CVS commit: src/sys/uvm

2017-10-27 Thread Kamil Rytarowski
On 28.10.2017 00:14, Taylor R Campbell wrote: >> Date: Fri, 27 Oct 2017 21:46:48 + (UTC) >> This needs to be pulled up to -8. > > Yes. > I've verified that the reported bug is gone. signature.asc Description: OpenPGP digital signature

Re: CVS commit: src/sys/uvm

2017-10-27 Thread Taylor R Campbell
> Date: Fri, 27 Oct 2017 21:46:48 + (UTC) > From: chris...@astron.com (Christos Zoulas) > > In article , > Paul Goyette wrote: > >Can you please PLEASE provide an actual description of the problem > >you're fixing. > > Due to incorrect error recovery mmap requests that were denied due to >

Re: CVS commit: src/sys/uvm

2017-10-27 Thread Christos Zoulas
In article , Paul Goyette wrote: >On Fri, 27 Oct 2017, Utkarsh Anand wrote: > >> Module Name: src >> Committed By:utkarsh009 >> Date:Fri Oct 27 12:01:08 UTC 2017 >> >> Modified Files: >> src/sys/uvm: uvm_mmap.c >> >> Log Message: >> [syzkaller] Fix for PR #52658 as su

Re: CVS commit: src/sys/uvm

2017-10-27 Thread Paul Goyette
On Fri, 27 Oct 2017, Utkarsh Anand wrote: Module Name:src Committed By: utkarsh009 Date: Fri Oct 27 12:01:08 UTC 2017 Modified Files: src/sys/uvm: uvm_mmap.c Log Message: [syzkaller] Fix for PR #52658 as suggested by riastradh@ The bug was found by Dmitry Vyukov (dvyu.

Re: src/sys/kern/subr_prf.c r1.161

2017-10-27 Thread Christos Zoulas
In article , Christos Zoulas wrote: >In article <85853ac7-a205-b412-2f5b-39101dc51...@gmx.com>, >Kamil Rytarowski wrote: Or better, since we provide the decl anyway: --- a/sys/syz-extract/fetch.go +++ b/sys/syz-extract/fetch.go @@ -144,7 +144,9 @@ var srcTemplate = template.Must(template.New("

Re: src/sys/kern/subr_prf.c r1.161

2017-10-27 Thread Christos Zoulas
In article <85853ac7-a205-b412-2f5b-39101dc51...@gmx.com>, Kamil Rytarowski wrote: > >Fixed in HEAD. printf(9) change has been reverted. It is simple enough to fix in syscaller: Anyway, this needs a lot more work but commenting out the os.Remove() calls (so that you can see what it produces) and

Re: src/sys/kern/subr_prf.c r1.161

2017-10-27 Thread Kamil Rytarowski
On 27.10.2017 14:13, Frédéric Fauberteau wrote: > Hi, > > Since revision 1.161 of src/sys/kern/subr_prf.c, I encounter the > following error: > /home/triaxx/netbsd8/usr/src/sys/kern/subr_lockdebug.c: In function > 'lockdebug_barrier': > /home/triaxx/netbsd8/usr/src/sys/kern/subr_lockdebug.c:683:24

src/sys/kern/subr_prf.c r1.161

2017-10-27 Thread Frédéric Fauberteau
Hi, Since revision 1.161 of src/sys/kern/subr_prf.c, I encounter the following error: /home/triaxx/netbsd8/usr/src/sys/kern/subr_lockdebug.c: In function 'lockdebug_barrier': /home/triaxx/netbsd8/usr/src/sys/kern/subr_lockdebug.c:683:24: error: passing argument 2 of 'lockdebug_dump' from incompati

Re: CVS commit: src/sys

2017-10-27 Thread Kamil Rytarowski
On 27.10.2017 14:05, Joerg Sonnenberger wrote: > On Fri, Oct 27, 2017 at 01:44:18PM +0200, Kamil Rytarowski wrote: >> Just a remainder that casting function pointers to data pointers is a >> bold hack and not portable from the C language point of view... but I >> know that it's there for as the las

Re: CVS commit: src/sys

2017-10-27 Thread Joerg Sonnenberger
On Fri, Oct 27, 2017 at 01:44:18PM +0200, Kamil Rytarowski wrote: > Just a remainder that casting function pointers to data pointers is a > bold hack and not portable from the C language point of view... but I > know that it's there for as the last resort. 3rd party software isn't a > legitimate re

Re: CVS commit: src/sys

2017-10-27 Thread Valery Ushakov
On Fri, Oct 27, 2017 at 19:51:30 +0800, Utkarsh Anand wrote: > > To me this looks like a syzkaller bug. The error message doesn't > > even say where the conflicting declaration is coming from. > >For more details on the issue please visit: >https://github.com/google/syzkaller/issues/399 R

Re: CVS commit: src/sys

2017-10-27 Thread Kamil Rytarowski
On 27.10.2017 13:44, Christos Zoulas wrote: > In article <20171027113720.gb5...@britannica.bec.de>, > Joerg Sonnenberger wrote: >> On Fri, Oct 27, 2017 at 09:59:17AM +, Utkarsh Anand wrote: >>> Module Name:src >>> Committed By: utkarsh009 >>> Date: Fri Oct 27 09:59

Re: CVS commit: src/sys

2017-10-27 Thread Christos Zoulas
In article <20171027113720.gb5...@britannica.bec.de>, Joerg Sonnenberger wrote: >On Fri, Oct 27, 2017 at 09:59:17AM +, Utkarsh Anand wrote: >> Module Name: src >> Committed By:utkarsh009 >> Date:Fri Oct 27 09:59:17 UTC 2017 >> >> Modified Files: >> src/sys/arch/x

Re: CVS commit: src/sys

2017-10-27 Thread Joerg Sonnenberger
On Fri, Oct 27, 2017 at 09:59:17AM +, Utkarsh Anand wrote: > Module Name: src > Committed By: utkarsh009 > Date: Fri Oct 27 09:59:17 UTC 2017 > > Modified Files: > src/sys/arch/x86/x86: intr.c > src/sys/ddb: db_interface.h db_panic.c > src/sys/kern: init_main.c subr_

Re: CVS commit: src/sys

2017-10-27 Thread Valery Ushakov
On Fri, Oct 27, 2017 at 09:59:17 +, Utkarsh Anand wrote: > syzkaller was failing to extract constants because of the above > mentioned issue so I had to redeclare printf in sys/sys/systm.h Was this ever discussed anywhere or even reviewed by your sponsor? It doesn't make me comfortable when

Re: CVS commit: src/sys

2017-10-27 Thread Martin Husemann
On Fri, Oct 27, 2017 at 09:59:17AM +, Utkarsh Anand wrote: > Module Name: src > Committed By: utkarsh009 > Date: Fri Oct 27 09:59:17 UTC 2017 > > Modified Files: > src/sys/arch/x86/x86: intr.c > src/sys/ddb: db_interface.h db_panic.c > src/sys/kern: init_main.c subr_

Re: CVS commit: src/sys

2017-10-27 Thread J. Hannken-Illjes
> On 27. Oct 2017, at 11:59, Utkarsh Anand wrote: > > Module Name: src > Committed By: utkarsh009 > Date: Fri Oct 27 09:59:17 UTC 2017 > > Modified Files: > src/sys/arch/x86/x86: intr.c > src/sys/ddb: db_interface.h db_panic.c > src/sys/kern: init_main.c subr_autoconf

Re: CVS commit: src/sys

2017-10-27 Thread Jared McNeill
On Fri, 27 Oct 2017, Utkarsh Anand wrote: Module Name:src Committed By: utkarsh009 Date: Fri Oct 27 09:59:17 UTC 2017 Modified Files: src/sys/arch/x86/x86: intr.c src/sys/ddb: db_interface.h db_panic.c src/sys/kern: init_main.c subr_autoconf.c subr_disk.c