On Tue 08-10-19 10:03:01, Qian Cai wrote: [..] > I don't think there is "removing useful messages" in this patch. That one > printk() in __offline_isolated_pages() basically as Michal mentioned it is that > useful, but could be converted to printk_deferred() if anyone objected. "remove from free list" printk is quite dubious as I've said. And I wouldn't be opposed to remove it altogether. It just swamps the low with something that is of no real use. It might have been useful while shaping and debugging the initial code but it has close to zero value for end users. But it makes no sense to make to make it printk_deferred on grounds mentioned so far. -- Michal Hocko SUSE Labs