On Wed, May 22, 2013 at 04:42:53PM +0300, Mika Kuoppala wrote: > Sometimes when user is trying to get error state out from > debugfs after gpu hang, the memory is low and/or fragmented > enough that kmalloc in seq_file will fail. > > Prevent big kmalloc by avoiding seq_file and instead convert > error state to string in smaller chunks. > > v2: better alloc flags, better truncate, correct > locking, and error handling improvements (Chris Wilson) > > Signed-off-by: Mika Kuoppala <mika.kuoppala at intel.com> I am reasonably happy with this. Every attempt to see how we might shoe-horn iterating over the error state with seq_file has so far been overly complicated, and the only benefit it may bring is the focus on iterating at a higher level than characters. Reviewed-by: Chris Wilson <chris at chris-wilson.co.uk> -Chris -- Chris Wilson, Intel Open Source Technology Centre