lkml.org 
[lkml]   [2011]   [Nov]   [16]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    SubjectRe: [PATCH v2] pstore: pass allocated memory region back to caller
    From
    On Wed, Nov 16, 2011 at 2:20 PM, Don Zickus <dzickus@redhat.com> wrote:
    > This is an interesting approach.  But are we leaving psinfo data exposed
    > when you have a reader and writer at the same time?

    I look at this as the first step in separating the read & write paths.

    I started out with the (good) idea that the back end should allocate & own
    the buffer for the write path ... this means that the buffer is ready to use
    when an oops/panic happens - which is obviously a bad time to need to
    allocate memory :-)

    Then I reused the same buffer for read - in hindsight this was not such a
    good idea - it led to all the discussions we've had about how to guarantee
    that the dmesg data gets saved on panic - even in the cases where we
    can't get the locks (so proposals have been made to bust the locks).

    So Kees' patch is the functional equivalent of busting the spinlock.

    Next step would be to look at the back end drivers to see whether
    they can handle a simultaneous read & write in a graceful way.

    I've queued this for linux-next. Probably missed the snapshot today,
    but I expect it should show up in next-20111118
    --
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to majordomo@vger.kernel.org
    More majordomo info at http://vger.kernel.org/majordomo-info.html
    Please read the FAQ at http://www.tux.org/lkml/

    \
     
     \ /
      Last update: 2011-11-16 23:47    [W:0.027 / U:0.292 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site