From: Cornelia Huck <cohuck@redhat.com>
To: peter.maydell@linaro.org
Cc: qemu-s390x@nongnu.org, qemu-devel@nongnu.org,
borntraeger@de.ibm.com, rth@twiddle.net, agraf@suse.de,
david@redhat.com, thuth@redhat.com,
Nia Alarie <nia.alarie@gmail.com>,
Cornelia Huck <cohuck@redhat.com>
Subject: [Qemu-devel] [PULL 06/11] s390x/sclpconsole: Remove dead code - remove exit handlers
Date: Thu, 8 Mar 2018 19:04:18 +0100 [thread overview]
Message-ID: <20180308180423.941-7-cohuck@redhat.com> (raw)
In-Reply-To: <20180308180423.941-1-cohuck@redhat.com>
From: Nia Alarie <nia.alarie@gmail.com>
The other event handlers (quiesce and cpu) do not define these
handlers, and this one does nothing, so it can be removed.
Signed-off-by: Nia Alarie <nia.alarie@gmail.com>
Reviewed-by: Christian Borntraeger <borntraeger@de.ibm.com>
Message-Id: <20180306100721.19419-1-nia.alarie@gmail.com>
Signed-off-by: Cornelia Huck <cohuck@redhat.com>
---
hw/char/sclpconsole-lm.c | 6 ------
hw/char/sclpconsole.c | 6 ------
hw/s390x/event-facility.c | 14 --------------
include/hw/s390x/event-facility.h | 1 -
4 files changed, 27 deletions(-)
diff --git a/hw/char/sclpconsole-lm.c b/hw/char/sclpconsole-lm.c
index c500bdaf29..84efd8df73 100644
--- a/hw/char/sclpconsole-lm.c
+++ b/hw/char/sclpconsole-lm.c
@@ -318,11 +318,6 @@ static int console_init(SCLPEvent *event)
return 0;
}
-static int console_exit(SCLPEvent *event)
-{
- return 0;
-}
-
static void console_reset(DeviceState *dev)
{
SCLPEvent *event = SCLP_EVENT(dev);
@@ -349,7 +344,6 @@ static void console_class_init(ObjectClass *klass, void *data)
dc->reset = console_reset;
dc->vmsd = &vmstate_sclplmconsole;
ec->init = console_init;
- ec->exit = console_exit;
ec->get_send_mask = send_mask;
ec->get_receive_mask = receive_mask;
ec->can_handle_event = can_handle_event;
diff --git a/hw/char/sclpconsole.c b/hw/char/sclpconsole.c
index d0265dfa7a..e6ba0a46bd 100644
--- a/hw/char/sclpconsole.c
+++ b/hw/char/sclpconsole.c
@@ -246,11 +246,6 @@ static void console_reset(DeviceState *dev)
scon->notify = false;
}
-static int console_exit(SCLPEvent *event)
-{
- return 0;
-}
-
static Property console_properties[] = {
DEFINE_PROP_CHR("chardev", SCLPConsole, chr),
DEFINE_PROP_END_OF_LIST(),
@@ -265,7 +260,6 @@ static void console_class_init(ObjectClass *klass, void *data)
dc->reset = console_reset;
dc->vmsd = &vmstate_sclpconsole;
ec->init = console_init;
- ec->exit = console_exit;
ec->get_send_mask = send_mask;
ec->get_receive_mask = receive_mask;
ec->can_handle_event = can_handle_event;
diff --git a/hw/s390x/event-facility.c b/hw/s390x/event-facility.c
index 155a69467b..79de3c738f 100644
--- a/hw/s390x/event-facility.c
+++ b/hw/s390x/event-facility.c
@@ -431,26 +431,12 @@ static void event_realize(DeviceState *qdev, Error **errp)
}
}
-static void event_unrealize(DeviceState *qdev, Error **errp)
-{
- SCLPEvent *event = SCLP_EVENT(qdev);
- SCLPEventClass *child = SCLP_EVENT_GET_CLASS(event);
- if (child->exit) {
- int rc = child->exit(event);
- if (rc < 0) {
- error_setg(errp, "SCLP event exit failed.");
- return;
- }
- }
-}
-
static void event_class_init(ObjectClass *klass, void *data)
{
DeviceClass *dc = DEVICE_CLASS(klass);
dc->bus_type = TYPE_SCLP_EVENTS_BUS;
dc->realize = event_realize;
- dc->unrealize = event_unrealize;
}
static const TypeInfo sclp_event_type_info = {
diff --git a/include/hw/s390x/event-facility.h b/include/hw/s390x/event-facility.h
index 5119b9b7f0..b913f85edc 100644
--- a/include/hw/s390x/event-facility.h
+++ b/include/hw/s390x/event-facility.h
@@ -174,7 +174,6 @@ typedef struct SCLPEvent {
typedef struct SCLPEventClass {
DeviceClass parent_class;
int (*init)(SCLPEvent *event);
- int (*exit)(SCLPEvent *event);
/* get SCLP's send mask */
unsigned int (*get_send_mask)(void);
--
2.13.6
next prev parent reply other threads:[~2018-03-08 18:04 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-03-08 18:04 [Qemu-devel] [PULL 00/11] s390x patches for the 2.12 softfreeze Cornelia Huck
2018-03-08 18:04 ` [Qemu-devel] [PULL 01/11] hw/s390x/ipl: Bail out if the network bootloader can not be found Cornelia Huck
2018-03-08 18:04 ` [Qemu-devel] [PULL 02/11] s390/ipl: only print boot menu error if -boot menu=on was specified Cornelia Huck
2018-03-08 18:04 ` [Qemu-devel] [PULL 03/11] target/s390x: Remove leading underscores from #defines Cornelia Huck
2018-03-08 18:04 ` [Qemu-devel] [PULL 04/11] hw/s390x: Add the possibility to specify the netboot image on the command line Cornelia Huck
2018-03-08 18:04 ` [Qemu-devel] [PULL 05/11] numa: we don't implement NUMA for s390x Cornelia Huck
2018-03-08 18:04 ` Cornelia Huck [this message]
2018-03-08 18:04 ` [Qemu-devel] [PULL 07/11] vfio-ccw: license text should indicate GPL v2 or later Cornelia Huck
2018-03-08 18:04 ` [Qemu-devel] [PULL 08/11] s390x/sclp: proper support of larger send and receive masks Cornelia Huck
2018-03-08 18:04 ` [Qemu-devel] [PULL 09/11] s390x/sclp: clean up sclp masks Cornelia Huck
2018-03-08 18:04 ` [Qemu-devel] [PULL 10/11] pc-bios/s390-ccw: Move string arrays from bootmap header to .c file Cornelia Huck
2018-03-08 18:04 ` [Qemu-devel] [PULL 11/11] s390x/virtio: Convert virtio-ccw from *_exit to *_unrealize Cornelia Huck
2018-03-09 10:36 ` [Qemu-devel] [PULL 00/11] s390x patches for the 2.12 softfreeze Peter Maydell
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20180308180423.941-7-cohuck@redhat.com \
--to=cohuck@redhat.com \
--cc=agraf@suse.de \
--cc=borntraeger@de.ibm.com \
--cc=david@redhat.com \
--cc=nia.alarie@gmail.com \
--cc=peter.maydell@linaro.org \
--cc=qemu-devel@nongnu.org \
--cc=qemu-s390x@nongnu.org \
--cc=rth@twiddle.net \
--cc=thuth@redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).