From owner-freebsd-bugs@FreeBSD.ORG Thu Dec 18 02:10:20 2003 Return-Path: Delivered-To: freebsd-bugs@hub.freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 5856D16A4CE for ; Thu, 18 Dec 2003 02:10:20 -0800 (PST) Received: from freefall.freebsd.org (freefall.freebsd.org [216.136.204.21]) by mx1.FreeBSD.org (Postfix) with ESMTP id 8B13543D1F for ; Thu, 18 Dec 2003 02:10:19 -0800 (PST) (envelope-from gnats@FreeBSD.org) Received: from freefall.freebsd.org (gnats@localhost [127.0.0.1]) hBIAAJFR044677 for ; Thu, 18 Dec 2003 02:10:19 -0800 (PST) (envelope-from gnats@freefall.freebsd.org) Received: (from gnats@localhost) by freefall.freebsd.org (8.12.10/8.12.10/Submit) id hBIAAJ6m044676; Thu, 18 Dec 2003 02:10:19 -0800 (PST) (envelope-from gnats) Date: Thu, 18 Dec 2003 02:10:19 -0800 (PST) Message-Id: <200312181010.hBIAAJ6m044676@freefall.freebsd.org> To: freebsd-bugs@FreeBSD.org From: Gleb Smirnoff Subject: Re: kern/47920: if ng_pppoe switches to nonstandard mode it stays in it forever X-BeenThere: freebsd-bugs@freebsd.org X-Mailman-Version: 2.1.1 Precedence: list Reply-To: Gleb Smirnoff List-Id: Bug reports List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 18 Dec 2003 10:10:20 -0000 The following reply was made to PR kern/47920; it has been noted by GNATS. From: Gleb Smirnoff To: Yar Tikhiy Cc: FreeBSD-gnats-submit@FreeBSD.ORG Subject: Re: kern/47920: if ng_pppoe switches to nonstandard mode it stays in it forever Date: Thu, 18 Dec 2003 13:08:48 +0300 On Wed, Dec 17, 2003 at 08:42:34PM +0300, Yar Tikhiy wrote: Y> Would you mind testing the below version of your patch, revised. Y> Please pay attention to its points: Y> a) sanity check values passed through sysctl; Y> b) avoid double setting nonstandard mode; Y> c) format source according to style(9); Y> d) initialize pppoe_mode for clarity since it's no longer Y> just a boolean trigger. I have tested it on STABLE. It works OK. I have also asked PR's originator to test it. -- Totus tuus, Glebius. GLEBIUS-RIPN GLEB-RIPE