Re: [patch] lpr style

Fritjof Bornebusch wrote: > Hi, > > this diff changes the following: > > - exit(3) to return at the end of main functions > - use /* NOTREACHED */ were it belongs according to style(9) I think that's obsolete. We no longer use lint. It's just noise now.

ifdef DIAGNOSTIC in azalia.c

Hi tech@. If there is a need to debug something in azalia.c, defining DIAGNOSTIC is overkill so replace two instances of DIAGNOSTIC with AZALIA_DEBUG (DPRINTF->printf suggested by ratchov@). Also, entirely remove 3rd instance of DIAGNOSTIC. Normally it is not compiled and, ratchov@ thinks,

mg(1) filevisit and dired mode

hi, If you find-file (C-x C-f) and select a directory instead of a file, mg will eventually call dired but due to the current location of the call to dired in mg, find-file calls showbuffer() multiple times. Apart from being wastefull, doing this results in the cursor ending up on the wrong line

Simply use rtdeletemsg()

This brings nd6_free() in line with arptfree() and adds a userland notification for free. ok? Index: netinet6/nd6.c === RCS file: /cvs/src/sys/netinet6/nd6.c,v retrieving revision 1.154 diff -u -p -r1.154 nd6.c --- netinet6/nd6.c

rcctl action multiple daemons

Hello tech@, I don't always want all daemons running all the time on my desktop, but when I want to start multiple daemons (for instance when doing some webdev) I have to call rcctl multiple times, which results in multiple password questions and long commands (I know I could set up doas.conf

Re: Black screen with inteldrm and recent snapshots

> Date: Fri, 25 Sep 2015 12:47:15 +0200 > From: Gregor Best > > Hi people, > > I just updated to a recent snapshot and once inteldrm attaches to my > > vga1 at pci0 dev 2 function 0 "Intel GM45 Video" rev 0x07 > > the screen turns blank. Disabling inteldrm with boot

[patch] lpr style

Hi, this diff changes the following: - exit(3) to return at the end of main functions - use /* NOTREACHED */ were it belongs according to style(9) - lpc.c and lpd.c lack a return at the end of the main functions, as the main loops exists the program. I'm not sure if this is a "coders choise"

Dead Link in faq13.html

A link to the media player 'ogle' in Section 13.9 is no longer available, so I removed it because there are still 3 other media player examples. Maybe someone can find a working website for ogle and add it back in? Index: faq13.html

Re: Black screen with inteldrm and recent snapshots

> Date: Fri, 25 Sep 2015 14:05:50 +0200 > From: Gregor Best > > On Fri, Sep 25, 2015 at 01:32:07PM +0200, Mark Kettenis wrote: > > [...] > > I'm already onto that one. For some reason the G45/GM45 chipset goes > > down a codepath that does a null-pointer dereference. > >

syslogd TLS accept

Hi, If syslogd is started with -S, it accepts TLS connections to receive encrypted traffic. The server certificates are taken from /etc/ssl like relayd does. ok? bluhm Index: usr.sbin/syslogd/evbuffer_tls.c === RCS file:

Typo fix in sun3.html

Corrects a typo in the "History" section Index: sun3.html === RCS file: /cvs/www/sun3.html,v retrieving revision 1.58 diff -u -p -r1.58 sun3.html --- sun3.html2 Jul 2015 05:49:04 -1.58 +++ sun3.html25 Sep 2015

Re: deprecating USRTEXT

> Date: Fri, 25 Sep 2015 18:44:07 + > From: Miod Vallat > > USRTEXT used to be the address of the beginning of the a.out code > segment. Today's world is ELF and PIE binaries, and USRTEXT has no > meaning. > > The following diff removes it, as well as trivial or no longer

Re: Dead Link in faq13.html

Hi, abrahs3 wrote on Fri, Sep 25, 2015 at 01:33:29PM -0400: > A link to the media player 'ogle' in Section 13.9 is no longer > available, so I removed it because there are still 3 other media > player examples. Maybe someone can find a working website for > ogle and add it back in? well, the

Invalid HTML entities in upgrade57.html

Using >, & in raw text is invalid HTML. Index: faq/upgrade57.html === RCS file: /cvs/www/faq/upgrade57.html,v retrieving revision 1.8 diff -u -p -r1.8 upgrade57.html --- faq/upgrade57.html 8 Jul 2015 01:11:32 - 1.8 +++

deprecating USRTEXT

USRTEXT used to be the address of the beginning of the a.out code segment. Today's world is ELF and PIE binaries, and USRTEXT has no meaning. The following diff removes it, as well as trivial or no longer true comments. Index: sys/arch/alpha/include/vmparam.h

Typo in sun3.html

This fixes a typo in the "history" section Index: sun3.html === RCS file: /cvs/www/sun3.html,v retrieving revision 1.58 diff -u -p -r1.58 sun3.html --- sun3.html2 Jul 2015 05:49:04 -1.58 +++ sun3.html25 Sep 2015

Re: Invalid HTML entities in upgrade57.html

Kevin Zhang writes: > Using >, & in raw text is invalid HTML. You're right about &, but > does not need to be escaped. -- Anthony J. Bentley

Re: Invalid HTML entities in upgrade57.html

Anthony J. Bentley wrote: > Kevin Zhang writes: > > Using >, & in raw text is invalid HTML. > > You're right about &, but > does not need to be escaped. Even & only needs escaping if the following letters could be interpreted as an entity... Since & is only followed by & and space, that's not

Clarify relevance of PF FAQ

Assuming that what's described applies to the most recent releases. You could also argue that this disclaimer is old enough to be deleted. Index: faq/pf/nat.html === RCS file: /cvs/www/faq/pf/nat.html,v retrieving revision 1.54 diff

Re: Enforce explicit bzeroing in sasyncd(8)

Ping. Michael McConville wrote: > I took the approach of trying to enforce the programmer's intentions. If > anyone thinks some or all of the cases don't make sense, let me know. > > > Index: monitor.c > === > RCS file:

afterboot.8 to reference ntpd(1) enabled by default

Since ntpd(1) is now enabled by default do the following changes make sense? Regards, Index: afterboot.8 === RCS file: /cvs/src/share/man/man8/afterboot.8,v retrieving revision 1.149 diff -u -p -r1.149 afterboot.8 --- afterboot.8 24

Re: afterboot.8 to reference ntpd(1) enabled by default

Hi Rob, No comment on if this is a good idea or not as I'm not an OpenBSD developer, but I left a few comments inline regarding the mdoc semantics. On 09/25/15 16:57, Rob Pierce wrote: > Since ntpd(1) is now enabled by default do the following changes make sense? > > Regards, > > Index:

update: afterboot.8 to reference ntpd(8) by default

Update diff below referencing the correct man page sections and with sentenced starting on a new line. Regards, Index: afterboot.8 === RCS file: /cvs/src/share/man/man8/afterboot.8,v retrieving revision 1.149 diff -u -p -r1.149

Re: rcctl action multiple daemons

On 2015/09/25 13:17, Martijn van Duren wrote: > I don't always want all daemons running all the time on my desktop, but when > I want to start multiple daemons (for instance when doing some webdev) I > have to call rcctl multiple times, which results in multiple password > questions and long

[patch] lpr atoi -> strtonum

Hi, change atoi(3) -> strtonum(3) in lpr(1) and lprm(1). lprm(1) avoids negative numbers to be the first argument by using getopt(3), but supported values like 2.2. --F. Index: lpr/lpr.c === RCS file:

Re: rcctl action multiple daemons

On 09/25/15 14:03, Stuart Henderson wrote: On 2015/09/25 13:17, Martijn van Duren wrote: I don't always want all daemons running all the time on my desktop, but when I want to start multiple daemons (for instance when doing some webdev) I have to call rcctl multiple times, which results in

Re: Black screen with inteldrm and recent snapshots

On Fri, Sep 25, 2015 at 01:32:07PM +0200, Mark Kettenis wrote: > [...] > I'm already onto that one. For some reason the G45/GM45 chipset goes > down a codepath that does a null-pointer dereference. > [...] If there's any way I can help, please let me know. -- Gregor

Black screen with inteldrm and recent snapshots

Hi people, I just updated to a recent snapshot and once inteldrm attaches to my vga1 at pci0 dev 2 function 0 "Intel GM45 Video" rev 0x07 the screen turns blank. Disabling inteldrm with boot -c prevents that, but then of course X won't start. I have already tried reverting the "Enable

Re: gif(4) cleanup

My tunnels are working as well. //mxb > On 25 sep. 2015, at 11:38, Martin Pieuchot wrote: > > As discussed in Calgary I don't think we need 6 different files for > gif(4). None of them are standalone. Since all our other pseudo- > drivers are self-contained, let's do the

Re: mpsafe ip_carp

On 13/09/15(Sun) 18:34, David Gwynne wrote: > i did this yesterday, but havent had a chance to beat on it properly > yet. > > if anyone would like to give it a go it would be much appreciated. > im particularly interested in stability while carp configuration > is made or changed. if it happens

Re: [patch] cleaner checksum modification for pf

On 2015/09/25 13:05, Richard Procter wrote: > > Well we've been thru it more than once; the argument presented here > > was that modifying the cksum instead of verify + recalc is better as > > it wouldn't hide cksum mismatches if the cksum engines on the NICs we > > offload to misbehave. After

gif(4) cleanup

As discussed in Calgary I don't think we need 6 different files for gif(4). None of them are standalone. Since all our other pseudo- drivers are self-contained, let's do the same! This will prevent new pseud-drivers to copy this pattern and add yet another 6 files! This merge the following

Re: Black screen with inteldrm and recent snapshots

> If there's any way I can help, please let me know. > I am also experiencing the same thing with my Thinkpad X200 (Intel GM45) and can be reached if assistance is needed.