On Tuesday, June 22, 2010, Maxim Levitsky wrote: > On Tue, 2010-06-22 at 11:19 +0200, Rafael J. Wysocki wrote: > > On Tuesday, June 22, 2010, Maxim Levitsky wrote: > > > On Mon, 2010-06-21 at 22:26 +0200, Rafael J. Wysocki wrote: ... > > > I *do* use freezable work-queue. > > > > I overlooked that, sorry. > > > > > However since this is pm notifier, it is called before userspace and the > > > workqueue is frozen. > > > Therefore I would like manually to freeze the workqueue from the pm > > > notifier. > > > > No, that won't work. You need to find an alternative solution. I guess you > > may insert a work item that's going to sleep until a condition is > > satisfied (analogous to a workqueue barrier) and wait for it to > This screams to be done in generic way. > Something like suspend_workqueue() and resume_workqueue(); Well, there was no need for that until now. :-) > In addition to that I just found that .suspend function sometimes can > return -ENOSYS, which triggers card removal. I wrongly remove that chunk > of code. > > To make the thing picture perfect I would have to invest more time to > it, I will do so as soon as I finish my exams. > > Meanwhile the current patch already fixes all but corner cases or rather > nasty hang on suspend with any MMC/SD card inserted. OK I think Andrew has already taken [2/2]. Andrew, who's maintaining MMC now? Rafael -- To unsubscribe from this list: send the line "unsubscribe linux-mmc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
