On Sat 2019-04-13 09:22:05, Alastair D'Silva wrote: > > -----Original Message----- > > From: Petr Mladek <[email protected]> > > Sent: Friday, 12 April 2019 11:48 PM > > To: Alastair D'Silva <[email protected]> > > Cc: [email protected]; Jani Nikula <[email protected]>; > Joonas > > Lahtinen <[email protected]>; Rodrigo Vivi > > <[email protected]>; David Airlie <[email protected]>; Daniel Vetter > > <[email protected]>; Karsten Keil <[email protected]>; Jassi Brar > > <[email protected]>; Tom Lendacky <[email protected]>; > > David S. Miller <[email protected]>; Jose Abreu > > <[email protected]>; Kalle Valo <[email protected]>; > > Stanislaw Gruszka <[email protected]>; Benson Leung > > <[email protected]>; Enric Balletbo i Serra > > <[email protected]>; James E.J. Bottomley > > <[email protected]>; Martin K. Petersen <[email protected]>; > > Greg Kroah-Hartman <[email protected]>; Alexander Viro > > <[email protected]>; Sergey Senozhatsky > > <[email protected]>; Steven Rostedt <[email protected]>; > > Andrew Morton <[email protected]>; intel- > > [email protected]; [email protected]; linux- > > [email protected]; [email protected]; > > [email protected]; [email protected]; linux- > > [email protected]; [email protected]; > > [email protected]; [email protected] > > Subject: Re: [PATCH 1/4] lib/hexdump.c: Allow 64 bytes per line > > > > On Wed 2019-04-10 13:17:17, Alastair D'Silva wrote: > > > From: Alastair D'Silva <[email protected]> > > > > > > With modern high resolution screens, we can display more data, which > > > makes life a bit easier when debugging. > > > > I have quite some doubts about this feature. > > > > We are talking about more than 256 characters per-line. I wonder if such a > > long line is really easier to read for a human. > > It's basically 2 separate panes of information side by side, the hexdump and > the ASCII version. > > I'm using this myself when dealing with the pmem labels, and it works quite > nicely.
I am sure that it works for you. But I do not believe that it would be useful in general. > > I am not expert but there is a reason why the standard is 80 characters > per- > > line. I guess that anything above 100 characters is questionable. > > https://en.wikipedia.org/wiki/Line_length > > somehow confirms that. > > > > Second, if we take 8 pixels per-character. Then we need > > 2048 to show the 256 characters. It is more than HD. > > IMHO, there is still huge number of people that even do not have HD > display, > > especially on a notebook. > > The intent is to make debugging easier when dealing with large chunks of > binary data. I don't expect end users to see this output. How is it supposed to be used then? Only by your temporary patches? Best Regards, Petr _______________________________________________ devel mailing list [email protected] http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
