On Fri, Aug 31, 2012 at 01:15:01PM -0500, Mark Tinguely wrote: > I see your point on fixing problems in older branches when/if they > are reported by an user. I am not glowing with pride with the patch, > it is something that survived a week of testing that would cause a > panic in a couple hours without the patch. Since we hit this problem > with such frequency, that we wanted to push for a little proactive > attention to prevent future panics. I'd love to see a relatively minimal patch which can also be backported. I also have to say that your patch as-is scares me a bit. Everytime we move the current xfs_sync_init/stop monsters around we created another set of problems. So I'd prefer at least taking the bits from Dave's series that kill these helpers and do individual calls, and only move those that are needed. The other thing that I don't like about the patch is that it causes assymetry in the mount/unmount path by moving the stop into xfs_unmountfs but not the start into xfs_mountfs. That might be needed in some cases, but that should be some detailed comments explaining why. _______________________________________________ xfs mailing list xfs@xxxxxxxxxxx http://oss.sgi.com/mailman/listinfo/xfs
