lkml.org 
[lkml]   [2014]   [Oct]   [22]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    SubjectRe: [PATCH v4 00/12] fs/seq_file: introduce seq_hex_dump() helper
    From
    Date
    On Thu, 2014-09-04 at 17:26 +0300, Andy Shevchenko wrote:
    > This introduces a new helper and switches current users to use it. All are
    > compiled tested, hexdump and kmemleak are tested by their test suits.
    >

    Gently ping on this series.
    Any comments?

    > Changelog v4:
    > - hexdump is amended to return value from which we can check an overflow
    > (patches 1/12 - 4/12)
    > - seq_hex_dump pushes bytes directly to buffer and returns an error in case of
    > overflow (addresses Al Viro's comment)
    > - append Acked-by for patches 8/12 and 9/12
    > - convert more users (patches 10/12 - 12/12)
    >
    > Changelog v3:
    > - append Mauro's Ack
    > - rebase on top of recent linux-next
    >
    > Changelog v2:
    > - append Acked-by and Reviewed-by tags
    > - update commit messages in patches 3/5. and 5/5
    > - update line size to be 32 bytes instead of 16 in patch 3/5
    > - Joe found that output is changed in patch 4/5, thus I update commit message
    > there
    >
    > Andy Shevchenko (12):
    > hexdump: introduce test suite
    > hexdump: fix ascii column for the tail of a dump
    > hexdump: do few calculations ahead
    > hexdump: makes it return amount of bytes placed in buffer
    > seq_file: provide an analogue of print_hex_dump()
    > saa7164: convert to seq_hex_dump()
    > crypto: qat - use seq_hex_dump() to dump buffers
    > parisc: use seq_hex_dump() to dump buffers
    > [S390] zcrypt: use seq_hex_dump() to dump buffers
    > staging: unisys: use seq_hex_dump() to dump buffers
    > kmemleak: use seq_hex_dump() to dump buffers
    > wil6210: use seq_hex_dump() to dump buffers
    >
    > .../crypto/qat/qat_common/adf_transport_debug.c | 16 +-
    > drivers/media/pci/saa7164/saa7164-core.c | 31 +---
    > drivers/net/wireless/ath/wil6210/debugfs.c | 32 +---
    > drivers/parisc/ccio-dma.c | 14 +-
    > drivers/parisc/sba_iommu.c | 11 +-
    > drivers/s390/crypto/zcrypt_api.c | 10 +-
    > .../unisys/visorchannel/visorchannel_funcs.c | 26 +--
    > fs/seq_file.c | 50 ++++++
    > include/linux/printk.h | 6 +-
    > include/linux/seq_file.h | 4 +
    > lib/Kconfig.debug | 3 +
    > lib/Makefile | 4 +-
    > lib/hexdump.c | 105 +++++++-----
    > lib/test-hexdump.c | 180 +++++++++++++++++++++
    > mm/kmemleak.c | 21 +--
    > 15 files changed, 337 insertions(+), 176 deletions(-)
    > create mode 100644 lib/test-hexdump.c
    >


    --
    Andy Shevchenko <andriy.shevchenko@intel.com>
    Intel Finland Oy



    \
     
     \ /
      Last update: 2014-10-22 18:21    [W:4.160 / U:0.220 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site