Re: Shorten efi regions output

2015-01-26 Thread Matt Fleming
On Wed, 21 Jan, at 11:06:33AM, Borislav Petkov wrote: > On Wed, Jan 21, 2015 at 12:48:58AM -0500, Jon Masters wrote: > > We have on a number of occasions found just this output useful in > > debugging boot issues on ARM servers > > We can turn it on by default on ARM and off on x86... Yeah, there

Re: Shorten efi regions output

2015-01-21 Thread Borislav Petkov
On Wed, Jan 21, 2015 at 12:48:58AM -0500, Jon Masters wrote: > We have on a number of occasions found just this output useful in > debugging boot issues on ARM servers We can turn it on by default on ARM and off on x86... -- Regards/Gruss, Boris. ECO tip #101: Trim your mails when you reply

Re: Shorten efi regions output

2015-01-20 Thread Jon Masters
On 1/5/15, 10:00 AM, Laszlo Ersek wrote: On 01/05/15 15:03, Matt Fleming wrote: On Wed, 10 Dec, at 11:46:28AM, Borislav Petkov wrote: On Wed, Dec 10, 2014 at 10:17:41AM +0800, Dave Young wrote: I have same feeling with you, it is too long for most of people. Since the printk code are for EFI_

Re: Shorten efi regions output

2015-01-05 Thread Laszlo Ersek
On 01/05/15 15:03, Matt Fleming wrote: > On Wed, 10 Dec, at 11:46:28AM, Borislav Petkov wrote: >> On Wed, Dec 10, 2014 at 10:17:41AM +0800, Dave Young wrote: >>> I have same feeling with you, it is too long for most of people. >>> >>> Since the printk code are for EFI_DEBUG, they are around the #if

Re: Shorten efi regions output

2015-01-05 Thread Matt Fleming
On Wed, 10 Dec, at 11:46:28AM, Borislav Petkov wrote: > On Wed, Dec 10, 2014 at 10:17:41AM +0800, Dave Young wrote: > > I have same feeling with you, it is too long for most of people. > > > > Since the printk code are for EFI_DEBUG, they are around the #ifdef > > so I would like to see a kernel pa

Re: Shorten efi regions output

2014-12-10 Thread Borislav Petkov
On Wed, Dec 10, 2014 at 10:17:41AM +0800, Dave Young wrote: > I have same feeling with you, it is too long for most of people. > > Since the printk code are for EFI_DEBUG, they are around the #ifdef > so I would like to see a kernel param like efi_debug=on, so only > efi_debug is specified then the

Re: Shorten efi regions output

2014-12-09 Thread Dave Young
On 12/09/14 at 10:58am, Borislav Petkov wrote: > Hi guys, > > so this decoded EFI regions output is nice but can we shorten it? It > sticks too far out in the terminal more than anything else in dmesg and > staring at it needs me to resize window and such. It probably is an even > bigger problem i

Re: Shorten efi regions output

2014-12-09 Thread Borislav Petkov
On Tue, Dec 09, 2014 at 04:35:33PM +0100, Laszlo Ersek wrote: > I disagree with the patch in general, and I strongly disagree with this > specific change in particular. This part: > > - breaks the alignment for the right side of the table Well, we can start by removing the sizes - they're useless

Re: Shorten efi regions output

2014-12-09 Thread Laszlo Ersek
On 12/09/14 10:58, Borislav Petkov wrote: > Hi guys, > > so this decoded EFI regions output is nice but can we shorten it? It > sticks too far out in the terminal more than anything else in dmesg and > staring at it needs me to resize window and such. It probably is an even > bigger problem if one

Re: Shorten efi regions output

2014-12-09 Thread Borislav Petkov
On Tue, Dec 09, 2014 at 01:42:44PM +0100, Ard Biesheuvel wrote: > The |] thing is easily fixed, though. > > [...] > > diff --git a/drivers/firmware/efi/efi.c b/drivers/firmware/efi/efi.c > > index 8590099ac148..6734072980ee 100644 > > --- a/drivers/firmware/efi/efi.c > > +++ b/drivers/firmware/efi

Re: Shorten efi regions output

2014-12-09 Thread Ard Biesheuvel
On 9 December 2014 at 10:58, Borislav Petkov wrote: > Hi guys, > > so this decoded EFI regions output is nice but can we shorten it? It > sticks too far out in the terminal more than anything else in dmesg and > staring at it needs me to resize window and such. It probably is an even > bigger prob