From: James Carlson <carlsonj@workingcode.com>
To: linux-ppp@vger.kernel.org
Subject: Re: Intervening IPCP Configure Requests
Date: Mon, 16 Aug 2010 14:07:10 +0000 [thread overview]
Message-ID: <4C69460E.4000309@workingcode.com> (raw)
In-Reply-To: <29408130.post@talk.nabble.com>
On 08/16/10 09:33, Jouko Nikula wrote:
> First of all thanks a lot for your reply and sorry about my triple post. It seems that nabble was too complicated for me.
No problem. I did wonder what happened there. ;-}
> James Carlson wrote 12.08.2010 16:20:
>> Of course, you've got the source, so if you just want to hack something
>> into the middle of ipcp_reqci() in pppd/ipcp.c, go ahead. Depending on
>> how your "validation" functions work, you might produce odd results on
>> the wire, including possibly non-converging behavior with some peers.
>> But if you're not worried about that or if it works for you, then go
>> for it.
>
> I did some checks and I think this might work for me. For my application it should be ok that an IPCP configure request blocks for maybe a long time, before it is either Ack'ed or connection is terminated. Do I understand correctly that blocking is the part where you think that problems could arise?
Yes, it's the blocking that I'd be worried about. Note that this blocks
everything in PPP negotiation -- not just IPCP, but all other protocols
that might be negotiating at the same time.
If it works for you, that's great, but I don't think it's the right
general approach to the problem.
> Also do I understand correctly that you are not interested in a patch where I would allow this behaviour with synchronous hooks?
I'd be a little concerned that someone else in the future would try the
same thing and run himself into trouble ... but if the hooks can be
inserted cleanly and coherently, I wouldn't be opposed.
--
James Carlson 42.703N 71.076W <carlsonj@workingcode.com>
next prev parent reply other threads:[~2010-08-16 14:07 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-08-11 13:12 Intervening IPCP Configure Requests Jouko Nikula
2010-08-12 13:20 ` James Carlson
2010-08-16 13:33 ` Jouko Nikula
2010-08-16 14:07 ` James Carlson [this message]
2010-08-18 10:59 ` Jouko Nikula
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4C69460E.4000309@workingcode.com \
--to=carlsonj@workingcode.com \
--cc=linux-ppp@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.