lkml.org 
[lkml]   [2006]   [May]   [18]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [openib-general] Re: [PATCH 35 of 53] ipath - some interrelated stability and cleanliness fixes
    On Wed, 17 May 2006, Roland Dreier wrote:
    | I do see obvious bugs in ipath_user_pages.c, though. In
    | ipath_release_user_pages_on_close(), you have:
    |
    | mm = get_task_mm(current);
    | if (!mm)
    | goto bail;

    It turns out that since this is called from ipath_close(),
    mm will always be NULL, so what we do is leak memory, and
    possibly leave some locked pages. I've been looking at this
    code this evening; fixing it is clearly needed, but doesn't help
    the long delays, hangs, and watchdogs, so far.

    Dave Olson
    olson@unixfolk.com
    http://www.unixfolk.com/dave
    -
    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: 2006-05-18 09:06    [W:0.021 / U:121.208 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site