[RESEND,00/11] s390/dasd: string handling cleanups

Message

Stefan Haberland Feb. 8, 2024, 4:42 p.m. UTC

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(-)

Comments

Jens Axboe Feb. 8, 2024, 5:12 p.m. UTC | #1

On Thu, 08 Feb 2024 17:42:39 +0100, Stefan Haberland wrote:
> 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.
> 
> [...]

Applied, thanks!

[01/11] s390/dasd: Simplify uid string generation
        commit: e5de34db1e95ef549236f9a2630d396a41c208a2
[02/11] s390/dasd: Use sysfs_emit() over sprintf()
        commit: 9c386d0f6e04fee1b4161e49f8e9f5756bcab04c
[03/11] s390/dasd: Remove unnecessary errorstring generation
        commit: 8d7ac904c90be7a1ed1aafa34953c40270c30bd0
[04/11] s390/dasd: Move allocation error message to DBF
        commit: 4849494f05994f411c9cc0504843c6adefd1f2de
[05/11] s390/dasd: Remove unused message logging macros
        commit: 32312cf229117b781bd02d93b0b11c8b47dfcc0a
[06/11] s390/dasd: Use dev_err() over printk()
        commit: 0b3644b475e2564abe26a916af8447beb7c4966b
[07/11] s390/dasd: Remove %p format specifier from error messages
        commit: 4ba6366dbb03c2a58d0e7cbe140942bcf715006c
[08/11] s390/dasd: Remove PRINTK_HEADER and KMSG_COMPONENT definitions
        commit: c6c6c69df6598aacf3921f26f89b12d5e321ea83
[09/11] s390/dasd: Use dev_*() for device log messages
        commit: 79ae56fc475869d636071f66d9e4ef2a3819eee6

Best regards,