Skip to content

s390/dasd: string handling cleanups

Stefan Haberland linux-block linux-s390 2 versions – Applied

Status from this message: Applied, thanks!

Get series with b4
b4 am [email protected]

Versions newest first · linked by author and title

  1. v1 Latest posting 2/29 of 11 patches 1 replyApplied
  2. v1 posting 1/29 of 10 patches 0 repliesSuperseded

9 of 11 patches are in this archive. The cover letter announces 11.

Patches in v1 posting 2/2 Applied

#Subject and filesSizeRepliesApplied
1/11s390/dasd: Simplify uid string generationdrivers/s390/block/dasd_devmap.c drivers/s390/block/dasd_eckd.c+7 -210 replies
2/11s390/dasd: Use sysfs_emit() over sprintf()drivers/s390/block/dasd_devmap.c+9 -90 replies
3/11s390/dasd: Remove unnecessary errorstring generationdrivers/s390/block/dasd.c drivers/s390/block/dasd_3990_erp.c drivers/s390/block/dasd_int.h+8 -290 replies
4/11s390/dasd: Move allocation error message to DBFdrivers/s390/block/dasd.c+2 -40 replies
5/11s390/dasd: Remove unused message logging macrosdrivers/s390/block/dasd_int.h+0 -260 replies
6/11s390/dasd: Use dev_err() over printk()drivers/s390/block/dasd_eckd.c drivers/s390/block/dasd_fba.c+76 -1180 replies
7/11s390/dasd: Remove %p format specifier from error messagesdrivers/s390/block/dasd.c drivers/s390/block/dasd_3990_erp.c drivers/s390/block/dasd_erp.c+15 -230 replies
8/11s390/dasd: Remove PRINTK_HEADER and KMSG_COMPONENT definitionsdrivers/s390/block/dasd.c drivers/s390/block/dasd_3990_erp.c drivers/s390/block/dasd_alias.c +8 more+1 -550 replies
9/11s390/dasd: Use dev_*() for device log messagesdrivers/s390/block/dasd.c+24 -260 replies

Reviews trailers per patch

All 9 patches have a Reviewed-by or Acked-by.

PatchStefan Haberland
1/11R-b
2/11R-b
3/11R-b
4/11R-b
5/11R-b
6/11R-b
7/11R-b
8/11R-b
9/11R-b

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v1 thread

Cover letter [PATCH RESEND 00/11] s390/dasd: string handling cleanups

Hi Jens,

looks like the patchset was not applied for the last merge window. So I am resending it rebased to the current for-next branch. Please apply the patches for the upcomming merge window. They clean up the string handling in the DASD driver.

In addition there is a patch that fixes double module refcount decrement.

Jan Höppner (10):
s390/dasd: Simplify uid string generation
s390/dasd: Use sysfs_emit() over sprintf() s390/dasd: Remove unnecessary errorstring generation s390/dasd: Move allocation error message to DBF s390/dasd: Remove unused message logging macros s390/dasd: Use dev_err() over printk()
s390/dasd: Remove %p format specifier from error messages s390/dasd: Remove PRINTK_HEADER and KMSG_COMPONENT definitions s390/dasd: Use dev_*() for device log messages s390/dasd: Improve ERP error messages

Miroslav Franc (1):
s390/dasd: fix double module refcount decrement

 drivers/s390/block/dasd.c          | 106 +++++++------------
 drivers/s390/block/dasd_3990_erp.c |  80 ++++-----------
 drivers/s390/block/dasd_alias.c    |   8 --
 drivers/s390/block/dasd_devmap.c   |  34 +++----
 drivers/s390/block/dasd_diag.c     |   4 -
 drivers/s390/block/dasd_eckd.c     | 157 +++++++++++------------------
 drivers/s390/block/dasd_eer.c      |   7 --
 drivers/s390/block/dasd_erp.c      |   9 +-
 drivers/s390/block/dasd_fba.c      |  55 ++++------
 drivers/s390/block/dasd_genhd.c    |   5 -
 drivers/s390/block/dasd_int.h      |  29 ------
 drivers/s390/block/dasd_ioctl.c    |   6 --
 drivers/s390/block/dasd_proc.c     |   5 -
 13 files changed, 154 insertions(+), 351 deletions(-)
Signature · 1 line
2.40.1

Read the cover letter in its thread