From owner-freebsd-net Fri Nov 22 3:44:34 2002 Delivered-To: freebsd-net@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id A853237B404 for ; Fri, 22 Nov 2002 03:44:32 -0800 (PST) Received: from hanoi.cronyx.ru (hanoi.cronyx.ru [144.206.181.53]) by mx1.FreeBSD.org (Postfix) with ESMTP id 0B55143EA3 for ; Fri, 22 Nov 2002 03:44:31 -0800 (PST) (envelope-from rik@cronyx.ru) Received: by hanoi.cronyx.ru id OAA33267 for freebsd-net@FreeBSD.org.checked; (8.9.3/vak/2.1) Fri, 22 Nov 2002 14:41:04 +0300 (MSK) (envelope-from rik@cronyx.ru) Received: from cronyx.ru by hanoi.cronyx.ru with ESMTP id OAA33114; (8.9.3/vak/2.1) Fri, 22 Nov 2002 14:38:24 +0300 (MSK) (envelope-from rik@cronyx.ru) Message-ID: <3DDE183D.6040206@cronyx.ru> Date: Fri, 22 Nov 2002 14:42:53 +0300 From: Roman Kurakin User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.0.1) Gecko/20020826 X-Accept-Language: en-us, en MIME-Version: 1.0 To: Joerg Wunsch Cc: freebsd-net@FreeBSD.org Subject: Re: sppp patch's References: <000901c1134b$827a69a0$48b5ce90@crox> <3BDABF7B.4060808@cronyx.ru> <3BE24EE4.2020506@cronyx.ru> <20011102192916.A43204@uriah.heep.sax.de> <3BE3ED17.3060603@cronyx.ru> <20011231165245.B73897@uriah.heep.sax.de> <3DDD1D67.4050905@cronyx.ru> <20021121221833.A70987@uriah.heep.sax.de> Content-Type: text/plain; charset=us-ascii; format=flowed Content-Transfer-Encoding: 7bit Sender: owner-freebsd-net@FreeBSD.ORG Precedence: bulk List-ID: List-Archive: (Web Archive) List-Help: (List Instructions) List-Subscribe: List-Unsubscribe: X-Loop: FreeBSD.org Hi, Joerg Wunsch wrote: >As Roman Kurakin wrote: > > > >> Sppp still have a quantity of bugs. Here is one of them: >> >>--- if_spppsubr.c.orig Wed Oct 16 18:41:16 2002 >>+++ if_spppsubr.c Thu Nov 21 20:13:16 2002 >>@@ -1672,12 +1672,12 @@ >> case STATE_ACK_SENT: >> break; >> case STATE_CLOSING: >>- sppp_cp_change_state(cp, sp, STATE_CLOSED); >> (cp->tlf)(sp); >>+ sppp_cp_change_state(cp, sp, STATE_CLOSED); >> break >> > >[...] > > >>In all cases we have the same problem: at first we should call tlf >>that will changes state and then we should set proper state. If we >>set some state and then call tlf we will get wrong final state. >> >> > >Can you please explain more, e. g. with a ifconfig debug trace? The > I couldn't make debug trace right now, because this fix was made long time ago and I don't remember how to reproduce all those problems. >RFC doesn't mandate a particular order between the actions and the >actual state change, and IIRC (without digging down into the code >right now), reverting the order has other unwanted side effects. > But RFC imply which final state should be at the and of all operations in particular case. If we call tlf and then change state we will get proper final state. If we choose the other way we will get wrong final state. Current sppp has some fixes that cure a problem in place it become apparent, but don't fix it in place where it occurs. So, probably many of the problems are solved, but they are solved incorrectly, and maybe this one is solved too. Best regards, Roman Kurakin > > > To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-net" in the body of the message