Re: CVS commit: src/external/apache2/mDNSResponder/dist/mDNSPosix

2018-11-19 Thread Jason Thorpe
Thanks! This has been bothering me for a while, but I hadn’t had a chance to look at it. -- thorpej Sent from my iPhone. > On Nov 19, 2018, at 8:14 AM, Christos Zoulas wrote: > > Module Name:src > Committed By:christos > Date:Mon Nov 19 08:14:28 UTC 2018 > > Modified Files:

Re: CVS commit: src/sys/dev

2018-11-19 Thread Jason Thorpe
Shouldn't this be instead? IMO, pulling in implicitly is generally harmful (there is a lot of namespace pollution there...) > On Nov 19, 2018, at 9:23 AM, Maya Rashish wrote: > > Module Name: src > Committed By: maya > Date: Mon Nov 19 09:23:05 UTC 2018 > > Modified Files: >

Re: CVS commit: src/external/apache2/mDNSResponder/dist/mDNSPosix

2018-11-19 Thread Robert Elz
Date:Mon, 19 Nov 2018 03:14:28 -0500 From:"Christos Zoulas" Message-ID: <20181119081428.def1cf...@cvs.netbsd.org> | Modified Files: | src/external/apache2/mDNSResponder/dist/mDNSPosix: mDNSPosix.c | It is not an error if we did not find any IPv4

Re: CVS commit: src/external/apache2/mDNSResponder/dist/mDNSPosix

2018-11-19 Thread Christos Zoulas
The code can handle having no interfaces, because interfaces can be configured in the future. The problem is that if this function returns an error (ie there were no configured interfaces during startup) the daemon will exit, and this is not desirable. Restructuring the code too much will

Re: CVS commit: src/sys/external/bsd/drm2/include/linux/regulator

2018-11-19 Thread Robert Elz
Date:Mon, 19 Nov 2018 10:48:59 + From:"Maya Rashish" Message-ID: <20181119104859.36f32f...@cvs.netbsd.org> | Module Name:src | Committed By: maya | Date: Mon Nov 19 10:48:59 UTC 2018 | | Modified Files: |

Re: CVS commit: src/sys/external/bsd/drm2/include/linux/regulator

2018-11-19 Thread maya
On Mon, Nov 19, 2018 at 06:24:12PM +0700, Robert Elz wrote: > Date:Mon, 19 Nov 2018 10:48:59 + > From:"Maya Rashish" > Message-ID: <20181119104859.36f32f...@cvs.netbsd.org> > > | Module Name: src > | Committed By: maya > | Date: Mon Nov

Re: CVS commit: src/sys/external/bsd/drm2/include/linux/regulator

2018-11-19 Thread Robert Elz
Date:Mon, 19 Nov 2018 11:29:27 + From:m...@netbsd.org Message-ID: <20181119112927.ga1...@homeworld.netbsd.org> | The linux includes are local shims on NetBSD. Ah, OK. | As I understand it, I can't use userland headers in the kernel. What's | the rule for

Re: CVS commit: src/external/apache2/mDNSResponder/dist/mDNSPosix

2018-11-19 Thread Roy Marples
On 19/11/2018 10:42, Robert Elz wrote: Date:Mon, 19 Nov 2018 03:14:28 -0500 From:"Christos Zoulas" Message-ID: <20181119081428.def1cf...@cvs.netbsd.org> | Modified Files: |src/external/apache2/mDNSResponder/dist/mDNSPosix: mDNSPosix.c | It is not

Re: CVS commit: src/external/apache2/mDNSResponder/dist/mDNSPosix

2018-11-19 Thread Robert Elz
Date:Mon, 19 Nov 2018 12:25:23 + From:Roy Marples Message-ID: <9b4f1054-1005-eac3-cb12-af6227d36...@marples.name> | I can plug a usb stick in later which mdnsd could work with, so xtos's | change is fine. I didn't ever doubt that the change would work, just

Re: CVS commit: src/sys/arch

2018-11-19 Thread Rin Okuyama
On 2018/11/19 19:05, Robert Elz wrote: Module Name:src Committed By: kre Date: Mon Nov 19 10:05:09 UTC 2018 Modified Files: src/sys/arch/amd64/include: frame.h src/sys/arch/i386/include: frame.h src/sys/arch/xen/x86: hypervisor_machdep.c Log Message:

Re: CVS commit: src/sys/arch

2018-11-19 Thread Martin Husemann
On Mon, Nov 19, 2018 at 10:40:38PM +0900, Rin Okuyama wrote: > > - ci->ci_xen_clockf_usermode = USERMODE(regs->if_tf.tf_cs); > > - ci->ci_xen_clockf_pc = regs->if_tf.tf_rip; > > + ci->ci_xen_clockf_usermode = USERMODE(regs->_INTRFRAME_CS); > > + ci->ci_xen_clockf_pc = regs->_INTRFRAME_IP;

Re: CVS commit: src/sys/arch

2018-11-19 Thread Robert Elz
Date:Mon, 19 Nov 2018 14:46:53 +0100 From:Martin Husemann Message-ID: <20181119134653.ga23...@mail.duskware.de> | > Let us stop committing untested codes in order to just fix build. Yes, sorry about that, I should have tested that one. | This also applies to

Re: CVS commit: src/sys/arch

2018-11-19 Thread Martin Husemann
On Mon, Nov 19, 2018 at 10:45:32PM +0700, Robert Elz wrote: > | This also applies to the original commit which broke the build ;-) > > In fairness, as it was, it did work on amd64, just not on i386. Asking > developers to test build every possible port just in case would be a > bit much I