> -----Original Message----- > From: Alan Stern [mailto:[email protected]] > Sent: Monday, February 04, 2013 11:16 AM > To: Li, Fei > Cc: Rafael J. Wysocki; [email protected]; > [email protected]; [email protected]; Liu, Chuansheng > Subject: RE: [PATCH V4] suspend: enable freeze timeout configuration through > sys > > On Mon, 4 Feb 2013, Li, Fei wrote: > > > > -> the situation described above shouldn't happen and if it does, then > > > there > > > is a bug that needs to be fixed. > > > > Yes, we agree with that it's a bug that needs to be fixed, and we are > > already > done > > for some cases. > > During the process, we realize that tuning the timeout value to smaller > > value > will help > > to expose freezing failure for some cases. > > Besides, the underlying bug is well known (fuse filesystems hanging > because the fuse daemon gets frozen before the process doing file I/O, > or something like that) and so far nobody has a good idea of how to > fix it. Indeed, we have some well-known deadlock issues existed there. And once there is deadlock case, this patch is helpful to the user experience, because 20s is pointless for those deadlock case.
Also even adjusting the 20s to 2s, we still can get to know the process stack who rejecting the freezing, so this freezing configuration is really helpful to avoid 20s waiting for that deadlock case. Alan and Rafael, could you consider this patch? Thanks. > > Alan Stern -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
