Re: Have installboot default to the device carrying /

2017-11-06 Thread Theo de Raadt
>On Mon, Nov 06, 2017 at 03:24:30PM -0700, Theo de Raadt wrote: >| > Here's a diff that will make it default to the device carrying the >| > root filesystem. Result this: >| >| I disagree strongly with this. installboot is a disruptive operation, >| and you should specify the device. > >Fair

Re: Have installboot default to the device carrying /

2017-11-06 Thread Paul de Weerd
On Mon, Nov 06, 2017 at 03:24:30PM -0700, Theo de Raadt wrote: | > Here's a diff that will make it default to the device carrying the | > root filesystem. Result this: | | I disagree strongly with this. installboot is a disruptive operation, | and you should specify the device. Fair enough. I

Re: vmd: add minimal DHCP support

2017-11-06 Thread Mike Larkin
On Sun, Nov 05, 2017 at 09:34:53PM +0100, Reyk Floeter wrote: > On Thu, Nov 02, 2017 at 10:26:55PM +0100, Reyk Floeter wrote: > > Hi, > > > > the problem got reported a few times and a similar diff was floating > > around: vmd's "local interface" implements a simple auto-magic BOOTP > > server,

Re: how does ld.so work when dlsym'ing a weak symbol (environ)

2017-11-06 Thread Philip Guenther
On Mon, Nov 6, 2017 at 2:37 PM, Christian Kellermann wrote: > > While porting a smalltalk VM to OpenBSD I came across this peculiar > quirk and I don't quite understand what's going on. > > This VM does its FFI calls through dlopen'ing and dlsym'ing the > symbols in question.

Re: [PATCH] amd64/bsd.rd: add growfs(8)

2017-11-06 Thread Job Snijders
Thanks for the feedback. I'll get to work on a MI patch and test on amd64 + i386, then pass it on to you.

Re: [PATCH] amd64/bsd.rd: add growfs(8)

2017-11-06 Thread Theo de Raadt
>I'm not sure the growfs functionality is worth the trouble *for >space-constrained ramdisks*, but it would be nice to have growfs on >regular ramdisks. I agree on that. So please put it into the correct lists files for all the unlimited ramdisks. Job, the situation is a little nit-picky but

how does ld.so work when dlsym'ing a weak symbol (environ)

2017-11-06 Thread Christian Kellermann
Dear list, While porting a smalltalk VM to OpenBSD I came across this peculiar quirk and I don't quite understand what's going on. This VM does its FFI calls through dlopen'ing and dlsym'ing the symbols in question. This seems to work fine for all kinds of functions. However it fails with libc's

Re: xhci: fix off-by-one-trb

2017-11-06 Thread Stefan Sperling
On Mon, Nov 06, 2017 at 08:53:59PM +0200, Paul Irofti wrote: > The following diff fixes the overrun issues I have been seing with isoc > transfers when using video(1). Doesn't mean video(1) works now :) > > OK? I doubt this is correct. I suspect your interrupt storm appears to be fixed only

Re: Have installboot default to the device carrying /

2017-11-06 Thread Theo de Raadt
> Here's a diff that will make it default to the device carrying the > root filesystem. Result this: I disagree strongly with this. installboot is a disruptive operation, and you should specify the device.

Re: ARP/LLINFO: route contains no arp information

2017-11-06 Thread Christian Weisgerber
Martin Pieuchot: > Diff below makes them use rtisvalid(9) instead which check for the > RTF_UP flag. This should make many "route contains no arp information" > messages disappear. > > But I'm afraid this can expose another weird driver/NFS setup were non > UP routes are used, Indeed. Booting

Re: Vlan on T2000

2017-11-06 Thread Mark Kettenis
> From: Andrew Grillet > Date: Mon, 6 Nov 2017 19:20:02 + > > Hi, > > I have been running OBSD6.0 and 6.1 on my T2000. > > I now wish to upgrade to 6.2, and I managed to boot a guest domain from a > miniroot62.fs image > with no problems. > > It is now asking: > >

Vlan on T2000

2017-11-06 Thread Andrew Grillet
Hi, I have been running OBSD6.0 and 6.1 on my T2000. I now wish to upgrade to 6.2, and I managed to boot a guest domain from a miniroot62.fs image with no problems. It is now asking: Which network interface do you wish to configure? (or 'done') [vlan0] Which interface:tag should vlan0 be on?

Re: netinet DPRINTF

2017-11-06 Thread Paul Irofti
On Mon, Nov 06, 2017 at 02:06:58PM +0100, Martin Pieuchot wrote: > I'm auditing code related to pr_input() to take them outside of the > KERNEL_LOCK(). It's hard because many DPRINTF() statements include > the name of the function they are in. Diff below convert most of > them to "%s" +

xhci: fix off-by-one-trb

2017-11-06 Thread Paul Irofti
The following diff fixes the overrun issues I have been seing with isoc transfers when using video(1). Doesn't mean video(1) works now :) OK? Index: xhci.c === RCS file: /cvs/src/sys/dev/usb/xhci.c,v retrieving revision 1.77 diff

Re: document how ospfd interacts with carp

2017-11-06 Thread Jason McIntyre
On Mon, Nov 06, 2017 at 01:26:27PM +0100, Remi Locherer wrote: > Hi, > > ospfd.conf(5) should mention what ospfd does automatically when > configured on carp interfaces. > > ok? > > Remi > > > Index: ospfd.conf.5 > === > RCS

Re: [PATCH] amd64/bsd.rd: add growfs(8)

2017-11-06 Thread Jeremie Courreges-Anglas
On Mon, Nov 06 2017, Jeremie Courreges-Anglas wrote: > On Mon, Nov 06 2017, Jeremie Courreges-Anglas wrote: >> On Sun, Nov 05 2017, Job Snijders wrote: >>> Goodmorning everyone, >>> >>> While quite some resizing scenarios can be done from

Re: [PATCH] amd64/bsd.rd: add growfs(8)

2017-11-06 Thread Jeremie Courreges-Anglas
On Mon, Nov 06 2017, Jeremie Courreges-Anglas wrote: > On Sun, Nov 05 2017, Job Snijders wrote: >> Goodmorning everyone, >> >> While quite some resizing scenarios can be done from within single user >> mode, resizing the root partition requires you to bring

Re: [PATCH] amd64/bsd.rd: add growfs(8)

2017-11-06 Thread Jeremie Courreges-Anglas
On Sun, Nov 05 2017, Job Snijders wrote: > Goodmorning everyone, > > While quite some resizing scenarios can be done from within single user > mode, resizing the root partition requires you to bring your own > growfs(8) binary into the ramdisk environment. The below patch adds >

Re: netinet DPRINTF

2017-11-06 Thread Sebastian Benoit
ok! Martin Pieuchot(m...@openbsd.org) on 2017.11.06 14:06:58 +0100: > I'm auditing code related to pr_input() to take them outside of the > KERNEL_LOCK(). It's hard because many DPRINTF() statements include > the name of the function they are in. Diff below convert most of > them to "%s" +

Re: libfuse: improved command line parsing

2017-11-06 Thread Martin Pieuchot
On 29/10/17(Sun) 12:05, Helg Bredow wrote: > > > [...] > > > + } else if (strchr(o->templ, '%') == NULL) { > > > + *((int *)(data + o->off)) = o->val; > > > > Are you sure you can simply deference "data + o->off" w/o sanity check? > > I don't know what

Re: Have installboot default to the device carrying /

2017-11-06 Thread Paul de Weerd
Now with manpage diff, fixed usage and less magic numbers (thanks to florian@ again for the last two). Paul On Mon, Nov 06, 2017 at 03:58:50PM +0100, Paul de Weerd wrote: | installboot(8) will error out when not given a device to install to: | | [weerd@pom] $ doas installboot | usage:

Re: document how ospfd interacts with carp

2017-11-06 Thread Jeremie Courreges-Anglas
On Mon, Nov 06 2017, Remi Locherer wrote: > Hi, > > ospfd.conf(5) should mention what ospfd does automatically when > configured on carp interfaces. > > ok? ok jca@ > Remi > > > Index: ospfd.conf.5 > === >

Have installboot default to the device carrying /

2017-11-06 Thread Paul de Weerd
installboot(8) will error out when not given a device to install to: [weerd@pom] $ doas installboot usage: installboot [-nv] [-r root] disk [stage1 [stage2]] Here's a diff that will make it default to the device carrying the root filesystem. Result this: [weerd@pom] $ doas obj/installboot -v

Re: netinet DPRINTF

2017-11-06 Thread Mark Kettenis
> Date: Mon, 6 Nov 2017 14:06:58 +0100 > From: Martin Pieuchot > > I'm auditing code related to pr_input() to take them outside of the > KERNEL_LOCK(). It's hard because many DPRINTF() statements include > the name of the function they are in. Diff below convert most of >

Re: xhci: isoc trbs forward traversal

2017-11-06 Thread Martin Pieuchot
On 06/11/17(Mon) 14:12, Paul Irofti wrote: > Hi, > > The following diff makes the trb iteration code easier to read and > understand. I don't know why it needed to be done the other way around. Could you keep xhci_device_generic_start() in sync with this change? With that ok. > General

document how ospfd interacts with carp

2017-11-06 Thread Remi Locherer
Hi, ospfd.conf(5) should mention what ospfd does automatically when configured on carp interfaces. ok? Remi Index: ospfd.conf.5 === RCS file: /cvs/src/usr.sbin/ospfd/ospfd.conf.5,v retrieving revision 1.48 diff -u -p -r1.48

xhci: isoc trbs forward traversal

2017-11-06 Thread Paul Irofti
Hi, The following diff makes the trb iteration code easier to read and understand. I don't know why it needed to be done the other way around. General agreement and support from stsp@. Paul Index: dev/usb/xhci.c === RCS file:

Re: TLS with static non-PIE binaries

2017-11-06 Thread Charles Collicutt
On Sun, Nov 05, 2017 at 04:17:50PM -0800, Philip Guenther wrote: > BTW, in the .s file in your original message you had this line: > .type foo, @object > > Since foo is in an SHF_TLS section, it has to be of type STT_TLS. You're right. But... > Indeed, binutils is silently

Re: optind and friends are already declared

2017-11-06 Thread Jan Stary
If there is any interest in this at all, there's a similar diff for usr.bin/ and usr.sbin/ Jan > On Nov 03 18:34:11, h...@stare.cz wrote: > > Including when using getopt(3) also makes > > extern int opterr, optind, optopt, optreset; > > and friends declared, but many utils redeclare

Re: ieee80211: disable wpa/wep when changing nwid

2017-11-06 Thread Jan Stary
On Nov 05 17:57:25, phess...@openbsd.org wrote: > Changing nwid on a wifi network means it is a new network. By definition > the WPA crypto keys use the nwid as part of the crypto hash. And it is > super unlikely that a differently named network will have the same WEP > key. In that case, you

ARP/LLINFO: route contains no arp information

2017-11-06 Thread Martin Pieuchot
benno@ reported a kernel bug that makes ospfd(8) exit. When a L2 entry is removed the message sent to userland no longer contains the RTF_LLINFO flag. This is inconsistent with what userland listeners expect because the same entry has the flag when it is added. The problem comes to the fact

Re: xhci: dequeue isoc trbs events on under/overrun

2017-11-06 Thread Paul Irofti
On Mon, Nov 06, 2017 at 10:11:59AM +0100, Martin Pieuchot wrote: > On 06/11/17(Mon) 00:24, Paul Irofti wrote: > > Hi, > > > > This is a first step towards hopefully getting uvideo(4) to work with > > xhci(4). On my x260 I get buffer overrun and my machine freezes when > > running video(1) with

Re: xhci: dequeue isoc trbs events on under/overrun

2017-11-06 Thread Paul Irofti
On Mon, Nov 06, 2017 at 10:11:59AM +0100, Martin Pieuchot wrote: > On 06/11/17(Mon) 00:24, Paul Irofti wrote: > > Hi, > > > > This is a first step towards hopefully getting uvideo(4) to work with > > xhci(4). On my x260 I get buffer overrun and my machine freezes when > > running video(1) with

Re: xhci: dequeue isoc trbs events on under/overrun

2017-11-06 Thread Martin Pieuchot
On 06/11/17(Mon) 00:24, Paul Irofti wrote: > Hi, > > This is a first step towards hopefully getting uvideo(4) to work with > xhci(4). On my x260 I get buffer overrun and my machine freezes when > running video(1) with xhci and uvideo debug messages enabled. > Following fixes that. > > The