Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 02 Nov 2001 10:44:36 +0300
From:      Roman Kurakin <rik@cronyx.ru>
To:        Roman Kurakin <rik@cronyx.ru>
Cc:        freebsd-net@FreeBSD.org, Serge Vakulenko <vak@cronyx.ru>, mike@FreeBSD.org, freebsd-bugs@FreeBSD.org, julian@FreeBSD.org, archie@FreeBSD.org, joerg@FreeBSD.org
Subject:   Re: kern/11238, kern/14848, kern/21771, sppp patch's patch_id #1
Message-ID:  <3BE24EE4.2020506@cronyx.ru>
References:  <000901c1134b$827a69a0$48b5ce90@crox> <3BDABF7B.4060808@cronyx.ru>

next in thread | previous in thread | raw e-mail | index | archive | help
  Hi,

    This letter was sent last Saturday. When should I expect any reaction?

Best regards,
                        Kurakin Roman

Roman Kurakin wrote:

> Hi,
>      This is the first patch of set of patches that I plan to make. 
> These patches ware send several
> times as a big patch and last one wasn't even discussed. So I will try 
> to send them by small
> pieces and will try to comment them.
>
>    Last one big patch was kern/21771. Last our version of sppp and 
> adapter drivers could be found
> at http://www.cronyx.ru/software/
>
>    First portion contains following changes:
> 1) Just a header changes.
> 2) Changes like that:
>                case STATE_CLOSING:
> -                       sppp_cp_change_state(cp, sp, STATE_CLOSED);
>                        (cp->tlf)(sp);
> +                       sppp_cp_change_state(cp, sp, STATE_CLOSED);
>                        break;
>
> Comment:
>
>    If you change state at first and then call tlf you will get wrong 
> final state cause tlf will
> lead to "Close" event and you will get (for this example) final state 
> "Initial".
>
>    In some cases this isn't so fatal but in other this will put out of 
> action a link.
>
>    All this changes was made according to RFC1661.
>
> Best regards,
>
> Roman Kurakin, Software Engineer,
> Cronyx Engineering
>
>>> Synopsis: Frame Relay support, corrected
>>>
>>> State-Changed-From-To: open->suspended
>>> State-Changed-By: mike
>>> State-Changed-When: Fri Jul 20 19:54:47 PDT 2001
>>> State-Changed-Why:
>>> With a little bit of work, this could probably be committed.
>>> Awaiting committer.
>>>
>>> http://www.FreeBSD.org/cgi/query-pr.cgi?pr=14848
>>>
>>
>
>
>------------------------------------------------------------------------
>
>--- if_spppsubr.c.orig	Sat Oct 27 16:37:21 2001
>+++ if_spppsubr.c	Sat Oct 27 16:56:32 2001
>@@ -1,13 +1,21 @@
> /*
>- * Synchronous PPP/Cisco link level subroutines.
>+ * Synchronous PPP/Cisco/Frame Relay link level subroutines.
>  * Keepalive protocol implemented in both Cisco and PPP modes.
>+ * ANSI T1.617-compaible link management signaling
>+ * implemented for Frame Relay mode.
>+ * Cisco-type Frame Relay framing added, thanks Alex Tutubalin.
>+ * Only one DLCI per channel for now.
>  *
>- * Copyright (C) 1994-1996 Cronyx Engineering Ltd.
>+ * Copyright (C) 1994-2001 Cronyx Engineering Ltd.
>  * Author: Serge Vakulenko, <vak@cronyx.ru> <mailto:vak@cronyx.ru>
>  *
>  * Heavily revamped to conform to RFC 1661.
>  * Copyright (C) 1997, Joerg Wunsch.
>  *
>+ * Slightly revamped to conform to real life.
>+ * Copyright (C) 1999-2001 Cron
>yx Engineering Ltd.
>+ * Author: Kurakin Roman, <rik@cronyx.ru> <mailto:rik@cronyx.ru>
>+ *
>  * This software is distributed with NO WARRANTIES, not even the implied
>  * warranties for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
>  *
>@@ -222,7 +230,7 @@
> 	u_short time0;
> 	u_short time1;
> };
>-#define CISCO_PACKET_LEN 18
>+#define CISCO_PACKET_LEN 14
> 
> /*
>  * We follow the spelling and capitalization of RFC 1661 here, to make
>@@ -1532,12 +1540,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;
> 		case STATE_STOPPING:
>-			sppp_cp_change_state(cp, sp, STATE_STOPPED);
> 			(cp->tlf)(sp);
>+			sppp_cp_change_state(cp, sp, STATE_STOPPED);
> 			break;
> 		case STATE_ACK_RCVD:
> 			sppp_cp_change_state(cp, sp
>, STATE_REQ_SENT);
>@@ -1850,8 +1858,8 @@
> 	case STATE_CLOSING:
> 		break;
> 	case STATE_STARTING:
>-		sppp_cp_change_state(cp, sp, STATE_INITIAL);
> 		(cp->tlf)(sp);
>+		sppp_cp_change_state(cp, sp, STATE_INITIAL);
> 		break;
> 	case STATE_STOPPED:
> 		sppp_cp_change_state(cp, sp, STATE_CLOSED);
>@@ -1890,18 +1898,18 @@
> 		/* TO- event */
> 		switch (sp->state[cp->protoidx]) {
> 		case STATE_CLOSING:
>-			sppp_cp_change_state(cp, sp, STATE_CLOSED);
> 			(cp->tlf)(sp);
>+			sppp_cp_change_state(cp, sp, STATE_CLOSED);
> 			break;
> 		case STATE_STOPPING:
>-			sppp_cp_change_state(cp, sp, STATE_STOPPED);
> 			(cp->tlf)(sp);
>+			sppp_cp_change_state(cp, sp, STATE_STOPPED);
> 			break;
> 		case STATE_REQ_SENT:
> 		case STATE_ACK_RCVD:
> 		case STATE_ACK_SENT:
>-			sppp_cp_change_state(cp, sp, STATE_STOPPED);
> 			(cp->tlf)(sp);
>+			sppp_cp_change_state(cp, sp, STATE_STOPPED);
> 			break;
> 		}
>
> 	else
>
> sppp1.pch
>
> Content-Type:
>
> text/plain
> Content-Encoding:
>
> 7bit
>
>




To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe freebsd-net" in the body of the message




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?3BE24EE4.2020506>