lkml.org 
[lkml]   [2012]   [Oct]   [10]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH 2/5] aio: kiocb_cancel()
    On Tue, Oct 09, 2012 at 02:37:00PM -0700, Kent Overstreet wrote:
    > > Honestly: I wouldn't bother. Nothing of consequence uses cancel.
    > >
    > > I have an RFC patch series that tears it out. Let me polish that up
    > > send it out, I'll cc: you.
    >
    > Even better :)
    >
    > I've been looking at aio locking the past few days, and I was getting
    > ready to write something up about cancellation to the lists.

    I can definitely think of use cases (inside of Google) where we could
    really use aio_cancel. The issue is it's hard use it safely (i.e., we
    need to know whether the file system has already extended the file
    before we can know whether or not we can safely cancel the I/O).

    > Short version, supporting cancellation without global sychronization is
    > possible but it'd require help from the allocator.

    Well, we would need some kind of flag to indicate whether cancellation
    is possible, yes. But if we're doing AIO to a raw disk, or we're
    talking about a read request, we wouldn't need any help from the
    file system's block allocator.

    And maybe the current way of doing things isn't the best way. But it
    would be nice if we didn't completely give up on the functionality of
    aio_cancel.

    - Ted


    \
     
     \ /
      Last update: 2012-10-10 18:01    [W:3.601 / U:0.676 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site