All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH v2 0/7] Hexdump Enhancements
@ 2019-05-08  7:01 ` Alastair D'Silva
  0 siblings, 0 replies; 64+ messages in thread
From: Alastair D'Silva @ 2019-05-08  7:01 UTC (permalink / raw)
  To: alastair
  Cc: Jani Nikula, Joonas Lahtinen, Rodrigo Vivi, David Airlie,
	Daniel Vetter, Dan Carpenter, Karsten Keil, Jassi Brar,
	Tom Lendacky, David S. Miller, Jose Abreu, Kalle Valo,
	Stanislaw Gruszka, Benson Leung, Enric Balletbo i Serra,
	James E.J. Bottomley, Martin K. Petersen, Greg Kroah-Hartman,
	Alexander Viro, Petr Mladek, Sergey Senozhatsky, Steven Rostedt,
	David Laight, Andrew Morton, intel-gfx, dri-devel, linux-kernel,
	netdev, ath10k, linux-wireless, linux-scsi, linux-fbdev, devel,
	linux-fsdevel

From: Alastair D'Silva <alastair@d-silva.org>

Apologies for the large CC list, it's a heads up for those responsible
for subsystems where a prototype change in generic code causes a change
in those subsystems.

This series enhances hexdump.

These improve the readability of the dumped data in certain situations
(eg. wide terminals are available, many lines of empty bytes exist, etc).

The default behaviour of hexdump is unchanged, however, the prototype
for hex_dump_to_buffer() has changed, and print_hex_dump() has been
renamed to print_hex_dump_ext(), with a wrapper replacing it for
compatibility with existing code, which would have been too invasive to
change.

Hexdump selftests have be run & confirmed passed.

Changelog:
 - Fix failing selftests
 - Fix precedence bug in 'Replace ascii bool in hex_dump_to_buffer...'
 - Remove hardcoded new lengths & instead relax the checks in
   hex_dump_to_buffer, allocating the buffer from the heap instead of the
   stack.
 - Replace the skipping of lines of 0x00/0xff with skipping lines of
   repeated characters, announcing what has been skipped.
 - Add spaces as an optional N-group separator
 - Allow byte ordering to be maintained when HEXDUMP_RETAIN_BYTE_ORDERING
   is set.
 - Updated selftests to cover 'Relax rowsize checks' &
   'Optionally retain byte ordering'

Alastair D'Silva (7):
  lib/hexdump.c: Fix selftests
  lib/hexdump.c: Relax rowsize checks in hex_dump_to_buffer
  lib/hexdump.c: Optionally suppress lines of repeated bytes
  lib/hexdump.c: Replace ascii bool in hex_dump_to_buffer with flags
  lib/hexdump.c: Allow multiple groups to be separated by lines '|'
  lib/hexdump.c: Allow multiple groups to be separated by spaces
  lib/hexdump.c: Optionally retain byte ordering

 drivers/gpu/drm/i915/intel_engine_cs.c        |   2 +-
 drivers/isdn/hardware/mISDN/mISDNisar.c       |   6 +-
 drivers/mailbox/mailbox-test.c                |   2 +-
 drivers/net/ethernet/amd/xgbe/xgbe-drv.c      |   2 +-
 .../net/ethernet/synopsys/dwc-xlgmac-common.c |   2 +-
 drivers/net/wireless/ath/ath10k/debug.c       |   3 +-
 .../net/wireless/intel/iwlegacy/3945-mac.c    |   2 +-
 drivers/platform/chrome/wilco_ec/debugfs.c    |   2 +-
 drivers/scsi/scsi_logging.c                   |   8 +-
 drivers/staging/fbtft/fbtft-core.c            |   2 +-
 fs/seq_file.c                                 |   3 +-
 include/linux/printk.h                        |  34 ++-
 lib/hexdump.c                                 | 260 +++++++++++++++---
 lib/test_hexdump.c                            | 146 +++++++---
 14 files changed, 372 insertions(+), 102 deletions(-)

-- 
2.21.0


^ permalink raw reply	[flat|nested] 64+ messages in thread

end of thread, other threads:[~2019-05-13  7:36 UTC | newest]

Thread overview: 64+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-05-08  7:01 [PATCH v2 0/7] Hexdump Enhancements Alastair D'Silva
2019-05-08  7:01 ` Alastair D'Silva
2019-05-08  7:01 ` Alastair D'Silva
2019-05-08  7:01 ` Alastair D'Silva
2019-05-08  7:01 ` [PATCH v2 1/7] lib/hexdump.c: Fix selftests Alastair D'Silva
2019-05-08  7:01   ` Alastair D'Silva
2019-05-08  7:01   ` Alastair D'Silva
2019-05-08  7:01   ` Alastair D'Silva
2019-05-08  7:01 ` [PATCH v2 2/7] lib/hexdump.c: Relax rowsize checks in hex_dump_to_buffer Alastair D'Silva
2019-05-08  7:01   ` Alastair D'Silva
2019-05-08  7:01   ` Alastair D'Silva
2019-05-08  7:01   ` Alastair D'Silva
2019-05-08  7:01 ` [PATCH v2 3/7] lib/hexdump.c: Optionally suppress lines of repeated bytes Alastair D'Silva
2019-05-08  7:01   ` Alastair D'Silva
2019-05-08  7:01   ` Alastair D'Silva
2019-05-08  7:01   ` Alastair D'Silva
2019-05-09  0:58   ` Randy Dunlap
2019-05-09  0:58     ` Randy Dunlap
2019-05-09  0:58     ` Randy Dunlap
2019-05-09  0:58     ` Randy Dunlap
2019-05-10  0:16     ` Alastair D'Silva
2019-05-10  0:16       ` Alastair D'Silva
2019-05-10  0:16       ` Alastair D'Silva
2019-05-10  0:16       ` Alastair D'Silva
2019-05-13  7:01   ` Geert Uytterhoeven
2019-05-13  7:01     ` Geert Uytterhoeven
2019-05-13  7:01     ` Geert Uytterhoeven
2019-05-13  7:01     ` Geert Uytterhoeven
2019-05-13  7:35     ` Alastair D'Silva
2019-05-13  7:35       ` Alastair D'Silva
2019-05-13  7:35       ` Alastair D'Silva
2019-05-13  7:35       ` Alastair D'Silva
2019-05-08  7:01 ` [PATCH v2 4/7] lib/hexdump.c: Replace ascii bool in hex_dump_to_buffer with flags Alastair D'Silva
2019-05-08  7:01   ` Alastair D'Silva
2019-05-08  7:01   ` Alastair D'Silva
2019-05-08  7:01   ` Alastair D'Silva
2019-05-08  8:58   ` Jani Nikula
2019-05-08  8:58     ` Jani Nikula
2019-05-08  8:58     ` Jani Nikula
2019-05-08  8:58     ` Jani Nikula
2019-05-08  9:14   ` Greg Kroah-Hartman
2019-05-08  9:14     ` Greg Kroah-Hartman
2019-05-08  9:14     ` Greg Kroah-Hartman
2019-05-08  9:14     ` Greg Kroah-Hartman
2019-05-08  9:19   ` David Laight
2019-05-08  9:19     ` David Laight
2019-05-08  9:19     ` David Laight
2019-05-08  9:19     ` David Laight
2019-05-08 11:41     ` Alastair D'Silva
2019-05-08 11:41       ` Alastair D'Silva
2019-05-08 11:41       ` Alastair D'Silva
2019-05-08 11:41       ` Alastair D'Silva
2019-05-08  7:01 ` [PATCH v2 5/7] lib/hexdump.c: Allow multiple groups to be separated by lines '|' Alastair D'Silva
2019-05-08  7:01   ` Alastair D'Silva
2019-05-08  7:01   ` Alastair D'Silva
2019-05-08  7:01   ` Alastair D'Silva
2019-05-08  7:01 ` [PATCH v2 6/7] lib/hexdump.c: Allow multiple groups to be separated by spaces Alastair D'Silva
2019-05-08  7:01   ` Alastair D'Silva
2019-05-08  7:01   ` Alastair D'Silva
2019-05-08  7:01   ` Alastair D'Silva
2019-05-08  7:01 ` [PATCH v2 7/7] lib/hexdump.c: Optionally retain byte ordering Alastair D'Silva
2019-05-08  7:01   ` Alastair D'Silva
2019-05-08  7:01   ` Alastair D'Silva
2019-05-08  7:01   ` Alastair D'Silva

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.