[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 04/11] usb-ccid: convert CCIDCardClass::init ->
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [Qemu-devel] [PATCH 04/11] usb-ccid: convert CCIDCardClass::init -> realize |
Date: |
Thu, 18 Jan 2018 14:05:42 -0300 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.5.2 |
On 01/16/2018 10:15 AM, Philippe Mathieu-Daudé wrote:
> Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
> ---
> hw/usb/ccid.h | 9 +++++--
> hw/usb/ccid-card-emulated.c | 42 ++++++++++++++++---------------
> hw/usb/ccid-card-passthru.c | 10 ++++----
> hw/usb/dev-smartcard-reader.c | 57
> +++++++++++++++----------------------------
> 4 files changed, 54 insertions(+), 64 deletions(-)
>
> diff --git a/hw/usb/ccid.h b/hw/usb/ccid.h
> index 1f070116d6..11056b91ee 100644
> --- a/hw/usb/ccid.h
> +++ b/hw/usb/ccid.h
> @@ -28,20 +28,25 @@ typedef struct CCIDCardInfo CCIDCardInfo;
> * into the smartcard device (hw/ccid-card-*.c)
> */
> typedef struct CCIDCardClass {
> + /*< private >*/
> DeviceClass parent_class;
> + /*< public >*/
> +
> + void (*realize)(CCIDCardState *card, Error **errp);
> + void (*unrealize)(CCIDCardState *card, Error **errp);
> const uint8_t *(*get_atr)(CCIDCardState *card, uint32_t *len);
> void (*apdu_from_guest)(CCIDCardState *card,
> const uint8_t *apdu,
> uint32_t len);
> - void (*exitfn)(CCIDCardState *card);
> - int (*initfn)(CCIDCardState *card);
> } CCIDCardClass;
>
> /*
> * state of the CCID Card device (i.e. hw/ccid-card-*.c)
> */
> struct CCIDCardState {
> + /*< private >*/
> DeviceState qdev;
> + /*< public >*/
> uint32_t slot; /* For future use with multiple slot reader. */
> };
>
> diff --git a/hw/usb/ccid-card-emulated.c b/hw/usb/ccid-card-emulated.c
> index e646eb243b..8a512e73df 100644
> --- a/hw/usb/ccid-card-emulated.c
> +++ b/hw/usb/ccid-card-emulated.c
> @@ -27,6 +27,7 @@
> */
>
> #include "qemu/osdep.h"
> +#include "qapi/error.h"
> #include <eventt.h>
> #include <vevent.h>
> #include <vreader.h>
> @@ -480,7 +481,7 @@ static uint32_t parse_enumeration(char *str,
> return ret;
> }
>
> -static int emulated_initfn(CCIDCardState *base)
> +static void emulated_realize(CCIDCardState *base, Error **errp)
> {
> EmulatedState *card = EMULATED_CCID_CARD(base);
> VCardEmulError ret;
> @@ -495,7 +496,8 @@ static int emulated_initfn(CCIDCardState *base)
> card->reader = NULL;
> card->quit_apdu_thread = 0;
> if (init_event_notifier(card) < 0) {
> - return -1;
> + error_setg(errp, TYPE_EMULATED_CCID ": event notifier creation
> failed");
> + return;
> }
>
> card->backend = 0;
> @@ -505,11 +507,12 @@ static int emulated_initfn(CCIDCardState *base)
> }
>
> if (card->backend == 0) {
> - printf("backend must be one of:\n");
> + error_setg(errp, TYPE_EMULATED_CCID ": no backend specified.");
> + printf("backend must be one of:\n"); /* TODO remove */
> for (ptable = backend_enum_table; ptable->name != NULL; ++ptable) {
> printf("%s\n", ptable->name);
Oops I missed this printf(), now reading "qapi/error.h" I guess I have
to use error_append_hint() here.
> }
> - return -1;
> + return;
> }
>
> /* TODO: a passthru backened that works on local machine. third card
> type?*/
> @@ -517,37 +520,36 @@ static int emulated_initfn(CCIDCardState *base)
> if (card->cert1 != NULL && card->cert2 != NULL && card->cert3 !=
> NULL) {
> ret = emulated_initialize_vcard_from_certificates(card);
> } else {
> - printf("%s: you must provide all three certs for"
> - " certificates backend\n", TYPE_EMULATED_CCID);
> - return -1;
> + error_setg(errp, TYPE_EMULATED_CCID ": you must provide all
> three "
> + "certs for certificates backend");
> + return;
> }
> } else {
> if (card->backend != BACKEND_NSS_EMULATED) {
> - printf("%s: bad backend specified. The options are:\n%s
> (default),"
> - " %s.\n", TYPE_EMULATED_CCID, BACKEND_NSS_EMULATED_NAME,
> - BACKEND_CERTIFICATES_NAME);
> - return -1;
> + error_setg(errp, TYPE_EMULATED_CCID ": bad backend specified. "
> + "The options are: %s (default), %s.",
> + BACKEND_NSS_EMULATED_NAME, BACKEND_CERTIFICATES_NAME);
same here.
> + return;
> }
> if (card->cert1 != NULL || card->cert2 != NULL || card->cert3 !=
> NULL) {
> - printf("%s: unexpected cert parameters to nss emulated
> backend\n",
> - TYPE_EMULATED_CCID);
> - return -1;
> + error_setg(errp, TYPE_EMULATED_CCID ": unexpected cert
> parameters "
> + "to nss emulated backend");
> + return;
> }
> /* default to mirroring the local hardware readers */
> ret = wrap_vcard_emul_init(NULL);
> }
> if (ret != VCARD_EMUL_OK) {
> - printf("%s: failed to initialize vcard\n", TYPE_EMULATED_CCID);
> - return -1;
> + error_setg(errp, TYPE_EMULATED_CCID ": failed to initialize vcard");
> + return;
> }
> qemu_thread_create(&card->event_thread_id, "ccid/event", event_thread,
> card, QEMU_THREAD_JOINABLE);
> qemu_thread_create(&card->apdu_thread_id, "ccid/apdu",
> handle_apdu_thread,
> card, QEMU_THREAD_JOINABLE);
> - return 0;
> }
>
> -static void emulated_exitfn(CCIDCardState *base)
> +static void emulated_unrealize(CCIDCardState *base, Error **errp)
> {
> EmulatedState *card = EMULATED_CCID_CARD(base);
> VEvent *vevent = vevent_new(VEVENT_LAST, NULL, NULL);
> @@ -581,8 +583,8 @@ static void emulated_class_initfn(ObjectClass *klass,
> void *data)
> DeviceClass *dc = DEVICE_CLASS(klass);
> CCIDCardClass *cc = CCID_CARD_CLASS(klass);
>
> - cc->initfn = emulated_initfn;
> - cc->exitfn = emulated_exitfn;
> + cc->realize = emulated_realize;
> + cc->unrealize = emulated_unrealize;
> cc->get_atr = emulated_get_atr;
> cc->apdu_from_guest = emulated_apdu_from_guest;
> set_bit(DEVICE_CATEGORY_INPUT, dc->categories);
> diff --git a/hw/usb/ccid-card-passthru.c b/hw/usb/ccid-card-passthru.c
> index 117711862e..844ad2f878 100644
> --- a/hw/usb/ccid-card-passthru.c
> +++ b/hw/usb/ccid-card-passthru.c
> @@ -9,6 +9,7 @@
> */
>
> #include "qemu/osdep.h"
> +#include "qapi/error.h"
> #include <cacard/vscard_common.h>
> #include "chardev/char-fe.h"
> #include "qemu/error-report.h"
> @@ -337,7 +338,7 @@ static const uint8_t *passthru_get_atr(CCIDCardState
> *base, uint32_t *len)
> return card->atr;
> }
>
> -static int passthru_initfn(CCIDCardState *base)
> +static void passthru_realize(CCIDCardState *base, Error **errp)
> {
> PassthruState *card = PASSTHRU_CCID_CARD(base);
>
> @@ -351,15 +352,14 @@ static int passthru_initfn(CCIDCardState *base)
> ccid_card_vscard_event, NULL, card, NULL, true);
> ccid_card_vscard_send_init(card);
> } else {
> - error_report("missing chardev");
> - return -1;
> + error_setg(errp, "missing chardev");
> + return;
> }
> card->debug = parse_debug_env("QEMU_CCID_PASSTHRU_DEBUG", D_VERBOSE,
> card->debug);
> assert(sizeof(DEFAULT_ATR) <= MAX_ATR_SIZE);
> memcpy(card->atr, DEFAULT_ATR, sizeof(DEFAULT_ATR));
> card->atr_length = sizeof(DEFAULT_ATR);
> - return 0;
> }
>
> static VMStateDescription passthru_vmstate = {
> @@ -387,7 +387,7 @@ static void passthru_class_initfn(ObjectClass *klass,
> void *data)
> DeviceClass *dc = DEVICE_CLASS(klass);
> CCIDCardClass *cc = CCID_CARD_CLASS(klass);
>
> - cc->initfn = passthru_initfn;
> + cc->realize = passthru_realize;
> cc->get_atr = passthru_get_atr;
> cc->apdu_from_guest = passthru_apdu_from_guest;
> set_bit(DEVICE_CATEGORY_INPUT, dc->categories);
> diff --git a/hw/usb/dev-smartcard-reader.c b/hw/usb/dev-smartcard-reader.c
> index e334d3be11..e36b190118 100644
> --- a/hw/usb/dev-smartcard-reader.c
> +++ b/hw/usb/dev-smartcard-reader.c
> @@ -37,7 +37,6 @@
> #include "qemu/osdep.h"
> #include "qapi/error.h"
> #include "qemu-common.h"
> -#include "qemu/error-report.h"
> #include "hw/usb.h"
> #include "hw/usb/desc.h"
>
> @@ -500,26 +499,6 @@ static void ccid_card_apdu_from_guest(CCIDCardState
> *card,
> }
> }
>
> -static void ccid_card_exitfn(CCIDCardState *card)
> -{
> - CCIDCardClass *cc = CCID_CARD_GET_CLASS(card);
> -
> - if (cc->exitfn) {
> - cc->exitfn(card);
> - }
> -
> -}
> -
> -static int ccid_card_initfn(CCIDCardState *card)
> -{
> - CCIDCardClass *cc = CCID_CARD_GET_CLASS(card);
> -
> - if (cc->initfn) {
> - return cc->initfn(card);
> - }
> - return 0;
> -}
> -
> static bool ccid_has_pending_answers(USBCCIDState *s)
> {
> return s->pending_answers_num > 0;
> @@ -1281,41 +1260,45 @@ void ccid_card_card_inserted(CCIDCardState *card)
> ccid_on_slot_change(s, true);
> }
>
> -static int ccid_card_exit(DeviceState *qdev)
> +static void ccid_card_unrealize(DeviceState *qdev, Error **errp)
> {
> CCIDCardState *card = CCID_CARD(qdev);
> + CCIDCardClass *cc = CCID_CARD_GET_CLASS(card);
> USBDevice *dev = USB_DEVICE(qdev->parent_bus->parent);
> USBCCIDState *s = USB_CCID_DEV(dev);
>
> if (ccid_card_inserted(s)) {
> ccid_card_card_removed(card);
> }
> - ccid_card_exitfn(card);
> + if (cc->unrealize) {
> + cc->unrealize(card, errp);
> + }
> s->card = NULL;
> - return 0;
> }
>
> -static int ccid_card_init(DeviceState *qdev)
> +static void ccid_card_realize(DeviceState *qdev, Error **errp)
> {
> CCIDCardState *card = CCID_CARD(qdev);
> + CCIDCardClass *cc = CCID_CARD_GET_CLASS(card);
> USBDevice *dev = USB_DEVICE(qdev->parent_bus->parent);
> USBCCIDState *s = USB_CCID_DEV(dev);
> - int ret = 0;
>
> if (card->slot != 0) {
> - warn_report("usb-ccid supports one slot, can't add %d",
> - card->slot);
> - return -1;
> + error_setg(errp, "usb-ccid supports one slot, can't add %d",
> + card->slot);
> + return;
> }
> if (s->card != NULL) {
> - warn_report("usb-ccid card already full, not adding");
> - return -1;
> + error_setg(errp, "usb-ccid card already full, not adding");
> + return;
> }
> - ret = ccid_card_initfn(card);
> - if (ret == 0) {
> - s->card = card;
> + if (cc->realize) {
> + cc->realize(card, errp);
> + if (errp && *errp) {
> + return;
> + }
> }
> - return ret;
> + s->card = card;
> }
>
> static void ccid_realize(USBDevice *dev, Error **errp)
> @@ -1477,8 +1460,8 @@ static void ccid_card_class_init(ObjectClass *klass,
> void *data)
> {
> DeviceClass *k = DEVICE_CLASS(klass);
> k->bus_type = TYPE_CCID_BUS;
> - k->init = ccid_card_init;
> - k->exit = ccid_card_exit;
> + k->realize = ccid_card_realize;
> + k->unrealize = ccid_card_unrealize;
> k->props = ccid_props;
> }
>
>
- Re: [Qemu-devel] [PATCH 03/11] hw/i2c: convert I2CSlaveClass::init -> realize, (continued)
- [Qemu-devel] [PATCH 04/11] usb-ccid: convert CCIDCardClass::init -> realize, Philippe Mathieu-Daudé, 2018/01/16
- Re: [Qemu-devel] [PATCH 04/11] usb-ccid: convert CCIDCardClass::init -> realize,
Philippe Mathieu-Daudé <=
- [Qemu-devel] [PATCH 06/11] sysbus: add realize() and unrealize(), Philippe Mathieu-Daudé, 2018/01/16
- [Qemu-devel] [PATCH 07/11] qdev: simplify the SysBusDeviceClass::init path, Philippe Mathieu-Daudé, 2018/01/16
- [Qemu-devel] [PATCH 08/11] qdev: remove DeviceClass::init, Philippe Mathieu-Daudé, 2018/01/16
- [Qemu-devel] [PATCH 09/11] qdev: remove DeviceClass::exit, Philippe Mathieu-Daudé, 2018/01/16
- [Qemu-devel] [PATCH 10/11] qdev: remove empty realize/unrealize stubs, Philippe Mathieu-Daudé, 2018/01/16
- [Qemu-devel] [PATCH 11/11] qdev: rename typedef qdev_resetfn() -> DeviceReset(), Philippe Mathieu-Daudé, 2018/01/16
- Re: [Qemu-devel] [PATCH 00/11] qdev: remove DeviceClass::init/exit(), no-reply, 2018/01/17