Hi Michael,<br><br>On Wednesday, December 09, 2015 11:17:12 AM Michael Ellerman wrote:<br>> Hi Wen,<br>> <br>> Thanks for the patch.<br>> <br>> On Tue, 2015-12-08 at 15:30 +0800, wen.pu@powercore.com.cn wrote:<br>> > I found that there are unimplemented OPAL message types "OPAL_MSG_EPOW"<br>> > and<br>> > "OPAL_MSG_DPO" in PowerNV Platform. If skiboot send such messages to<br>> > kernel, nothing will happen(notifier_call_chain() will silently drop it).<br>> > So I add a warning to inform what has happened, is it necessary?<br>> <br>> What kernel version are you using?<br>The kernel version is v4.2.1-openpower2, from the newest version at <br>https://github.com/open-power/linux.<br><br>> <br>> Looking at mainline (4.4-rc4), I do see handlers registered for EPOW and<br>> DPO:<br>> <br>>  <br>> https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/tree/arch/p<br>> owerpc/platforms/powernv/opal-power.c?#n82<br>> <br>> That code was merged in 4.3.<br>I do find that EPOW and DPO are registered in kernel 4.3. So maybe in the near <br>future it will be merged into open-power/linux.<br><br>> <br>> <br>> In general I don't think we want to warn for unhandled message types,<br>> because we support old kernels running on new skiboot. In that case if we<br>> warned on every unhandled message we'd potentially spam the console with<br>> warnings.<br>> <br>> But perhaps we can come up with something smarter, that only warns once for<br>> each unhandled message type, or something like that? Did you have a bug that<br>> would have been easier to debug with a warning here?<br>Yes, I think it is a good idea that only warns once, or warns every a few <br>seconds. <br>I found that EPOW and DPO only used in IBM FSP platform, not in Habanero <br>platform that I had. So I modified the skiboot code and sent a EPOW/DPO message <br>to kernel, and found that kernel warn nothing. I have no bug with a warning <br>here, and just get a log message via dmesg.<br><br>> <br>> cheers<br><br>Thanks!<br>Wen Pu<!--<![endif]-->