* [PATCH] S390-DASD: Delete an error message for a failed memory allocation in two functions
@ 2017-05-16 13:36 ` SF Markus Elfring
0 siblings, 0 replies; 2+ messages in thread
From: SF Markus Elfring @ 2017-05-16 13:36 UTC (permalink / raw)
To: linux-s390, Heiko Carstens, Jan Höppner, Martin Schwidefsky,
Stefan Haberland
Cc: LKML, kernel-janitors, Wolfram Sang
From: Markus Elfring <elfring@users.sourceforge.net>
Date: Tue, 16 May 2017 15:30:44 +0200
Omit an extra message for a memory allocation failure in this function.
This issue was detected by using the Coccinelle software.
Link: http://events.linuxfoundation.org/sites/events/files/slides/LCJ16-Refactor_Strings-WSang_0.pdf
Signed-off-by: Markus Elfring <elfring@users.sourceforge.net>
---
drivers/s390/block/dasd_eckd.c | 7 ++-----
drivers/s390/block/dasd_fba.c | 7 ++-----
2 files changed, 4 insertions(+), 10 deletions(-)
diff --git a/drivers/s390/block/dasd_eckd.c b/drivers/s390/block/dasd_eckd.c
index 122456e4db89..99c73decae49 100644
--- a/drivers/s390/block/dasd_eckd.c
+++ b/drivers/s390/block/dasd_eckd.c
@@ -1721,12 +1721,9 @@ dasd_eckd_check_characteristics(struct dasd_device *device)
}
if (!private) {
private = kzalloc(sizeof(*private), GFP_KERNEL | GFP_DMA);
- if (!private) {
- dev_warn(&device->cdev->dev,
- "Allocating memory for private DASD data "
- "failed\n");
+ if (!private)
return -ENOMEM;
- }
+
device->private = private;
} else {
memset(private, 0, sizeof(*private));
diff --git a/drivers/s390/block/dasd_fba.c b/drivers/s390/block/dasd_fba.c
index 462cab5d4302..2b2f9b0da5c2 100644
--- a/drivers/s390/block/dasd_fba.c
+++ b/drivers/s390/block/dasd_fba.c
@@ -132,12 +132,9 @@ dasd_fba_check_characteristics(struct dasd_device *device)
if (!private) {
private = kzalloc(sizeof(*private), GFP_KERNEL | GFP_DMA);
- if (!private) {
- dev_warn(&device->cdev->dev,
- "Allocating memory for private DASD "
- "data failed\n");
+ if (!private)
return -ENOMEM;
- }
+
device->private = private;
} else {
memset(private, 0, sizeof(*private));
--
2.13.0
^ permalink raw reply related [flat|nested] 2+ messages in thread
* [PATCH] S390-DASD: Delete an error message for a failed memory allocation in two functions
@ 2017-05-16 13:36 ` SF Markus Elfring
0 siblings, 0 replies; 2+ messages in thread
From: SF Markus Elfring @ 2017-05-16 13:36 UTC (permalink / raw)
To: linux-s390, Heiko Carstens, Jan Höppner, Martin Schwidefsky,
Stefan Haberland
Cc: LKML, kernel-janitors, Wolfram Sang
From: Markus Elfring <elfring@users.sourceforge.net>
Date: Tue, 16 May 2017 15:30:44 +0200
Omit an extra message for a memory allocation failure in this function.
This issue was detected by using the Coccinelle software.
Link: http://events.linuxfoundation.org/sites/events/files/slides/LCJ16-Refactor_Strings-WSang_0.pdf
Signed-off-by: Markus Elfring <elfring@users.sourceforge.net>
---
drivers/s390/block/dasd_eckd.c | 7 ++-----
drivers/s390/block/dasd_fba.c | 7 ++-----
2 files changed, 4 insertions(+), 10 deletions(-)
diff --git a/drivers/s390/block/dasd_eckd.c b/drivers/s390/block/dasd_eckd.c
index 122456e4db89..99c73decae49 100644
--- a/drivers/s390/block/dasd_eckd.c
+++ b/drivers/s390/block/dasd_eckd.c
@@ -1721,12 +1721,9 @@ dasd_eckd_check_characteristics(struct dasd_device *device)
}
if (!private) {
private = kzalloc(sizeof(*private), GFP_KERNEL | GFP_DMA);
- if (!private) {
- dev_warn(&device->cdev->dev,
- "Allocating memory for private DASD data "
- "failed\n");
+ if (!private)
return -ENOMEM;
- }
+
device->private = private;
} else {
memset(private, 0, sizeof(*private));
diff --git a/drivers/s390/block/dasd_fba.c b/drivers/s390/block/dasd_fba.c
index 462cab5d4302..2b2f9b0da5c2 100644
--- a/drivers/s390/block/dasd_fba.c
+++ b/drivers/s390/block/dasd_fba.c
@@ -132,12 +132,9 @@ dasd_fba_check_characteristics(struct dasd_device *device)
if (!private) {
private = kzalloc(sizeof(*private), GFP_KERNEL | GFP_DMA);
- if (!private) {
- dev_warn(&device->cdev->dev,
- "Allocating memory for private DASD "
- "data failed\n");
+ if (!private)
return -ENOMEM;
- }
+
device->private = private;
} else {
memset(private, 0, sizeof(*private));
--
2.13.0
^ permalink raw reply related [flat|nested] 2+ messages in thread
end of thread, other threads:[~2017-05-16 13:36 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-05-16 13:36 [PATCH] S390-DASD: Delete an error message for a failed memory allocation in two functions SF Markus Elfring
2017-05-16 13:36 ` SF Markus Elfring
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.