Michael Ellerman [[email protected]] wrote: | On Tue, 2015-17-02 at 22:00:27 UTC, Sukadev Bhattiprolu wrote: | > Use pr_notice_ratelimited() to log error messages and remove | > the 'success_expected' parameter. | | I don't understand how this is equivalent?
They are two unrelated changes that I should have separated. | | The current code uses success_expected to indicate that once it's done the | request once and found that it works, it then expects the request to continue | working, and if it doesn't then that is an error. The current code is using success_expected to _not_ log an error if that initial request fails. i.e we silently return -EIO here. I think the 'success_expected' parameter is not really necessary. We can simply log the message even for that initial request. And we can log it a lower priority than KERN_ERR since the message is mostly for developers rather than users who would use event names (which encode/abstract the domain and offset values). | | Using pr_ratelimited() will do the opposite, ie. the first failure will print a | message, but that may not really indicate an error, it may just be a badly | configured request. | | Or at least that's how I understand it, please convince me I'm wrong :) | | cheers | _______________________________________________ | Linuxppc-dev mailing list | [email protected] | https://lists.ozlabs.org/listinfo/linuxppc-dev -- 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/
