On Tue, Feb 07, 2023 at 02:42:14AM +0000, David Rau wrote: > > > You mean just keep the above two patches and revert 969357ec94e6 ? > > Sure, I can do that, but feedback from the field would take some > > 2-3 months. Is that what you recommend to do for now ? > > > Thanks, > > Guenter > > Thanks for the feedback. > What I mean is just do rollback to remove the "sleep" patch I did in your repository. > > After the rollback, could you please let me know the average number of crashes per affected system with the same test conditions? > Will it still take some 2-3 months? > The msleep() patch has been reverted in R111 and dev releases of ChromeOS. I did not get permission to land the revert in R110, meaning we'll continue to see the crashes there. R111 is expected to go to Beta shortly, so we should get _some_ feedback in the next few weeks. Still, it would be great to get a more permanent fix for the underlying problem. Also, the msleep() patch is still upstream, so a solution is still needed there. I can try to get and play with one of the affected Chromebooks, but I don't think that would help much since we still don't know what the undocumented ground switch is supposed to do. Thanks, Guenter
