kexec-tools-2.0.2 prints unnecessary message on stderr when loaded in VESA framebuffer
Cong Wang
xiyou.wangcong at gmail.com
Fri Mar 9 02:44:10 EST 2012
On Thu, 08 Mar 2012 at 17:29 GMT, Maxim Kammerer <mk at dee.su> wrote:
> Hi,
>
> In kexec/arch/i386/x86-linux-setup.c:setup_linux_vesafb():
>
> fprintf(stderr, "%s: %dx%dx%d @ %lx +%x\n", __FUNCTION__,
> var.xres, var.yres, var.bits_per_pixel,
> fix.smem_start, fix.smem_len);
>
> This message is pretty annoying (especially if kexec is loaded several
> times with different parameters during startup/shutdown). Looking in
> that source file, that's the only fprintf which is not an error. Is it
> really necessary?
>
This should be just a debugging message, and should not be printed
unless debugging is enabled, so s/fprintf/dbgprintf/...
More information about the kexec
mailing list