On 2017-05-22 20:07, Alan Cox wrote: > On Mon, 22 May 2017 16:06:36 +0200 > Rasmus Villemoes <rasmus.villemoes@xxxxxxxxx> wrote: > >> If a watchdog driver tells the framework that the device is running, >> the framework takes care of feeding the watchdog until userspace opens >> the device. If the userspace application which is supposed to do that >> never comes up properly, the watchdog is fed indefinitely by the >> kernel. This can be especially problematic for embedded devices. >> >> These patches allow one to set a maximum time for which the kernel >> will feed the watchdog, thus ensuring that either userspace has come >> up, or the board gets reset. This allows fallback logic in the >> bootloader to attempt some recovery (for example, if an automatic >> update is in progress, it could roll back to the previous version). > > > This makes sense except for being a CONFIG_ option not a boot parameter. > If it's a boot parameter then the same kernel works for multiple systems > and is general. If it's compile time then you have to build a custom > kernel. I don't understand. Patch 3 simply allows the default to be set in the config, but it is very much still possible to set it via the boot command line, overriding the default (whether that was a CONFIG or a hardcoded 0). > For some embedded stuff that might not matter (although I bet they'd > prefer it command line/device tree too) When building a full BSP, yes, one can cook it into the boot command line in u-boot, but that means that to update it one has to edit one's u-boot recipe. It also often makes testing-turnaround times longer, since updating the kernel can often be done by just scp'ing a new image to the board, while updating the bootloader is sometimes a much more involved procedure (not to mention that that's sometimes not even possible to do remotely). There's also the issue of the command line getting rather crowded if every aspect of the kernel has to be configured at boot time. As an embedded developer, I'm all for being able to override this timeout as well lots of other kernel behavior via the command line, but at the same time, I also prefer being able to set the default values in the kernel's .config. but for something like an x86 > platform where you are deploying a standard vendor supplied kernel it's > bad to do it that way IMHO. A "standard vendor supplied kernel" will never set that CONFIG to anything but its default - how could it possibly decide how long userspace should have to open /dev/watchdog0, or even if userspace will ever do that? Again, patch 3 doesn't in any way remove the possiblity of setting this on the command line, and the end user is no worse off just because the vendor had to keep a default setting. > In other words I think you should drop patch 3 but the rest is good. For the life of me, I cannot see the downside of patch 3, or how it's any different from CONSOLE_LOGLEVEL_DEFAULT, BLK_DEV_RAM_SIZE, or any number of other parameters-with-defaults-from-Kconfig. Rasmus -- To unsubscribe from this list: send the line "unsubscribe linux-doc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
