lkml.org 
[lkml]   [2018]   [Sep]   [18]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH v2] kcm: remove any offset before parsing messages
    David Miller wrote on Mon, Sep 17, 2018:
    > From: Dominique Martinet <asmadeus@codewreck.org>
    > Date: Wed, 12 Sep 2018 07:36:42 +0200
    >
    > > Dominique Martinet wrote on Tue, Sep 11, 2018:
    > >> Hmm, while trying to benchmark this, I sometimes got hangs in
    > >> kcm_wait_data() for the last packet somehow?
    > >> The sender program was done (exited (zombie) so I assumed the sender
    > >> socket flushed), but the receiver was in kcm_wait_data in kcm_recvmsg
    > >> indicating it parsed a header but there was no skb to peek at?
    > >> But the sock is locked so this shouldn't be racy...
    > >>
    > >> I can get it fairly often with this patch and small messages with an
    > >> offset, but I think it's just because the pull changes some timing - I
    > >> can't hit it with just the clone, and I can hit it with a pull without
    > >> clone as well.... And I don't see how pulling a cloned skb can impact
    > >> the original socket, but I'm a bit fuzzy on this.
    > >
    > > This is weird, I cannot reproduce at all without that pull, even if I
    > > add another delay there instead of the pull, so it's not just timing...
    >
    > I really can't apply this patch until you resolve this.
    >
    > It is weird, given your description, though...

    Thanks for the reminder! I totally agree with you here and did not
    expect this to be merged as it is (in retrospect, I probably should have
    written something to that extent in the subject, "RFC"?)

    I really don't have much time to give to that right now as I'm doing
    this on my freetime, and the lack of reply has been rather demotivating
    so it got pushed back a few times...
    Given you did reply now I'll try to spend some time to figure that out
    in the next couple of weeks but it might not make it for this cycle
    depending on the number of rc we'll get and time you want this to soak
    it -next.


    (I can start by putting the pull back in netparser and try to reproduce,
    it's really weird that I never got it to happen at the time...)

    --
    Dominique

    \
     
     \ /
      Last update: 2018-09-18 03:58    [W:4.185 / U:0.728 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site