Re: [PATCH v2] efi, x86: Add a debug option to the efi= cmdline

2015-02-24 Thread Matt Fleming
On Thu, 05 Feb, at 11:44:41AM, Borislav Petkov wrote: From: Borislav Petkov b...@suse.de Subject: [PATCH v2] efi, x86: Add a debug option to the efi= cmdline ... and hide the memory regions dump behind it. Make it default-off. Signed-off-by: Borislav Petkov b...@suse.de Link: http

Re: [PATCH v2] efi, x86: Add a debug option to the efi= cmdline

2015-02-06 Thread Borislav Petkov
On Fri, Feb 06, 2015 at 01:00:12AM -0500, Parmeshwr Prasad wrote: I am really sorry that I could not mentioned for what purpose this patch is . I wanted this patch to be included along with your patch. As your patch is To add “debug” cmdline in efi=”option”. There are some messages in

Re: [PATCH v2] efi, x86: Add a debug option to the efi= cmdline

2015-02-05 Thread Parmeshwr Prasad
On Thu, Feb 05, 2015 at 08:28:58AM -0600, Borislav Petkov wrote: On Thu, Feb 05, 2015 at 07:45:10AM -0500, Parmeshwr Prasad wrote: It is better if we add some printk from efifb messages. Please review the below patch. First of all, we saw you patch. Then, From

Re: [PATCH v2] efi, x86: Add a debug option to the efi= cmdline

2015-02-05 Thread Parmeshwr Prasad
); + } efifb_fix.ypanstep = 0; efifb_fix.ywrapstep = 0; -- 1.9.3 On Thu, Feb 05, 2015 at 04:44:41AM -0600, Borislav Petkov wrote: From: Borislav Petkov b...@suse.de Subject: [PATCH v2] efi, x86: Add a debug option to the efi= cmdline ... and hide the memory regions dump behind it. Make

Re: [PATCH v2] efi, x86: Add a debug option to the efi= cmdline

2015-02-05 Thread Borislav Petkov
On Thu, Feb 05, 2015 at 07:45:10AM -0500, Parmeshwr Prasad wrote: It is better if we add some printk from efifb messages. Please review the below patch. First of all, we saw you patch. Then, From 7fbac896ab87f1b37646ac2f49bb8216ec330642 Mon Sep 17 00:00:00 2001 From: Parmeshwr Prasad