Re: [PATCH 7] npppd: malloc/memset = calloc

Thanks, commited. On Mon, 26 May 2014 22:24:23 +0200 Benjamin Baier program...@netzbasis.de wrote: and a big one. Index: common/bytebuf.c === RCS file: /cvs/src/usr.sbin/npppd/common/bytebuf.c,v retrieving revision 1.5 diff

diff: add tuning variables to config(8)

Followings are our kernel variables' default: - sb_max: 256K - tcbhash_size: 128 - udbhash_size: 128 These variables are sometime too small for busy server or gateway. I'd like to modify config(8) to customize these variables without recompiling the kernel. Comment or ok? Index:

Re: diff: add tuning variables to config(8)

On Mon, 16 Jun 2014 21:28:55 +0900 (JST) YASUOKA Masahiko yasu...@yasuoka.net wrote: Followings are our kernel variables' default: - sb_max: 256K - tcbhash_size: 128 - udbhash_size: 128 These variables are sometime too small for busy server or gateway. I'd like to modify config(8

Re: diff: add tuning variables to config(8)

On Mon, 16 Jun 2014 11:35:07 -0600 Theo de Raadt dera...@cvs.openbsd.org wrote: Followings are our kernel variables' default: - sb_max: 256K - tcbhash_size: 128 - udbhash_size: 128 These variables are sometime too small for busy server or gateway. I'd like to modify

Re: diff: add tuning variables to config(8)

Hi, On Tue, 17 Jun 2014 09:43:07 +0200 Claudio Jeker cje...@diehard.n-r-g.com wrote: On Mon, Jun 16, 2014 at 11:35:07AM -0600, Theo de Raadt wrote: Followings are our kernel variables' default: - sb_max: 256K - tcbhash_size: 128 - udbhash_size: 128 These variables

diff: Fix udp_usrreq.c to compile without IPSEC.

ok? Fix compile without IPSEC. Pointed out by Ivan Solonin. Index: sys/netinet/udp_usrreq.c === RCS file: /disk/cvs/openbsd/src/sys/netinet/udp_usrreq.c,v retrieving revision 1.184 diff -u -p -r1.184 udp_usrreq.c ---

Re: diff: Fix udp_usrreq.c to compile without IPSEC.

On Mon, 23 Jun 2014 16:33:46 -0700 Philip Guenther guent...@gmail.com wrote: On Sun, Jun 22, 2014 at 7:28 PM, YASUOKA Masahiko yasu...@yasuoka.net wrote: ok? Fix compile without IPSEC. Pointed out by Ivan Solonin. ...and Theo just reverted this because it broke RAMDISK compiles

pcb table auto resize (Re: diff: add tuning variables to config(8))

Hi, On Tue, 17 Jun 2014 12:48:24 +0900 (JST) YASUOKA Masahiko yasu...@yasuoka.net wrote: On Mon, 16 Jun 2014 11:35:07 -0600 Theo de Raadt dera...@cvs.openbsd.org wrote: Followings are our kernel variables' default: - sb_max: 256K - tcbhash_size: 128 - udbhash_size: 128

diff: class-static-route of dhcp-option

ok? Fix classless-{ms-,}static-routes to comply RFC 3442. Number of octets should be changed by corresponding to the prefix length. And 0 should be allowed for the prefix length. Also fix white spaces. Based on diff from Yuuichi Someya. Index: usr.sbin/dhcpd/confpars.c

Re: diff: class-static-route of dhcp-option

On Fri, 4 Jul 2014 09:26:04 +0200 Kenneth Westerback kwesterb...@gmail.com wrote: @@ -847,7 +847,7 @@ parse_cidr(FILE *cfile, unsigned char *a if (token == TOK_NUMBER) convert_num(prefix, val, 10, 8); - if (token != TOK_NUMBER || *prefix 1 || *prefix 32) { +

Re: diff: class-static-route of dhcp-option

On Fri, 04 Jul 2014 22:25:32 +0900 (JST) YASUOKA Masahiko yasu...@yasuoka.net wrote: On Fri, 4 Jul 2014 09:26:04 +0200 Kenneth Westerback kwesterb...@gmail.com wrote: This check may be present to ensure our dhclient does not get confused, which I will check. As my reading the code, it seems

diff: dhcpd on DLT_LOOP interfaces

Hi, Some users of npppd(8) want to use dhcpd(8) on tunneling interface to provide additional routing entries and so on to VPN clients. So I'd like to make the dhcpd work on the DLT_LOOP interfaces. comment or ok? Make dhcpd(8) work on the DLT_LOOP interfaces. worked with Yuuichi Someya

diff: fix dhcpinform to work without lease

This diff fixes dhcpinform to work without lease. ok? Fix dhcpinform to work without lease. Diff from Yuuichi Someya. Index: usr.sbin/dhcpd/dhcp.c === RCS file: /disk/cvs/openbsd/src/usr.sbin/dhcpd/dhcp.c,v retrieving revision

Re: diff: fix dhcpinform to work without lease

On Wed, 9 Jul 2014 19:08:09 +0200 Kenneth Westerback kwesterb...@gmail.com wrote: On 9 July 2014 16:26, YASUOKA Masahiko yasu...@yasuoka.net wrote: This diff fixes dhcpinform to work without lease. ok? Fix dhcpinform to work without lease. Diff from Yuuichi Someya. This seems

dhcpd udp socket mode (was Re: diff: dhcpd on DLT_LOOP interfaces)

On Wed, 9 Jul 2014 16:57:44 +0200 Kenneth Westerback kwesterb...@gmail.com wrote: On 9 July 2014 16:23, YASUOKA Masahiko yasu...@yasuoka.net wrote: Some users of npppd(8) want to use dhcpd(8) on tunneling interface to provide additional routing entries and so on to VPN clients. So I'd like

Re: dhcpd udp socket mode

On Thu, 10 Jul 2014 10:17:29 +0200 (CEST) YASUOKA Masahiko yasu...@yasuoka.net wrote: On Wed, 9 Jul 2014 16:57:44 +0200 Kenneth Westerback kwesterb...@gmail.com wrote: On 9 July 2014 16:23, YASUOKA Masahiko yasu...@yasuoka.net wrote: Some users of npppd(8) want to use dhcpd(8) on tunneling

Re: diff: fix dhcpinform to work without lease

On Wed, 09 Jul 2014 20:40:36 +0200 (CEST) YASUOKA Masahiko yasu...@yasuoka.net wrote: On Wed, 9 Jul 2014 19:08:09 +0200 Kenneth Westerback kwesterb...@gmail.com wrote: On 9 July 2014 16:26, YASUOKA Masahiko yasu...@yasuoka.net wrote: This diff fixes dhcpinform to work without lease. ok

Re: use siphash for in_pcb hash key lookups

On Tue, 4 Nov 2014 14:50:49 +1000 David Gwynne da...@gwynne.id.au wrote: this uses siphash to protect the in_pcb hashes. this is pretty much a textbook example of what siphash should be used for. tests? ok? Giving inpcb_hash() to LIST_* macros via INPCB*HASH() caused calling inpcb_hash()

Re: use siphash for in_pcb hash key lookups

On Fri, 07 Nov 2014 01:19:45 +0900 (JST) YASUOKA Masahiko yasu...@openbsd.org wrote: On Tue, 4 Nov 2014 14:50:49 +1000 David Gwynne da...@gwynne.id.au wrote: this uses siphash to protect the in_pcb hashes. this is pretty much a textbook example of what siphash should be used for. tests? ok

Re: LAC LNS server with OpenBSD

Hello, On Thu, 18 Aug 2011 00:32:22 +0200 (CEST) Gruel Bruno b.gr...@woody.hopto.org wrote: First thank's for your help et very good jobs for npppd, it's realy a good tool. But it seem not to do what i want. (http://fai.woody.hopto.org/Docs/bsdrp-example-pppoe-l2tp.png). I will try rp-l2tp

Re: LAC LNS server with OpenBSD

Original Message From: YASUOKA Masahiko yasu...@yasuoka.net To: b.gr...@woody.hopto.org Cc: ktulu+o...@wxcvbn.org, tech@openbsd.org Sent: Jeu, Aou 18, 2011, 8:04 AM Subject: Re: LAC LNS server with OpenBSD Hello, On Thu, 18 Aug 2011 00:32:22 +0200 (CEST) Gruel Bruno b.gr

Re: LAC LNS server with OpenBSD

. Unable to authenticate via the L2TP no ?? Thank's Bruno. Original Message From: YASUOKA Masahiko yasu...@yasuoka.net To: b.gr...@woody.hopto.org Cc: tech@openbsd.org Sent: Ven, Aou 19, 2011, 16:37 PM Subject: Re: LAC LNS server with OpenBSD Hi, On Fri, 19 Aug 2011 16:05:27 +0200

Re: LAC LNS server with OpenBSD

in npppd ? So thank's for your work and you availability. Bruno Gruel Original Message From: YASUOKA Masahiko yasu...@yasuoka.net To: b.gr...@woody.hopto.org Cc: tech@openbsd.org Sent: Dim, Aou 21, 2011, 17:32 PM Subject: Re: LAC LNS server with OpenBSD Hello, On Fri, 19 Aug 2011 20

tiny diff for usr.sbin/pppoe/server.c

We don't need to break after calling err(3) because this function will not return. This diff produces no binary change. First diff from Michael W. Bombardieri. ok? Index: server.c === RCS file: /cvs/src/usr.sbin/pppoe/server.c,v

small diff: unused struct member in pppoe_session

Delete unused field s_first in 'struct pppoe_session' that is assigned 1 in session_new(), but it is never used. Diff from Michael W. Bombardieri ok? Index: session.c === RCS file: /usr/src/cvs/src/usr.sbin/pppoe/session.c,v

fix compiling w/o INET6 error on pipex.c

Attached diff fixes compile errors without INET6 on pipex.c pointed out by Julien Crapovich on misc@ I've tested the diff does not create any binary change on GENERIC (w/o DIAGNOSTICS) ok? Index: net/pipex.c === RCS file:

Re: RFC 4638 support for pppoe(4)

Hello, On Wed, 11 Jan 2012 18:31:49 -0500 Matt Dainty m...@bodgit-n-scarper.com wrote: @@ -911,7 +933,7 @@ return (ENXIO); } - if (sc-sc_sppp.pp_if.if_mtu + if (sc-sc_sppp.pp_if.if_mtu !=

diff: fix compiler warnings of npppd

This fixes many compiler warnings and some styles. ok? Index: l2tp/l2tp.h === RCS file: /cvs/src/usr.sbin/npppd/l2tp/l2tp.h,v retrieving revision 1.5 diff -u -p -r1.5 l2tp.h --- l2tp/l2tp.h 15 Oct 2011 03:24:11 - 1.5 +++

diff: ingress filter of npppd/pipex.

The diff will make the ingress filter of pipex and npppd configurable and disable it by default. After this change we need to add ppp.ingress_filter: true to npppd.conf if it is needed. I promise to write about this configuration in the man page when the man page becomes available. ok?

diff: ignore mandatory rx-connect-speed avp

add handling of `rx connect speed' avp to ignore the bug of xl2tpd. reported by sebastia@ on misc@. ok? Index: usr.sbin/npppd/l2tp/l2tp_call.c === RCS file: /cvs/src/usr.sbin/npppd/l2tp/l2tp_call.c,v retrieving revision 1.9 diff -u

diff: fix LCP keepalive failures on L2TP.

pipex hook in udp_usrreq() mistakenly assumed that `inp' is connected. The hook could not use the destination address properly, so it failed to find the pipex session. This bug caused LCP keepalive failures on L2TP from client that does LCP keepalive and uses sequence number on the L2TP data

Re: diff: fix LCP keepalive failures on L2TP.

On Mon, 30 Jan 2012 22:49:22 +0900 (JST) YASUOKA Masahiko yasu...@yasuoka.net wrote: pipex hook in udp_usrreq() mistakenly assumed that `inp' is connected. The hook could not use the destination address properly, so it failed to find the pipex session. This bug caused LCP keepalive failures

Re: diff: fix LCP keepalive failures on L2TP.

Hi, On Tue, 31 Jan 2012 13:59:17 +0100 Sebastian Reitenbach sebas...@l00-bugdead-prods.de wrote: However, I noted with tcpdump, listening on tun0: # tcpdump -n -i tun0 tcpdump: listening on tun0, link-type LOOP 13:51:15.354776 tcpdump: WARNING: compensating for unaligned libpcap packets

trasport-mode ESP packet and pf `rdr-to'

Hi, In ip_input(), there is a filter to disable all packets to 127.0.0.0/27. That filter drops a packet that was a transport-mode ESP packet and that has been redirected to 127.0.0.1 with pf `rdr-to' rule. Below diff will fix the filter not to drop such packets. ok? or comment? The problem was

Re: trasport-mode ESP packet and pf `rdr-to'

On Thu, 15 Mar 2012 09:16:42 +0100 Alexander Bluhm alexander.bl...@gmx.net wrote: On Wed, Mar 14, 2012 at 03:32:08PM +0900, YASUOKA Masahiko wrote: In ip_input(), there is a filter to disable all packets to 127.0.0.0/27. That filter drops a packet that was a transport-mode ESP packet

Re: trasport-mode ESP packet and pf `rdr-to'

On Thu, 15 Mar 2012 12:22:52 +0100 Mike Belopuhov m...@crypt.org.ru wrote: On Thu, Mar 15, 2012 at 19:34 +0900, YASUOKA Masahiko wrote: On Thu, 15 Mar 2012 09:16:42 +0100 Alexander Bluhm alexander.bl...@gmx.net wrote: On Wed, Mar 14, 2012 at 03:32:08PM +0900, YASUOKA Masahiko wrote

Re: npppd_arp.h missing in the tree

Hi, On Sat, 31 Mar 2012 16:30:37 +0200 mxb m...@alumni.chalmers.se wrote: looks like npppd_arp.h is missing in the tree. Is it in purpose or just a miss? There are npppd_arp.[ch] in IIJ(upstream and company I work for)'s local tree. But they are work-in-progress. So they should be deleted

diff: fix bpf problem of pipex (was Re: diff: fix LCP keepalive failures on L2TP.)

On Tue, 31 Jan 2012 13:59:17 +0100 Sebastian Reitenbach sebas...@l00-bugdead-prods.de wrote: However, I noted with tcpdump, listening on tun0: # tcpdump -n -i tun0 tcpdump: listening on tun0, link-type LOOP 13:51:15.354776 tcpdump: WARNING: compensating for unaligned libpcap packets

Re: diff: fix bpf problem of pipex

Hi, On Wed, 4 Apr 2012 13:19:06 +0200 Claudio Jeker cje...@diehard.n-r-g.com wrote: On Wed, Apr 04, 2012 at 02:34:46PM +0900, Yasuoka Masahiko wrote: On Tue, 31 Jan 2012 13:59:17 +0100 Sebastian Reitenbach sebas...@l00-bugdead-prods.de wrote: However, I noted with tcpdump, listening on tun0

diff: small diff for pipex.c

Remove redundant returns from functions returning void. Wrap return values in () for consistency. diff from Michael W. Bombardieri. ok? Index: sys/net/pipex.c === RCS file: /cvs/src/sys/net/pipex.c,v retrieving revision 1.27 diff

diff: missing break in switch

npppd used wrong AVPs as a `calling number' because `break' in switch case was missing. ok? Index: npppd/l2tp/l2tp_call.c === RCS file: /cvs/src/usr.sbin/npppd/l2tp/l2tp_call.c,v retrieving revision 1.12 diff -u -p -r1.12

Re: [PATCH] Add IPV6_RECVDSTPORT socket option

ok to commit? comment? On Fri, 07 Sep 2012 16:13:53 +0900 UMEZAWA Takeshi umez...@iij.ad.jp wrote: Hello, I have added IPV6_RECVDSTPORT socket option, which enables us to get original (= before divert) destination port of a UDP packet. The way to use this option is same as IP_RECVDSTPORT.

Re: [PATCH] Add send(2) MSG_DONTWAIT support

I'm going to commit. ok or commet are still welcome. On Tue, 18 Sep 2012 22:27:19 +0200 Alexander Bluhm alexander.bl...@gmx.net wrote: On Fri, Sep 07, 2012 at 01:43:29PM +0900, UMEZAWA Takeshi wrote: I have added send(2) MSG_DONTWAIT support, which enables us to choose nonblocking or blocking

Re: Wake from zzz causes panic on thinkpad x60

Hi, On Fri, 1 Mar 2013 13:43:00 + Wade, Daniel dw...@meridium.com wrote: -Original Message- From: owner-t...@openbsd.org [mailto:owner-t...@openbsd.org] On Behalf Of Stefan Sperling Sent: Thursday, February 28, 2013 12:16 PM To: Edd Barrett Cc: tech@openbsd.org Subject: Re:

Re: Wake from zzz causes panic on thinkpad x60

Hi, On Fri, 17 May 2013 14:06:49 +0200 Martin Pieuchot mpieuc...@nolizard.org wrote: On 17/05/13(Fri) 19:25, YASUOKA Masahiko wrote: On Fri, 1 Mar 2013 13:43:00 + Wade, Daniel dw...@meridium.com wrote: -Original Message- From: owner-t...@openbsd.org [mailto:owner-t

Re: [patch] radiusd: rm -h flag, fail earlier (and print usage) given an unknown option

Thanks for the diffs for radius{d,ctl}. Commited both. On Mon, 24 Aug 2015 15:54:22 -0400 Michael Reed m.r...@mykolab.com wrote: This patch removes the -h flag because it seems very rare in OpenBSD programs, perhaps because options are usually already documented in manuals, as is the case for

Re: inteldrm(4) diff that needs testing

On Sat, 24 Oct 2015 23:48:01 +0200 (CEST) Mark Kettenis wrote: > The diff below makes inteldrm(4) attach directly to pci(4) instead of > vga(1). Because inteldrm(4) depends on intagp(4), this also make > intagp(4) a child of inteldrm(4). Ultimately I'd like to integrate

Re: npppd(8): sync option parsing with other userland utils

On Sun, 8 Nov 2015 22:20:33 +0100 Benjamin Baier wrote: > This patch brings npppd(8)'s option parsing in sync with all the > other userland utils that don't recognize -h for help/usage (except > for some games, still). > > Also exit(3) at the end of the usage() function

Re: npppd: simplify and lock down priv_open()

ok yasuoka, the diff is tested. On Sat, 10 Oct 2015 21:58:02 -0700 Philip Guenther wrote: > On Sat, 10 Oct 2015, Theo de Raadt wrote: >> I don't know the code either, but it is probably better if privsep's had >> more narrow task-specific operations. Like

Re: efifb(4) improvements

ok yasuoka The diff speeded up boot time 32.5 sec => 18.0 sec (with -s, until appearing prompt of single user mode). On Sun, 6 Sep 2015 00:49:09 +0200 (CEST) Mark Kettenis wrote: > On most modern hardware reading from a framebuffer is much slower than > writing to it.

Re: npppd: simplify and lock down priv_open()

Hi, On Sat, 10 Oct 2015 21:32:29 -0700 Philip Guenther wrote: > So: > * kill the 'mode' argument to PRIVSEP_OPEN and priv_open() > * fail a PRIVSEP_OPEN call if it's passed any flags other than >O_ACCMODE or O_NONBLOCK > * paranoia: mask O_CREAT when calling open()

Re: npppd: simplify and lock down priv_open()

On Fri, 4 Dec 2015 09:47:46 -0800 Philip Guenther <guent...@gmail.com> wrote: > On Fri, Dec 4, 2015 at 3:41 AM, YASUOKA Masahiko <yasu...@openbsd.org> wrote: >> On Sat, 10 Oct 2015 21:32:29 -0700 >> Philip Guenther <guent...@gmail.com> wrote: >&g

minor build fix in libssl

Hi, Compiling sha256-x86_64.S fails if the "src" is located a directory which includes "512". The diff below fixes this problem. ok? Index: lib/libssl/src/crypto/sha/asm/sha512-x86_64.pl === RCS file:

Re: npppd: simplify and lock down priv_open()

I'm sorry for delay. ok yasuoka tested. On Fri, 1 Jan 2016 10:55:14 -0800 Philip Guenther wrote: > On Fri, 4 Dec 2015, Todd C. Miller wrote: >> On Fri, 04 Dec 2015 09:47:46 -0800, Philip Guenther wrote: >> > We should fix open(2); please try the diff below. Are you sure

Re: minor build fix in libssl

On Thu, 17 Dec 2015 12:04:10 +0900 (JST) YASUOKA Masahiko <yasu...@openbsd.org> wrote: > Compiling sha256-x86_64.S fails if the "src" is located a directory > which includes "512". Actually I hit this problem like below since it's 2015-12: sha512-x86_64.so: In f

Re: Vaio Pro 11" SVP112A1CL can't suspend to ram or hibernate

On Sat, 20 Feb 2016 04:17:05 +0100 Noth wrote: > I installed -CURRENT on my laptop today, so far so good on the UEFI > boot, crypto disk and wifi with .11n. Unfortunately the suspend to ram > and hibernate options don't work... Suspend goes to suspend but wakes >

Re: npppd_local.h - Forgotten include guard

commited. Thanks, On Sat, 05 Mar 2016 09:34:07 + "Tiago Silva" wrote: > Hi to all, > > The 1.13 revision of /src/usr.sbin/npppd/npppd/npppd_local.h forgot to remove > the bottom __cplusplus include guard. > > --- /src/usr.sbin/npppd/npppd/npppd_local.h Thu Jul 23

Re: efiboot 240x56 mode

Hi, On Sat, 12 Mar 2016 10:22:13 +0100 Matthieu Herrb wrote: > a collegue of mine recently tried to get OpenBSD running on an HP E820 > laptop. Apparently this machine is UEFI only and its BIOS only works > in a 240x56 text mode. The original efi_video_init() is to switch the

Re: EFIboot and HP Stream 13

On Wed, 13 Apr 2016 09:26:23 +0200 "Vegar Linge Haaland" wrote: > probing: pc0 mem[444K 88K 511M 1374M 19M] > Disk:BlockSize=256 LastBlock=16383 > Hit any key to reboot The result above means the disk is 4MB and its bllock size 256 bytes. This seems strange for me. > Also, I

Re: EFIboot and HP Stream 13

_rtt(void) { -#ifdef EFI_DEBUG +//#ifdef EFI_DEBUG printf("Hit any key to reboot\n"); efi_cons_getc(0); -#endif +//#endif EFI_CALL(RS->ResetSystem, EfiResetCold, EFI_SUCCESS, 0, NULL); while (1) { } } On Wed, 13 Apr 2016 10:49:40 +0900 (JST) YASUOKA Masahiko &

Re: EFIboot and HP Stream 13

Hi, Thank you for your report. The diff seems to make sense for the issue. I'll fix it on the CVS soon. On Mon, 11 Apr 2016 21:26:37 +0200 "Vegar Linge Haaland" wrote: > Hi! I am hitting an issue in efiboot on my HP Stream 13. Patch below. Sorry > if this is the wrong list.

Re: EFIboot and HP Stream 13

Hello, On Wed, 13 Apr 2016 12:27:51 +0200 "Vegar Linge Haaland" <ve...@vegarlh.com> wrote: >> -Original Message- >> From: owner-t...@openbsd.org [mailto:owner-t...@openbsd.org] On >> Behalf Of YASUOKA Masahiko >> Sent: onsdag 13. april 2016 09

Re: Patches for improving npppd with rdomain support

Hi, On Tue, 1 Mar 2016 17:32:22 +0100 Claer wrote: > I asked a friend, Mathieu Papineau, to help me with an issue I have with > npppd. Thank you for the diffs. I already commited the fix of the path. > For a project here, I thought building multitenancy firewall with

Re: npppd: reload enables stripping NT domains on radius

On Tue, 21 Feb 2017 08:31:13 +0100 Patrick Wildt <patr...@blueri.se> wrote: > On Tue, Feb 21, 2017 at 02:11:05PM +0900, YASUOKA Masahiko wrote: >> On Mon, 20 Feb 2017 11:38:19 +0100 >> Patrick Wildt <patr...@blueri.se> wrote: >> > when using RADIUS, the NT

Re: npppd: reload enables stripping NT domains on radius

Hi, On Mon, 20 Feb 2017 11:38:19 +0100 Patrick Wildt wrote: > when using RADIUS, the NT domains should not be stripped from the > username. I suppose it depends on the use-case. npppd.conf(5) mentions "strip-nt-domain" is "yes" by default and adding "strip-nt-domain no" in

Re: efiboot: boot device lookup typo

Hi, On Mon, 27 Feb 2017 11:10:31 +0100 Patrick Wildt wrote: > I'm surprised this didn't come up earlier, but I think the for-loop is a > bit wrong. What the code is supposed to be doing is going over each > device path node for the loaded image, which is supposed to be the

Re: efiboot: boot device lookup typo

Let me update the diff, On Tue, 28 Feb 2017 22:41:21 +0900 (JST) YASUOKA Masahiko <yasu...@openbsd.org> wrote: > On Mon, 27 Feb 2017 11:10:31 +0100 > Patrick Wildt <patr...@blueri.se> wrote: >> I'm surprised this didn't come up earlier, but I think the for-loop is a >

Re: efiboot: boot device lookup typo

Hi, On Tue, 28 Feb 2017 15:16:58 +0100 Patrick Wildt <patr...@blueri.se> wrote: > On Tue, Feb 28, 2017 at 10:41:21PM +0900, YASUOKA Masahiko wrote: >> On Mon, 27 Feb 2017 11:10:31 +0100 >> Patrick Wildt <patr...@blueri.se> wrote: >> > I'm surprised this

Re: efiboot: boot device lookup typo

does I/O against disks (not partitions), so we need to find the device path of the disk. --yasuoka On Tue, 28 Feb 2017 16:48:19 +0100 Patrick Wildt <patr...@blueri.se> wrote: > On Tue, Feb 28, 2017 at 04:33:14PM +0100, Patrick Wildt wrote: >> On Wed, Mar 01, 2017 at 12:17:02AM +0900

Re: [patch] make npppd do as the man page says

Sorry for long delay. Using empty sting for hostname in PPTP is intentional. So I will fix the man page. Thank you for your report. On Tue, 8 Mar 2016 11:20:40 + Joe Holden wrote: > Below is a patch that corrects pptp to set the pptp hostname as the > system

Enabling rasops24

Enabling rasops24 in files.amd64 makes QEMU with UEFI start working. But.. the background color of the kernel message is sometimes red or green where it should be blue. ok for files.amd64? comments for the color problem? Index: sys/arch/amd64/conf/files.amd64

Re: etherip alignment issues

Hi, I could repeat the problem on my octeon. Tested the diff on it. ok? Use a new mbufs for a tunnel header to make sure it is aligned correctly. Index: sys/net/if_etherip.c === RCS file: /cvs/src/sys/net/if_etherip.c,v

Re: etherip alignment issues

Hi, The problem doesn't repeat on my Octeon. Can you try the diff below? - I assume the diff fixes the problem - A kernel message is added. please let me know if it appears. Thanks, diff --git a/sys/net/if_etherip.c b/sys/net/if_etherip.c index ad58c52..c0af82e 100644 ---

Re: etherip alignment issues

On Sat, 24 Sep 2016 13:08:18 -0400 (EDT) Martin Brandenburg <mar...@martinbrandenburg.com> wrote: > On Sat, 24 Sep 2016, YASUOKA Masahiko wrote: >> The problem doesn't repeat on my Octeon. >> >> Can you try the diff below? >> >> - I assume the diff fi

Re: add in6 multicast support to vxlan(4) ; question on mbufs

On Wed, 5 Oct 2016 14:30:27 +0200 Mike Belopuhov <m...@belopuhov.com> wrote: > On Wed, Oct 05, 2016 at 10:58 +0900, YASUOKA Masahiko wrote: >> On Tue, 4 Oct 2016 17:27:12 +0200 >> Mike Belopuhov <m...@belopuhov.com> wrote: >> > On Tue, Oct 04, 2016

Re: regress for etherip and vxlan

On Wed, 5 Oct 2016 15:31:39 +0200 Alexander Bluhm <alexander.bl...@gmx.net> wrote: > On Wed, Oct 05, 2016 at 04:48:55PM +0900, YASUOKA Masahiko wrote: >> The diff add regress scripts for vxlan(4) and etherip(4). >> >> This will be my first commit to regress/. ok? >

regress for etherip and vxlan

The diff add regress scripts for vxlan(4) and etherip(4). This will be my first commit to regress/. ok? diff --git a/regress/sys/net/Makefile b/regress/sys/net/Makefile index 40f49cc..f46c3dd 100644 --- a/regress/sys/net/Makefile +++ b/regress/sys/net/Makefile @@ -1,5 +1,6 @@ # $OpenBSD:

Re: add in6 multicast support to vxlan(4) ; question on mbufs

On Tue, 4 Oct 2016 17:27:12 +0200 Mike Belopuhov wrote: > On Tue, Oct 04, 2016 at 01:07 +0200, Vincent Gross wrote: >> On Sat, 24 Sep 2016 10:58:10 +0200 >> Vincent Gross wrote: >> >> > Hi, >> > >> [snip] >> > >> > Aside from the mbuf issue, is this Ok

Re: etherip alignment issues

On Sat, 24 Sep 2016 16:45:09 -0400 (EDT) Martin Brandenburg <mar...@martinbrandenburg.com> wrote: > On Sun, 25 Sep 2016, YASUOKA Masahiko wrote: > >> On Sat, 24 Sep 2016 13:08:18 -0400 (EDT) >> Martin Brandenburg <mar...@martinbrandenburg.com> wrote: >> >

Re: request for test: mfii

On Wed, 26 Oct 2016 10:26:19 +1100 Jonathan Gray <j...@jsg.id.au> wrote: > On Tue, Oct 25, 2016 at 05:29:55PM +0900, YASUOKA Masahiko wrote: >> I'm working on making mfii(4) bio(4) capable. >> >> If you have a machine which has mfii(4), I'd like you to test the di

fix carp "carppeer" + "real mac-address"

Hi, Currenlty when carppeer + "real mac-address" are used at once, changing MASTER by carpdemote causes "MASTER-MASTER" problem, BACKUP is to become MASTER but the present MASTER also will keep its state. The diff following will fix this problem. ok? Accept CARP advertisement packets whose

use x2apic if it is enabled by BIOS

Hi, I'm working on NEC Express5800/R110h-1 (dmesg is attached). On this machine, our kernel panics with following message. cpu0 at mainbus0panic: cpu at apic id 0 already attached? This seems to happen since x2APIC on the machine is enabled by BIOS and the kernel doesn't assume that. The

Re: fix carp "carppeer" + "real mac-address"

FYI. I just commited regress tests on regress/netinet/carp/ . carp_4.sh demostrates the problem which the diff fixes. On Tue, 18 Oct 2016 19:33:01 +0900 (JST) YASUOKA Masahiko <yasu...@openbsd.org> wrote: > Currenlty when carppeer + "real mac-address" are used at once

request for test: mfii

Hi, I'm working on making mfii(4) bio(4) capable. If you have a machine which has mfii(4), I'd like you to test the diff following. (It might be risky for production machines for this moment.) After the diff applied, bioctl(8) against the disk (eg. sd0) starts working and also "sysctl

Re: M_PREPEND in vxlan encap on not strict-alignment archs

ok yasuoka thanks, On Mon, 10 Oct 2016 13:48:18 +1000 David Gwynne wrote: > if the arch can cope with prepending on an unaligned address in > vxlan, then let it do it. > > this means less work if we can get away with it. > > ok? > > Index: if_vxlan.c >

Re: use x2apic if it is enabled by BIOS

Hi, Thanks your comments, On Fri, 14 Oct 2016 16:23:40 +0200 (CEST) s...@openbsd.org wrote: > On Fri, 14 Oct 2016, YASUOKA Masahiko wrote: >> I'm working on NEC Express5800/R110h-1 (dmesg is attached). On this >> machine, our kernel panics with following message. >> >&

Re: efifb(4): WSDISPLAYIO_{GET,SET}PARAM ioctl support

comitted. Thanks! On Sun, 11 Dec 2016 10:32:59 +0100 Anton Lindqvist wrote: > This allows the brightness on my Dell Latitude 3160 to be adjusted > through wsconsctl(1). > > Index: arch/amd64/amd64/efifb.c >

Re: vxlan bug wrt IN6_ANY as source Was: Re: tweak in6_selectsrc()

Hi, > I think we should not allow at all empty source addresses, as it can > make things confusing when troubleshooting. goda@ yasuoka@ reyk@ : > what is your take on this ? I have the same opinion of you. Since tunneling protocol like vxlan (or etherip) is used with statically assigned IP

Re: delete pf states by exact flow info and speed it up

Hi, On Fri, 21 Apr 2017 00:27:04 +0200 Alexandr Nedvedicky wrote: > I finally got to your patch. I'm fine with your to extend pfctl kill > operation. > > I've found just few nits in your change. > > 1) your manpage diff should include one more change below:

Re: delete pf states by exact flow info and speed it up

On Sun, 23 Apr 2017 00:18:07 +0200 Hiltjo Posthuma <hil...@codemadness.org> wrote: > On Sat, Apr 22, 2017 at 08:14:06AM +0900, YASUOKA Masahiko wrote: >> On Fri, 21 Apr 2017 13:52:51 +0900 (JST) >> YASUOKA Masahiko <yasu...@openbsd.org> wrote: >> > +int >

delete pf states by exact flow info and speed it up

Hi, I'd like to add "key" modifier to "pfctl -k" to delete pf states, like below: # pfctl -k key -k 'tcp 10.0.0.1:80 <- 10.0.0.101:32123' This deletion will be very fast if the RB tree of pf state table is used to find the state. The diff also includes the diff for that part (DIOCKILLSTATES

Re: efiboot boot path matching

Hi, On Sun, 30 Jul 2017 12:20:27 +1000 Jonathan Gray wrote: > On Sat, Jul 29, 2017 at 09:01:35PM +0200, Patrick Wildt wrote: >> On Sat, Jul 29, 2017 at 02:59:19PM +0200, Mark Kettenis wrote: >> > This is apparently very hard. Caught this on arm64 where >> >

Re: delete pf states by exact flow info and speed it up

On Fri, 21 Apr 2017 13:52:51 +0900 (JST) YASUOKA Masahiko <yasu...@openbsd.org> wrote: > +int > +pfctl_parse_host(char *str, struct pf_rule_addr *addr) > +{ (snip) > + if ((sbs = strchr(s, '[')) != NULL || (sbe = strrchr(s, ']')) != NULL) { > + hints

Re: tun/tap vs splnet()

On Fri, 26 May 2017 18:05:30 +0200 Martin Pieuchot wrote: > On 26/05/17(Fri) 17:43, Alexander Bluhm wrote: >> > @@ -569,8 +554,6 @@ tun_output(struct ifnet *ifp, struct mbu >> >af = mtod(m0, u_int32_t *); >> >*af = htonl(dst->sa_family); >> > >> > - s = splnet(); >> >

Re: efiboot serial console support

Thanks, Let me update the diff. On Tue, 30 May 2017 13:58:29 +0200 Patrick Wildt <patr...@blueri.se> wrote: > On Tue, May 30, 2017 at 12:58:48PM +0200, YASUOKA Masahiko wrote: >> The following diff is to support serial console on efiboot. > > Nice. >> >> It u

efiboot serial console support

Hi, The following diff is to support serial console on efiboot. It uses ACPI UID to identify the port number (com0, com1 and so on) of probed serial interface. But I'm not sure wether com0-com3 are always mapped UID 0-3 as expected. Though I think this is good enough. Comment? diff --git

Re: efiboot serial console support

Yes, I think this is the proper fix. Please commit. ok yasuoka On Thu, 1 Jun 2017 10:41:57 +0200 Patrick Wildt <patr...@blueri.se> wrote: > On Thu, Jun 01, 2017 at 10:27:28AM +0200, Stefan Sperling wrote: >> On Tue, May 30, 2017 at 02:31:48PM +0200, YASUOKA Masahiko wrote:

Re: amd64: EFI boot over network try to load kernel from hd0

Hi, Tested the diff. It works fine. On Wed, 7 Jun 2017 20:23:43 +0200 Patrick Wildt wrote: > Do you want it to run diskless with root on NFS? Well, I haven't > implemented that, but here's a diff that should allow you to load > the kernel from a TFTP server. > > I have a

Re: efiboot: Restore GOP mode on SetMode() failure

Hi, On Tue, 3 Oct 2017 18:15:33 -0500 Andrew Daugherity wrote: > From: Klemens Nanni > Date: 2017-09-16 13:58:21 > Message-ID: <20170916135821.4245umgvha23b...@x230.example.com> >>Updated diff introducing efi_gop_setmode() as helper to reduce duplicate

Re: efiboot: Restore GOP mode on SetMode() failure

On Sat, 7 Oct 2017 09:24:20 +0200 Klemens Nanni <k...@posteo.org> wrote: > On Sat, Oct 07, 2017 at 01:15:40AM +, YASUOKA Masahiko wrote: >> > See my updated diff for reusing the gopi struct, please. >> >> ok, but the diff seems to be against wrong revision. Yo

  1   2   3   4   >