On Wed, Sep 03, 2014 at 09:00:42PM +0200, Johannes Berg wrote: > On Wed, 2014-09-03 at 09:38 -0500, Seth Forshee wrote: > > > Overall I think this looks pretty sensible. The thing that worries me > > though is firmware which might crash repeatedly in a short period of > > time, resulting in a proliferation of coredumps and eating up all > > available RAM. How about a limit of one active coredump per device or > > something similar? > > Oh, right, I completely forgot about that. Not sure what context I can > iterate the class device list, but worst case I have to add another > list.
Before you add another device, walk the list of all devices of the class to see if there is already a device with the same parent. If so, delete that one and then continue and add the one you wanted to create. > We likely want to store the first dump then, I suppose? Or the last one? I don't know... greg k-h -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/