lkml.org 
[lkml]   [2023]   [Aug]   [6]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    From
    SubjectRe: [PATCHSET v4] Add io_uring futex/futexv support
    Date
    Jens!

    On Sun, Aug 06 2023 at 10:44, Jens Axboe wrote:
    > On 7/31/23 10:06?AM, Thomas Gleixner wrote:
    >> Can you please just wait until the futex core bits have been agreed on
    >> and merged? No need to contribute more mess in everyones inbox.
    >
    > Also no need to keep dragging out the review of the other bits. The
    > dependency is only there so we can use FUTEX2 flags for this - which
    > does make sense to me, but we should probably split Peter's series in
    > two as there's no dependency on the functional bits on that patch
    > series. As we're getting ever closer to the merge window, and I have
    > other things sitting on top of the futex series, that's problematic for
    > me.

    Seriously?

    You are still trying to sell me "Features first - corrrectness
    later/never"?

    Go and look at the amount of fallout this has caused in the last years.
    io-urine is definitely the leader of the pack in my security@kernel.org
    inbox.

    Vs. the problem at hand. I've failed to catch a major issue with futex
    patches in the past and I'm not seeing any reason to rush any of this to
    repeat the experience just because.

    You know very well that your whatever depends on this series has to wait
    until the basics are sorted and there is absolutely no reason that your
    so important things have to be rushed into the next merge window.

    It surely makes sense to split these things up into independent series,
    but _you_ could have done that weeks ago instead of just reposting an
    umodified and unreviewed RFC series from Peter and then coming out now
    and complaining about the lack of progress.

    Sorry no.

    Thanks,

    tglx

    \
     
     \ /
      Last update: 2023-08-07 03:25    [W:9.014 / U:0.112 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site