On Fri, May 04, 2018 at 12:56:55PM +1000, Michael Ellerman wrote: > Sam Bobroff <sbobr...@linux.ibm.com> writes: > > > Correct two cases where eeh_pcid_get() is used to reference the driver's > > module but the reference is dropped before the driver pointer is used. > > > > In eeh_rmv_device() also refactor a little so that only two calls to > > eeh_pcid_put() are needed, rather than three and the reference isn't > > taken at all if it wasn't needed. > > This sounds like a crash or memory corruption bug? > > But it doesn't have Fixes or Cc: stable. Is it not a major problem for > some reason?
Only that I've exercised that code path a fair bit during testing and never managed to cause a problem with it. I found it by inspection. Do you think I should mark it fixes or stable in the next version? > cheers >
signature.asc
Description: PGP signature