Al Viro <viro@xxxxxxxxxxxxxxxxxx> writes: > Speaking of bogosities, apparently nobody has noticed that automagical > turning BSD process accounting off upon r/o remounts of the filesystem > hosting the log got broken several years ago. Suppose we find an opened > log when remounting. OK, it gets closed, which means filp_close(...). > Which means that actual closing that sucker will happen just before we > return to userland. Return with -EBUSY, since the filesystem still has > a file opened for write when we get around to checking that... > > I have kinda-sorta fixes for that (basically, restoring the situation we > used to have before delayed fput() and being damn careful about avoiding > deadlocks), but I would really love to just rip that idiocy out. IOW, > just let it fail with -EBUSY in such cases, same as for any other write-opened > file on that fs. > > That, BTW, is the most painful part of the acct series, so being able to > drop it would be very nice. Comments? Long ago in this discucssion someone mentioned that fedora and related distro's don't have (or at least didn't have) code to close the accounting files before unmount during shutdown. I just checked the debain acct package and it very clearly turns off accounting before unmount (so debian should not care). So the practical test is will returning -EBUSY cause fedora based distro's it fail to unmount filesystems cleanly (looking at the code and postualting bsd process accounting being left on) I think such a change will cause such systems to fail to unmount their filesystems on shutdown/reboot and cause problems because of that. Eric -- To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
