[lkml]   [2007]   [Jun]   [1]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [RFC] [PATCH] cpuset operations causes Badness at mm/slab.c:777 warning
    Christoph wrote:
    > Hmmmm.... Isnt it easier to set the pidarray to NULL in the case it
    > has no objects? Then any deferences will fail. Paul?

    From what I can tell, in 30 seconds of reading, either of these
    approaches, Christoph's or Srinivasa's, based on Vatsa's suggestions
    -could- be made to work.

    I don't understand how Christoph's approach works, as stated, however.

    What do you mean, Christoph, by "then any dereferences will fail" ?
    That sounds bad to me -- we don't want pointer dereferences failing
    in the kernel, do we? Wouldn't you have to add some more lines of
    code, a bit further down, where pidarray is used, to avoid trying to
    use it if its NULL?

    I think that Srinivasa's point is that it is legitimate for this array
    to be empty, and that we have to code for that case, one way or another.
    Now that we can't simply kmalloc zero bytes and get away with it, that
    requires explicit tests in the code for the empty case, one way or
    another. This sounds like a valid point to me.

    I won't rest till it's the best ...
    Programmer, Linux Scalability
    Paul Jackson <> 1.925.600.0401
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to
    More majordomo info at
    Please read the FAQ at

     \ /
      Last update: 2007-06-01 21:13    [W:0.021 / U:0.372 seconds]
    ©2003-2017 Jasper Spaans. hosted at Digital OceanAdvertise on this site