lkml.org 
[lkml]   [2008]   [Mar]   [25]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH v2] Re: WAN: new PPP code for generic HDLC
    Krzysztof Halasa wrote:
    > Hi,
    >
    > I wrote:
    >
    >> New synchronous PPP implementation for generic HDLC.
    >
    > So are we leaving the generic-HDLC+PPP in broken state for 2.6.25,
    > aren't we?
    >
    > If we do, perhaps a "|| BROKEN" in drivers/net/wan/Kconfig would make
    > sense? Any attempt to use it will cause kernel panic immediately (PPP
    > with generic HDLC only; drivers using syncppp.c directly are not
    > affected). I can make that trivial Kconfig patch of course.

    In your original email you said

    I guess it should go into 2.6.25, not sure about "stable"
    series. I will appreciate any feedback, review and/or test
    results.

    At the time of the posting 2.6.25-rc6 had already been released, which
    seems like an inappropriate time for all that new code, which has been
    given so little exposure to real world testing.

    Certainly your original message said PPP panics, but without even
    minimal testing how do we know that your new code doesn't have equally
    problematic issues?

    So quite honestly a CONFIG_BROKEN patch might indeed be more appropriate
    since generic HDLC works with Frame Relay at least...

    Comments? IMO the current code is a known risk (PPP panic, FR works)
    whereas the new code is an unknown risk very late in 2.6.25-rc -- but
    with a good chance to make HDLC+PPP work again.

    Jeff





    \
     
     \ /
      Last update: 2008-03-25 15:57    [W:0.036 / U:29.468 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site