Re: got me a 16-core octeon donated.

2014-05-15 Thread Jasper Lievisse Adriaanse
On Thu, May 15, 2014 at 11:03:10PM +0200, Mark Kettenis wrote: Date: Thu, 15 May 2014 22:04:16 +0200 From: Janne Johansson icepic...@gmail.com After some insight from jasper, I stripped away the randomdata section and voila: Copyright (c) 1982, 1986, 1989, 1991, 1993 The

Re: Typo in Tetris

2014-08-04 Thread Jasper Lievisse Adriaanse
On Sun, Aug 03, 2014 at 02:27:37PM +0100, Sam Hart wrote: Absolutely trivial, but none the less ... It's not a typo if and only if iff was meant. --- tetris-orig/shapes.c 2014-08-03 14:12:09.0 +0100 +++ tetris/shapes.c 2014-08-03 14:12:26.0 +0100 @@ -76,7 +76,7 @@ };

Re: arch/: unitialized vars

2013-06-28 Thread Jasper Lievisse Adriaanse
On Fri, Jun 28, 2013 at 09:11:13AM +0200, Janne Johansson wrote: Now the hard part is to figure out exactly where it should have been initialized and to what. Just setting it to 0 somewhere is not necessarily better. It seems the mips64/mips64/cache_tfp.c one is a proper bug. Looking at the

Re: Allwinner

2013-10-21 Thread Jasper Lievisse Adriaanse
On Fri, Oct 11, 2013 at 11:46:39PM +0300, Artturi Alm wrote: On 10/11/13 20:39, Markus Hennecke wrote: On Sat, 5 Oct 2013, Artturi Alm wrote: Current version attached, extract to /sys/arch/armv7 and read the short notes file, no more out of allwinner/ patches needed thanks to armv7. A20

unbreak ftp progressbar

2014-01-29 Thread Jasper Lievisse Adriaanse
Hi, At least with ftp -Vm the progressbar was messing up the display as it was printing the output on the same line and would wrap around creating displays such as http://pbot.rmdir.de/gOeAYNv30HnQk-4I4xmKZg Seems like typo from the introduction of the -D flag. OK? Index: util.c

Re: YP=no in mk.conf and ypldap

2011-02-09 Thread Jasper Lievisse Adriaanse
On Wed, Feb 09, 2011 at 02:48:54PM -0500, Ted Unangst wrote: On Wed, Feb 9, 2011 at 3:00 AM, Dinar Talypov di...@yantel.ru wrote: While building system with YP=no set in mk.conf I have found a bug: ypldap depends on yp in libc. Thanks, I should be able to commit this later tonight. I

Re: azalia(4) resume diff

2011-03-01 Thread Jasper Lievisse Adriaanse
On Mon, Feb 28, 2011 at 08:56:57PM -0500, Brad wrote: On Mon, Feb 28, 2011 at 11:16:49PM +, Jacob Meuser wrote: some ATI azalia controllers are brojen after resume, as in PR 6550. the following diff gathers most of the pci conf register manipulation done in azalia_pci_attach() into a

Re: pmap prefer diff

2011-04-06 Thread Jasper Lievisse Adriaanse
On Tue, Apr 05, 2011 at 06:42:47PM +0200, Ariane van der Steldt wrote: Hi, So it turns out that my allocator is not capable of handling the pmap_prefer horror. This diff exports the actual parameters of pmap_prefer, so I can make the allocator deal with this intelligently. I need compile

Re: man.conf, pick up ports manpages by default

2011-04-11 Thread Jasper Lievisse Adriaanse
On Mon, Apr 11, 2011 at 09:19:57AM +0100, Stuart Henderson wrote: it's useful for ports developers to be able to read the infrastructure manpages (dpb, pkg_subst, update-patches, etc) without making changes to the default configuration. as with X11R6, this fails cleanly if the relevant

Re: adb(4)/pm_direct.c cleanup

2011-05-03 Thread Jasper Lievisse Adriaanse
On Tue, May 03, 2011 at 08:11:52AM +0530, Martin Pieuchot wrote: On macppc, the pm_* methods are always attached to a PMU (or PMU99), so no need to check for hardware. Tested here with a powerbook6,5. Ok? This appears to be working fine on my PowerBook4,1 (ibook g3), will do more testing

Re: Quick sanity test for sili(4)/ahci(4) diff

2011-05-06 Thread Jasper Lievisse Adriaanse
On Fri, May 06, 2011 at 10:52:19AM -0700, Matthew Dempsky wrote: If you have a sili(4) or ahci(4), I'd appreciate a quick test of the diff below. As long as you can still mount your disks without panicking, then it worked. :) One test for each is sufficient, so please reply to the list to

Re: Bus Pirate: bus hacking tool for hardware developers

2011-05-13 Thread Jasper Lievisse Adriaanse
On Thu, May 12, 2011 at 06:00:53PM +, Jona Joachim wrote: Hi, I just wanted to share this board that I discovered today: http://dangerousprototypes.com/bus-pirate-manual/ It's an uftdi(4) board that gives you access to the following bus protocols: 1-Wire, I2C, SPI, JTAG, RS-232, MIDI,

Re: NELEM() - nitems() in sys/i386/i386/est.c and bin/ksh/

2011-05-13 Thread Jasper Lievisse Adriaanse
On Fri, May 13, 2011 at 03:08:36AM +0400, Vadim Zhukov wrote: Hello all. This diff removes NELEM() definitions in favour of nitems(). Not sure about bin/ksh/ part, but sys/i386/i386/est.c is obviously a leftover. There are more NELEM() items in tree, but they belong to foreign stuff and

Re: switch to xkeyboard-config XKB configuration: please test

2011-05-31 Thread Jasper Lievisse Adriaanse
On Tue, May 31, 2011 at 08:22:09PM +0200, Antoine Jacoutot wrote: On Mon, 30 May 2011, Matthieu Herrb wrote: Hi, Some years ago X.Org decided to switch to a new set of XKB (the X Keyboard extension) data files known as xkeyboard-config. For various reasons (local changes,

Re: [Update] xenocara/xkeyboard-config

2011-06-24 Thread Jasper Lievisse Adriaanse
On Thu, Jun 23, 2011 at 06:08:47PM +0600, Alexandr Shadchin wrote: Hi, I prepared update package xkeyboard-config to the latest release 2.3. Patch available on http://koba.devio.us/distfiles/xkeyboard-config-2.3.diff Tested on amd64. -- Alexandr Shadchin No problems here on amd64 with

Re: minor mount_ntfs cleanup

2011-06-28 Thread Jasper Lievisse Adriaanse
On Mon, Jun 27, 2011 at 06:15:15PM -0400, Kenneth R Westerback wrote: On Mon, Jun 27, 2011 at 04:52:46PM -0400, Ted Unangst wrote: The original porting effort left behind some unneeded bits. I'm not going to encourage NTFS use by ok'ing it. But if I was, I would. Ken Yep, Ok with

Re: kdump: resolve sysctl numbers

2011-07-27 Thread Jasper Lievisse Adriaanse
On Wed, Jul 27, 2011 at 10:58:22AM -0400, Ted Unangst wrote: On Wed, Jul 27, 2011, Otto Moerbeek wrote: +#define SETNAME(name) do { names = (name); limit = nitems(name); } while (0) userland is not supposed to use nitems I think? But it keeps sneaking in because the kernel headers

Re: [PATCH] dired mg patch

2011-08-19 Thread Jasper Lievisse Adriaanse
On Thu, Aug 18, 2011 at 08:30:02AM -0400, Loganaden Velvindron wrote: I used a function for warping the dot. This makes the diff simpler. Since kjell@ has slacked out; any objections to committing this revision of the diff? Index: dired.c

Re: compat_linux: Fix counting in futex_wake.

2012-06-19 Thread Jasper Lievisse Adriaanse
On Tue, Jun 19, 2012 at 12:13:09PM +0300, Paul Irofti wrote: Count should always be zero no matter if we need to relocate or not. Okay? Makes sense to me (including the other diffs you sent which I didn't reply to individually). Index: linux_futex.c

Re: tinyscheme + mg

2012-06-28 Thread Jasper Lievisse Adriaanse
On Thu, Jun 28, 2012 at 08:35:18AM -0400, Kjell Wooding wrote: Thoughts, since we have been down this road before. 1. You can remap keys, in your ~/.mg file 2. I should point out that all of mg (other than theo.c) is currently PUBLIC DOMAIN, not merely BSD, so this change is significant,

Re: tinyscheme + mg

2012-06-28 Thread Jasper Lievisse Adriaanse
On Thu, Jun 28, 2012 at 10:12:38AM -0400, Ted Unangst wrote: On Thu, Jun 28, 2012 at 15:21, Jasper Lievisse Adriaanse wrote: On Thu, Jun 28, 2012 at 08:35:18AM -0400, Kjell Wooding wrote: 2. I should point out that all of mg (other than theo.c) is currently PUBLIC DOMAIN, not merely BSD

Re: Diff to support so-called gaming USB keyboards

2012-07-05 Thread Jasper Lievisse Adriaanse
On Tue, Jul 03, 2012 at 03:23:16PM -0700, Mike Larkin wrote: Many low-cost USB keyboards have a limit of either 3 or 6 simultaneous keypresses before they wedge and stop supplying any more keypress events (at least until you release one of the pressed keys). Some newer (usually called

make mg statusline a bit more informative

2012-10-10 Thread Jasper Lievisse Adriaanse
Currently the statusline in mg shows the line and column numbers, which is nice but doesn't let you know your relative position in the file. Emacs shows 'top', 'bot', 'all' or your relative position like '42%'. This diff implements that behaviour in mg too. From: (fundamental-fill)--L3--C31 to:

Re: make mg statusline a bit more informative

2012-10-10 Thread Jasper Lievisse Adriaanse
On Wed, Oct 10, 2012 at 03:31:30PM +0200, Mark Kettenis wrote: Date: Wed, 10 Oct 2012 15:12:59 +0200 From: Jasper Lievisse Adriaanse jas...@openbsd.org Currently the statusline in mg shows the line and column numbers, which is nice but doesn't let you know your relative position

mg: revert-buffer

2012-10-12 Thread Jasper Lievisse Adriaanse
Hi, Here's a diff that implement revert-buffer (C-x r). I've split gotoline into the 'goto-line' specifics and the code that actually jumps to the line so it can be re-used by revert-buffer to restore the current line. OK? -- Cheers, Jasper Stay Hungry. Stay Foolish Index: basic.c

Re: mg: revert-buffer

2012-10-12 Thread Jasper Lievisse Adriaanse
Seems I broke something between testing and sending out the diff. Updating diff coming later. On Fri, Oct 12, 2012 at 12:40:47PM +0200, Christiano F. Haesbaert wrote: I want this baadlyyy, ill have a look. On Oct 12, 2012 12:30 PM, Jasper Lievisse Adriaanse jas...@openbsd.org wrote: Hi

Re: mg: revert-buffer

2012-10-12 Thread Jasper Lievisse Adriaanse
-buffer +Revert the current buffer to the latest file on disk. .It save-buffer Save the contents of the current buffer if it has been changed, optionally creating a backup copy. On Fri, Oct 12, 2012 at 12:26:53PM +0200, Jasper Lievisse Adriaanse wrote: Hi, Here's a diff that implement

glib threading issues

2012-10-19 Thread Jasper Lievisse Adriaanse
Hi, As some of you may be aware we're having a lot of issues in the glib port related to threading. And it seems it's now affecting non-GNOME ports as well. irssi just crashed on sparc64: (gdb) bt #0 0x0003781ec820 in ?? () from /usr/lib/libpthread.so.17.0 #1 0x000377ee700c in

mg: make-directory

2012-11-15 Thread Jasper Lievisse Adriaanse
Hi, One of the things lacking in mg was support M-x make-directory, which comes quite handy. This diff mimics the rather silent behaviour of Emacs: there's basically no feedback in case creating the directory failed for whatever reason. Should we be more verbose about it, or just stay in line

mg: column-number-mode

2012-12-03 Thread Jasper Lievisse Adriaanse
Hi, Some weeks ago the column number display was removed from the modeline; while it's totally understandable this is a horror on slower displays, it's still usefull on faster displays. mg has a line-number-mode, but not a column-number-mode like Emacs has. This diff adds the column-number-mode,

Re: drm@macppc, please test

2012-12-07 Thread Jasper Lievisse Adriaanse
On Fri, Dec 07, 2012 at 05:28:42AM +0100, J??r??mie Courr??ges-Anglas wrote: Hi, this is seems to work as expected, known issues included. glxgears runs OK but displays some artifacts, and the console is full of garbage after exiting X. I've put the dmesg, Xorg.0.log, etc here:

Re: mg(1): shell-command

2013-05-22 Thread Jasper Lievisse Adriaanse
, argv, text, len, bp); -- Regards, Jasper Lievisse Adriaanse, Engineering team M:tier

Re: mg(1): shell-command

2013-05-22 Thread Jasper Lievisse Adriaanse
, 2013 at 09:32:45AM +0200, Jasper Lievisse Adriaanse wrote: On Tue, May 21, 2013 at 07:54:31PM +, Mark Lumsden wrote: This diff modifies the shell-command-on-region function and gives us shell-command. It makes getting output from other commands into mg really easy. Comments/oks

Re: RFC: changes to ports infrastructure

2010-08-19 Thread Jasper Lievisse Adriaanse
On Thu, Aug 19, 2010 at 11:55:22AM +0200, Marc Espie wrote: I want to tweak the directory structure for ports. I'd like to go for a lot of stuff to ports/infrastructure/bin ports/infrastructure/lib ports/infrastructure/man the current setup (build, fetch, install, package) is my

Re: A tiny feature for mg(1): beginning-of-line

2010-10-06 Thread Jasper Lievisse Adriaanse
Would this be OK with anyone? (With a tweak to start the new sentece on a new line). I've sent this to three usual mg developers, but none bothered to answer. On Sun, Sep 12, 2010 at 08:28:43PM -0700, Chris Palmer wrote: I have found this feature useful in other text editors. Maybe you will

Re: A tiny feature for mg(1): beginning-of-line

2010-10-06 Thread Jasper Lievisse Adriaanse
On Wed, Oct 06, 2010 at 11:37:39AM +, Christian Weisgerber wrote: Jasper Lievisse Adriaanse jas...@humppa.nl wrote: Would this be OK with anyone? -Move cursor to the beginning of the line. +Move cursor to the beginning of the line. Calling this function again moves

systat and livelocks

2010-11-25 Thread Jasper Lievisse Adriaanse
Hi, Currently the 'mbufs' view of systat only shows the difference of livelocks, this diff also adds the actual total (or sum). This is info most people are actually looking for anyway (as noticed by kettenis and sthen yesterday). OK? -- Cheers, Jasper Stay Hungry. Stay Foolish. Index:

drops NENTS in favor of nitems

2010-12-06 Thread Jasper Lievisse Adriaanse
This diff drops another nitems() copy we have in tree, NENTS(). Compile tested on amd64, no binary change. OK? -- Cheers, Jasper During times of universal deceit, telling the truth becomes a revolutionary act. Index: arch/alpha/stand/netboot/if_prom.c

Consistent 'include' rules in 'files.' files.

2010-12-06 Thread Jasper Lievisse Adriaanse
As per subject, tested on amd64/macppc. OK? -- Cheers, Jasper During times of universal deceit, telling the truth becomes a revolutionary act. Index: alpha/conf/files.alpha === RCS file: /cvs/src/sys/arch/alpha/conf/files.alpha,v

config -b needs -s (and visa versa)?

2011-01-05 Thread Jasper Lievisse Adriaanse
Right now, config doesn't work when either -b or -s is specified. It does work when both or none are specified. Does anyone know what the correct behaviour should be? Here is a diff that would at least make this behaviour explicit (if correct, OK?). -- Cheers, Jasper Capable, generous men do

Re: Workaround for data corruption issue with ALI M5229 IDE chip used with Sun Blade 100/Netra X1.

2011-01-13 Thread Jasper Lievisse Adriaanse
On Wed, Jan 12, 2011 at 08:32:12PM -0500, Brad wrote: The following diff is ported from NetBSD (the workaround originated from OpenSolaris) to workaround the issue of data corruption with the ALI M5229 IDE chipset when using UltraDMA. Same workaround is also used by FreeBSD/Linux. This

Re: Workaround for data corruption issue with ALI M5229 IDE chip used with Sun Blade 100/Netra X1.

2011-01-13 Thread Jasper Lievisse Adriaanse
On Thu, Jan 13, 2011 at 09:02:26AM +0100, Jasper Lievisse Adriaanse wrote: On Wed, Jan 12, 2011 at 08:32:12PM -0500, Brad wrote: The following diff is ported from NetBSD (the workaround originated from OpenSolaris) to workaround the issue of data corruption with the ALI M5229 IDE chipset

zap some dead code from bdes(1)

2011-01-20 Thread Jasper Lievisse Adriaanse
As per subject, since #define something test for in the very next line, makes it clear the other code can go. No binary change of course. OK? -- Cheers, Jasper Capable, generous men do not create victims, they nurture them. Index: bdes.c

recieve - receive (sys/)

2011-01-20 Thread Jasper Lievisse Adriaanse
Hi, Obvious typo, though the pxe.h parts may have to wait untill after the lock? Otherwise OK? -- Cheers, Jasper Capable, generous men do not create victims, they nurture them. Index: arch/i386/stand/libsa/pxe.h === RCS file:

recieve - receive (usr.sbin)

2011-01-20 Thread Jasper Lievisse Adriaanse
OK? -- Cheers, Jasper Capable, generous men do not create victims, they nurture them. Index: npppd/l2tp/l2tp_call.c === RCS file: /cvs/src/usr.sbin/npppd/l2tp/l2tp_call.c,v retrieving revision 1.6 diff -p -u -r1.6 l2tp_call.c ---

Re: recieve - receive (sys/)

2011-01-21 Thread Jasper Lievisse Adriaanse
The pxe.h defines are unused elsewhere, btw. On Thu, Jan 20, 2011 at 11:35:34PM +0100, Jasper Lievisse Adriaanse wrote: Hi, Obvious typo, though the pxe.h parts may have to wait untill after the lock? Otherwise OK? -- Cheers, Jasper Capable, generous men do not create victims

Re: spelling gnu/usr.bin/perl/util.c

2009-12-16 Thread Jasper Lievisse Adriaanse
On Tue, Dec 15, 2009 at 08:53:56PM -0500, Brad Tilley wrote: If these are annoying, I will stop sending. please submit this diff to upstream as it's maintained there, not in our tree. cheers, jasper # cvs diff -Nup gnu/usr.bin/perl/util.c Index: gnu/usr.bin/perl/util.c

rename sgi's power(4) to nmi(4)

2010-04-01 Thread Jasper Lievisse Adriaanse
@@ +.\ $OpenBSD$ +.\ +.\ Copyright (c) 2007,2010 Jasper Lievisse Adriaanse jas...@openbsd.org +.\ All rights reserved. +.\ +.\ Permission to use, copy, modify, and distribute this software for any +.\ purpose with or without fee is hereby granted, provided that the above +.\ copyright notice and this permission

Re: syslogd sending via tcp

2014-12-28 Thread Jasper Lievisse Adriaanse
On Sun, Dec 28, 2014 at 05:33:08PM +0100, Alexander Bluhm wrote: Jasper tested and found that it only worked on loopback. I have forgotten to check for EINPROGRESS after connect. So here is a new diff. bluhm Succesfully tested now with a remote logstash host. Index: privsep.c

Re: Do you need/prefer the non-DUID option in the installer?

2015-03-15 Thread Jasper Lievisse Adriaanse
On Sun, Mar 15, 2015 at 11:24:32AM -0400, Kenneth Westerback wrote: Using DUIDs in the installed /etc/fstab has been the default for some time now. We'd like to eliminate the question in the installer and just use DUIDs unconditionally. But first we need to know you are aware of any

Re: [PATCH] pkg-config compare() fails for libevent (at least)

2015-08-12 Thread Jasper Lievisse Adriaanse
On Tue, Aug 11, 2015 at 01:20:24PM -0500, attila wrote: Hello tech@, On the 6 Aug snap I ran into this issue: $ pkg_info | grep libevent libevent-2.0.22 event notification library $ pkg-config --atleast-version=2.0.1 libevent echo yes Argument 22-stabl isn't numeric in

sed -i

2015-07-17 Thread Jasper Lievisse Adriaanse
Hi, Here's a diff to add the '-i' flag to sed to do inplace edits. It's mostly from FreeBSD with some adjustments to prevent a race with unlink() and fopen() during the tempfile creation. It's been tested in a full ports bulk (thanks aja), and went through a build of base and xenocara. Regress

Re: WAPBL implementation

2015-10-28 Thread Jasper Lievisse Adriaanse
On Wed, Oct 28, 2015 at 09:06:54AM -0200, Walter Neto wrote: > Adding WAPBL support for dumpfs(8) > > next diffs: > - tunefs(8) showing log information and setting log size > - fsck_ffs(8) WAPBL support > > ok jasper@ Correction: I only pointed out that we should have the option to somehow get

Re: libgtop2: Use getifaddrs(3) instead of KVM

2015-12-06 Thread Jasper Lievisse Adriaanse
On Sat, Dec 05, 2015 at 05:05:07PM +0100, Martin Pieuchot wrote: > Here's a rewrite of glibtop_get_netload_p(). I tested it with custom > code because I could not trigger this code path with our ports. > > This unbreaks devel/libgtop2 after the recent commit. > > I believe this should go

Re: add EdgeRouter Pro to www/octeon.html

2016-03-19 Thread Jasper Lievisse Adriaanse
On Fri, Mar 18, 2016 at 08:44:11AM +0100, Marcus MERIGHI wrote: > Hello, > > "The patch seems to work fine on an EdgeRouter Pro. > OK visa@" > (http://marc.info/?l=openbsd-tech=145822792814191) > > EdgeRouter Pro is not on octeon.html, put it there? > > Bye, Marcus Last time I checked the

Re: generate pkg-config files at build time

2016-09-13 Thread Jasper Lievisse Adriaanse
On Tue, Sep 13, 2016 at 06:35:08PM +0200, Martin Natano wrote: > Currently pkg-config files are generated at install time, while they > should be generated at build time like everything else. One reason why > generating files during install is bad is that the two steps might be > run by two

libsa, stop saving memory

2016-09-14 Thread Jasper Lievisse Adriaanse
Hi, nothing defines SAVE_MEMORY nor has it been modified since -r1.1. ok to zap it? Index: cread.c === RCS file: /cvs/src/sys/lib/libsa/cread.c,v retrieving revision 1.13 diff -u -p -r1.13 cread.c --- cread.c 18 Jan 2009

db_trace.c: use __func__ instead of hardcoding filename

2016-09-09 Thread Jasper Lievisse Adriaanse
Hi, Do we really want the filename to be printed in the message? If so we should use __FILE__. On the other hand, having the function name makes more sense to me. OK? Index: amd64/amd64/db_trace.c === RCS file:

Re: db_trace.c: use __func__ instead of hardcoding filename

2016-09-09 Thread Jasper Lievisse Adriaanse
On Fri, Sep 09, 2016 at 01:54:46PM -0700, Philip Guenther wrote: > On Fri, Sep 9, 2016 at 11:38 AM, Jasper Lievisse Adriaanse > <jas...@openbsd.org> wrote: > > Do we really want the filename to be printed in the message? If so we should > > use __FILE__. On the other hand,

define ELF_CTF in exec_elf.h

2016-09-16 Thread Jasper Lievisse Adriaanse
Hi, This moves the definition of ELF_CTF into exec_elf.h and adjusts the loadfile_elf.c copies accordingly. OK? Index: share/man/man5/elf.5 === RCS file: /cvs/src/share/man/man5/elf.5,v retrieving revision 1.32 diff -u -p -r1.32

db_{interface,trace}.c: ansify function definitions

2016-09-19 Thread Jasper Lievisse Adriaanse
OK? Index: alpha/alpha/db_trace.c === RCS file: /cvs/src/sys/arch/alpha/alpha/db_trace.c,v retrieving revision 1.18 diff -u -p -r1.18 db_trace.c --- alpha/alpha/db_trace.c 19 Sep 2016 17:59:18 - 1.18 +++

sys/systm.h: drop read_symtab_from_file() proto

2016-09-17 Thread Jasper Lievisse Adriaanse
This seems to be a leftover from when the actual function itself was removed at some point? Index: systm.h === RCS file: /cvs/src/sys/sys/systm.h,v retrieving revision 1.117 diff -u -p -r1.117 systm.h --- systm.h 13 Sep 2016

Re: Remove DDB_STRUCTINFO

2017-09-11 Thread Jasper Lievisse Adriaanse
On Mon, Sep 11, 2017 at 10:01:27AM +, Martin Pieuchot wrote: > Now that dlg@ implemented 'show struct' using CTF, which makes it > available in GENERIC kernel we can retire DDB_STRUCTINFO. > > Diff below does that, ok? I suggested the same dlg, so OK with me. Cheers, Jasper > Index:

Re: [PATCH] urng.4 - Altusmetrum -> Altus Metrum

2017-09-09 Thread Jasper Lievisse Adriaanse
On Fri, Sep 08, 2017 at 10:25:41PM +0100, Raf Czlonka wrote: > Hi all, > > According to their web pages[0][1] Altus Metrum name comprises two > words - both capitalised. > > [0] http://altusmetrum.org/ > [1] http://shop.gag.com/about-magento-demo-store > > Regards, > > Raf This is indeed

combined usb rng driver

2017-08-27 Thread Jasper Lievisse Adriaanse
.4 --- /dev/null 1 Jan 1970 00:00:00 - +++ share/man/man4/urng.4 26 Aug 2017 20:34:54 - @@ -0,0 +1,63 @@ +.\"$OpenBSD$ +.\" +.\" Copyright (c) 2015 Sean Levy <att...@stalphonsos.com> +.\" Copyright (c) 2017 Jasper Lievisse Adriaanse <jas...@openbsd.org> +.\&qu

Re: [PATCH] vm.conf: Clarify VM name constraints

2017-08-31 Thread Jasper Lievisse Adriaanse
On Wed, Aug 30, 2017 at 05:11:26PM -0700, Carlos Cardenas wrote: > Add VM name constraints to match those in vmctl.8 manpage. > > Comments? Ok? Applied, thanks. > diff --git usr.sbin/vmd/vm.conf.5 usr.sbin/vmd/vm.conf.5 > index d1e68dbce5d..77a7a4e8cea 100644 > --- usr.sbin/vmd/vm.conf.5 > +++

w: don't print any header with -h

2017-12-14 Thread Jasper Lievisse Adriaanse
Hi, currently w(1) on OpenBSD differs from other implementations (GNU/Darwin/FreeBSD/SmartOS) in that 'w -h' does print the 'USER TTY FROM ...' header whereas the others don't. Is there a specific reason for it or could this diff below go in? Index: w.c

Re: w: don't print any header with -h

2017-12-14 Thread Jasper Lievisse Adriaanse
On Thu, Dec 14, 2017 at 01:35:18PM +0100, Martijn van Duren wrote: > Hello Jasper, > > On 12/14/17 13:22, Jasper Lievisse Adriaanse wrote: > > Hi, > > > > currently w(1) on OpenBSD differs from other implementations > > (GNU/Darwin/FreeBSD/SmartOS) in that 'w

panic when invalid randomness source is provided

2017-11-05 Thread Jasper Lievisse Adriaanse
Hi, Currently we appear to silently drop randomness if it comes from an invalid source. Should we not panic in this case? While here, update the comment for enqueue_randomness() after -r1.188. OK? Index: rnd.c === RCS file:

Re: panic when invalid randomness source is provided

2017-11-05 Thread Jasper Lievisse Adriaanse
On Sun, Nov 05, 2017 at 06:05:29PM +, Jasper Lievisse Adriaanse wrote: > Hi, > > Currently we appear to silently drop randomness if it comes from an invalid > source. Should we not panic in this case? > While here, update the comment for enqueue_randomness() after -r1.188.

httpd: allow for longer "tls ciphers"

2018-08-17 Thread Jasper Lievisse Adriaanse
Hi, The current limit on 'tls ciphers' is 255 characters which prevents using the cipher list as recommended by https://mozilla.github.io/server-side-tls/ssl-config-generator/ for example (clocks in just shy of 300 characters). tls ciphers

vmd: tweak mc146818 periodic interrupt updating

2019-05-26 Thread Jasper Lievisse Adriaanse
Hi, Whilst looking at the mc146818 code in vmd I noticed something that initially struck me as a pasto as the same code is present in the rtc_update_regb() function. However it led me to look at how other emulators handle the updating of registers. Based on that here's a diff to only

Re: vmd(8) i8042 device implementation questions

2019-06-02 Thread Jasper Lievisse Adriaanse
On Sat, Jun 01, 2019 at 06:12:16PM -0500, Katherine Rohl wrote: > Couple questions: > > > This means no interrupt will be injected. I'm not sure if that's what you > > want. > > See vm.c: vcpu_exit_inout(..). It looks like you may have manually asserted > > the > > IRQ in this file, which is a

rtcdrain: fix function reference in comment

2019-05-21 Thread Jasper Lievisse Adriaanse
Hi, This appears to have been wrong from the import, the comment explaining why rtcdrain() is needed is in rtcstart() in the same file. OK? Index: amd64/isa/clock.c === RCS file: /cvs/src/sys/arch/amd64/isa/clock.c,v retrieving

Re: sysupgrade: no new snapshot -> no error

2019-05-04 Thread Jasper Lievisse Adriaanse
> On 3 May 2019, at 17:04, Christian Weisgerber wrote: > > It is not an error condition if no new snapshot is available. > (Is it?) > > Index: sysupgrade.sh > === > RCS file: /cvs/src/usr.sbin/sysupgrade/sysupgrade.sh,v >

Re: un-boolean_t sys/ddb/

2019-11-05 Thread Jasper Lievisse Adriaanse
> On 5 Nov 2019, at 13:55, Martin Pieuchot wrote: > > Take the safe approach of converting `boolean_t' to `int', `TRUE' to `1' > and `FALSE' to `0'. > > This is to reduce the typedef mess that requires pulling MD/MI headers. > > Per-arch ddb code will follow, ok? I’m ok with the diff (yay

Re: [PATCH] add support for versions with '-' before a/b/rc

2019-12-12 Thread Jasper Lievisse Adriaanse
Hello Matija, Could you please provide a testcase for inclusion in src/regress/usr.bin/pkg-config too? Also, is there a particular port or pkg-config file in the wild that you ran into which exhibits this particular pattern? Cheers, Jasper > On 12 Dec 2019, at 18:28, Matija Skala wrote: > >

Re: vmctl: print root user in status owner field

2019-12-14 Thread Jasper Lievisse Adriaanse
> On 14 Dec 2019, at 02:16, Klemens Nanni wrote: > > With "owner root:wheel" (any group) the `vmctl status' output > will omit the "root" part in the OWNER column: > > vm "generic" { > owner "root:vms" > ... > } > > $ vmctl status > ID

Re: piixpm(4) add support for newer AMD chipsets

2019-12-16 Thread Jasper Lievisse Adriaanse
On Mon, Dec 16, 2019 at 12:37:51PM +0100, Claudio Jeker wrote: > This diff should add support for newer smbus controllers used on newer AMD > chipsets. Especially Hudson-2 and Kerncz based chipsets. On my Ryzen 5 the > iic(4) busses attach but there is nothing detected on them (well possible >

tcpdump: decode some Setup fields for Control transfers

2020-02-23 Thread Jasper Lievisse Adriaanse
Hi, Currently the usbpcap is fairly terse in that it prints the endpoints, type of transfer and data size. Below is a diff to start decoding more data from the packets, currently focusing on control transfers: - print the control stage - for the Setup control transfer stage, print additional

tcpdump: fix crash with invalid usb packets

2020-02-21 Thread Jasper Lievisse Adriaanse
Hi, When using -X/-x/-A with tcpdump on a packet where sizeof(*uph) is larger than 'length' we end up passing a negative value as the length to default_print(), which wraps around and suddenly we're attempting to print 4294976281 elements. Found with AFL. OK? Index: print-usbpcap.c

usbdevs: small addition

2020-02-22 Thread Jasper Lievisse Adriaanse
Hi, - add an AMD product found on the APU2 - add vendor id for "Synaptics, Inc. - add synaptics fingerprint reader found on recent thinkpads; I couldn't find a proper name for this device in the Linux usb.ids repository so I went with the generic 'Fingerprint Reader" that's also used

Re: usbdevs: small addition

2020-02-23 Thread Jasper Lievisse Adriaanse
On Sun, Feb 23, 2020 at 11:52:10AM +1100, Jonathan Gray wrote: > On Sat, Feb 22, 2020 at 04:22:25PM +0100, Jasper Lievisse Adriaanse wrote: > > Hi, > > > > - add an AMD product found on the APU2 > > I would not consider 0x7900 a root hub as it attaches to another hub

check disk_lookup return value

2020-03-11 Thread Jasper Lievisse Adriaanse
Hi, Check return value of disk_lookup before dereference as it can return NULL. These are Coverity CID 1452925, 1452951, 1452967. Other functions using disk_lookup (or the re-defined versions like wdlookup) check the return value already, including in the context of hibernation

uplcom: remove dead code

2020-03-11 Thread Jasper Lievisse Adriaanse
Hi, Remove dead code which is actually duplicated a few lines above right after err is set. Coverity ID 975917 OK? Index: dev/usb/uplcom.c === RCS file: /cvs/src/sys/dev/usb/uplcom.c,v retrieving revision 1.73 diff -u -p -U11

Re: acpitoshiba: remove dead code

2020-03-16 Thread Jasper Lievisse Adriaanse
> On 16 Mar 2020, at 09:49, Stefan Sperling wrote: > > On Mon, Mar 16, 2020 at 09:29:43AM +0100, Jasper Lievisse Adriaanse wrote: >> Hi, >> >> The type of brightness and video_output is uint32_t; therefore it >> can never be less than 0 (whi

acpitoshiba: remove dead code

2020-03-16 Thread Jasper Lievisse Adriaanse
Hi, The type of brightness and video_output is uint32_t; therefore it can never be less than 0 (which is what HCI_LCD_BRIGHTNESS_MIN and HCI_VIDEO_OUTPUT_CYCLE_MIN are defined to). So trim the checks by removig the impossible cases. Coverity CID 1453109, 1453169 OK? Index: acpi/acpitoshiba.c

dt: add static vfs probes

2020-09-14 Thread Jasper Lievisse Adriaanse
Hi, Whilst analyzing the cleaner I added tracepoints called 'cleaner' and 'bufcache_take' to track its behaviour. For the sake of symmetry I've added one in bufcache_release() too and moved the assignment of 'pages' until after the KASSERT(), following the flow of bufcache_take(). Sample

btrace: add boolean AND and OR operators

2020-09-14 Thread Jasper Lievisse Adriaanse
Hi, This diff adds support for the '&' and '|' operators, along with a new testcase. OK? Index: usr.sbin/btrace/bt_parse.y === RCS file: /cvs/src/usr.sbin/btrace/bt_parse.y,v retrieving revision 1.16 diff -u -p -r1.16 bt_parse.y

Re: btrace: add boolean AND and OR operators

2020-09-14 Thread Jasper Lievisse Adriaanse
On Mon, Sep 14, 2020 at 03:39:04PM +0200, Otto Moerbeek wrote: > On Mon, Sep 14, 2020 at 03:28:17PM +0200, Jasper Lievisse Adriaanse wrote: > > > Hi, > > > > This diff adds support for the '&' and '|' operators, along with > > a new testcase. > > > >

Re: [PATCH] wireguard: release correct lock on exceptional case

2020-10-31 Thread Jasper Lievisse Adriaanse
8000) at wg_handshake_worker+0x48 > taskq_thread(80049200) at taskq_thread+0x81 > end trace frame: 0x0, count: -8 > ddb{0}> machine ddbcpu 1 > > Reported-by: Jasper Lievisse Adriaanse > --- > sys/net/wg_noise.c | 2 +- > 1 file changed, 1 insertion(+),

ddb trace: fix output for too many arguments

2021-07-11 Thread Jasper Lievisse Adriaanse
Hi, When printing a trace from ddb, some architectures are limited by the number of registers which are used to pass arguments. If the number of arguments to a function exceeded this number, the code in db_stack_trace_print() would print that many arguments without any indication that one or

Re: ddb trace: fix output for too many arguments

2021-07-12 Thread Jasper Lievisse Adriaanse
On Sun, Jul 11, 2021 at 03:58:05PM +0200, Jasper Lievisse Adriaanse wrote: > Hi, > > When printing a trace from ddb, some architectures are limited by the number > of > registers which are used to pass arguments. If the number of arguments to a > function > exceeded

regress: don't needlessly fiddle with MALLOC_OPTIONS

2021-09-01 Thread Jasper Lievisse Adriaanse
Hi, As discussed earlier with bluhm, regress tests shouldn't set or modify MALLOC_OPTIONS (except under very specific situations, like malloc tests). It would be better to set the options globally through sysctl when running the suite, as bluhm does. So remove most cases of MALLOC_OPTIONS from

Re: ddb trace: fix output for too many arguments

2021-08-30 Thread Jasper Lievisse Adriaanse
On Mon, Jul 12, 2021 at 08:18:20PM +0200, Mark Kettenis wrote: > > Date: Mon, 12 Jul 2021 20:11:30 +0200 > > From: Jasper Lievisse Adriaanse > > > > On Sun, Jul 11, 2021 at 03:58:05PM +0200, Jasper Lievisse Adriaanse wrote: > > > Hi, > > >