From owner-freebsd-net@freebsd.org Thu Apr 20 13:38:35 2017 Return-Path: Delivered-To: freebsd-net@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id 19AE8D48401 for ; Thu, 20 Apr 2017 13:38:35 +0000 (UTC) (envelope-from peter.blok@bsd4all.org) Received: from smtpq2.tb.mail.iss.as9143.net (smtpq2.tb.mail.iss.as9143.net [212.54.42.165]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id CC799142 for ; Thu, 20 Apr 2017 13:38:34 +0000 (UTC) (envelope-from peter.blok@bsd4all.org) Received: from [212.54.42.136] (helo=smtp12.tb.mail.iss.as9143.net) by smtpq2.tb.mail.iss.as9143.net with esmtp (Exim 4.86_2) (envelope-from ) id 1d1CI3-0002jg-LC; Thu, 20 Apr 2017 15:38:31 +0200 Received: from 5ed15678.cm-7-2b.dynamic.ziggo.nl ([94.209.86.120] helo=wan0.bsd4all.org) by smtp12.tb.mail.iss.as9143.net with esmtp (Exim 4.86_2) (envelope-from ) id 1d1CI3-0003vk-FC; Thu, 20 Apr 2017 15:38:31 +0200 Received: from newnas (localhost [127.0.0.1]) by wan0.bsd4all.org (Postfix) with ESMTP id 20EA77616; Thu, 20 Apr 2017 15:38:31 +0200 (CEST) X-Virus-Scanned: amavisd-new at bsd4all.org Received: from wan0.bsd4all.org ([127.0.0.1]) by newnas (newnas.bsd4all.org [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id LP0sBVVi1b6P; Thu, 20 Apr 2017 15:38:30 +0200 (CEST) Received: from [192.168.1.64] (mm [192.168.1.64]) by wan0.bsd4all.org (Postfix) with ESMTPSA id 0D2617611; Thu, 20 Apr 2017 15:38:30 +0200 (CEST) Content-Type: text/plain; charset=utf-8 Mime-Version: 1.0 (Mac OS X Mail 10.3 \(3273\)) Subject: Re: MFC VIMAGE fixes to 11-stable From: peter.blok@bsd4all.org In-Reply-To: Date: Thu, 20 Apr 2017 15:38:29 +0200 Cc: Marko Zec , freebsd-net@freebsd.org Content-Transfer-Encoding: quoted-printable Message-Id: References: <8E6FC1CD-24D5-46D5-A6A1-760DD612F92D@bsd4all.org> <20170420124256.1190665d@x23> <60C3FBF7-7CF3-49AF-9DDF-0589AE9D9146@sigsegv.be> <20170420152853.019e5480@x23> To: Kristof Provost X-Mailer: Apple Mail (2.3273) X-SourceIP: 94.209.86.120 X-Ziggo-spambar: / X-Ziggo-spamscore: 0.0 X-Ziggo-spamreport: CMAE Analysis: v=2.2 cv=V8IN6avi c=1 sm=1 tr=0 a=IkzOOneQUJP1+bAPekPvBg==:17 a=IkcTkHD0fZMA:10 a=AzvcPWV-tVgA:10 a=6I5d2MoRAAAA:8 a=8wqUZSj_QJ7EFpuCjcoA:9 a=QEXdDO2ut3YA:10 a=IjZwj45LgO3ly-622nXo:22 none X-Ziggo-Spam-Status: No X-Spam-Status: No X-Spam-Flag: No X-BeenThere: freebsd-net@freebsd.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: Networking and TCP/IP with FreeBSD List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 20 Apr 2017 13:38:35 -0000 I=E2=80=99ll test this today. > On 20 Apr 2017, at 15:32, Kristof Provost wrote: >=20 > On 20 Apr 2017, at 15:28, Marko Zec wrote: >> Right. But pfi_attach_group_event() and the other handlers cited = above >> _do_ in fact invoke CURVNET_SET(vnet0) on entry, overriding the = proper >> vnet choice from the caller. >>=20 > Yes, that does look wrong. > I should have looked a bit further. >=20 >> Therefore the proper fix should be as simple as removing = CURVNET_SET() / >> CURVNET_RESTORE() macro pairs from the cited handlers. >>=20 > Hopefully, yes. I=E2=80=99ve still got some other pf/vnet issues on my = todo list, but > I=E2=80=99ve now added this too. It might actually explain some other = bug report I=E2=80=99ve > seen (but not looked at in any depth). >=20 > Regards, > Kristof > _______________________________________________ > freebsd-net@freebsd.org mailing list > https://lists.freebsd.org/mailman/listinfo/freebsd-net > To unsubscribe, send any mail to "freebsd-net-unsubscribe@freebsd.org"