qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Alon Levy <alevy@redhat.com>
To: qemu-devel@nongnu.org
Cc: Blue Swirl <blauwirbel@gmail.com>
Subject: Re: [Qemu-devel] [PATCH 0/6] usb-ccid (v8)
Date: Tue, 7 Dec 2010 12:20:20 +0200	[thread overview]
Message-ID: <20101207102019.GC31414@playa.tlv.redhat.com> (raw)
In-Reply-To: <1290955607-25821-1-git-send-email-alevy@redhat.com>

ping.

Blue Swirl - one patch I forgot is in a later message titled "..v8.1.."
with the removal of the libcaccard build.

On Sun, Nov 28, 2010 at 04:46:41PM +0200, Alon Levy wrote:
> This patchset adds three new devices, usb-ccid, ccid-card-passthru and
> ccid-card-emulated, providing a CCID bus, a simple passthru protocol
> implementing card requiring a client, and a standalone emulated card.
> 
> It also introduces a new directory libcaccard with CAC card emulation,
> CAC is a type of ISO 7816 smart card.
> 
> v7-v8 changes:
>  * Blue Swirl comments:
>   * usb-ccid: deannonymize some structs
>   * usb-ccid: coding style change - answer_t and bulk_in_t fixed
>   * usb-ccid: handle endianess conversion between guest and host
>  * usb-ccid: s/ccid_bulk_in_copy_out/ccid_bulk_in_copy_to_guest/
>  * ccid-card-emulated: fix segfault if backend not specified
>  * ccid-card-emulated: let last reader inserted win
>  * libcaccard: remove double vscard_common.h
> 
> v6->v7 changes:
>  * external libcaccard became internal directory libcaccard
>   * statically link object files into qemu
>   * produce libcaccard.so for usage by external projects
>   * applied coding style to new code (please check me)
>   - did not use the qemu options parsing for libcaccard, since
>    it seems to draw large amounts of qemu code (monitor for instance).
> 
> v5->v6 changes:
>  * really remove static debug (I apologize for claiming to have done so before)
> 
> v4->v5 changes:
>  * rebased to latest
>  * remove static debug in card devices
>  * fix --enable-smartcard to link
>  * stall instead of assert when exceeding BULK_OUT_DATA_SIZE
>  * make ccid_reserve_recv_buf for too large len discard message, not exit
>  * make ccid_reserve_recv_buf return void*
>  * fix typo
>  * remove commented code in VMState
> 
> v3->v4:
>  * remove ccid field in CCIDBus
>  * remove static debug in bus
>  * add back docs
> 
> v2->v3:
>  * split into bus (usb-ccid.c, uses ccid.h) and card (ccid-card-passthru.c).
>  * removed documentation (being revised).
> 
> v1->v2:
>  * all QSIMPLEQ turned into fixed sized rings
>  * all allocated buffers turned into fixed size buffers
>  * added migration support
>  * added a message to tell client qemu has migrated to ip:port
>   * for lack of monitor commands ip:port are 0:0, which causes the updated
>    vscclient to connect to one port higher on the same host. will add monitor
>    commands in a separate patch. tested with current setup.
> 
> Alon Levy (5):
>   usb-ccid: add CCID bus
>   ccid: add passthru card device
>   libcaccard: update configure to build and use internal libcaccard
>   ccid: add ccid-card-emulated device (v2)
>   ccid: add docs
> 
> Robert Relyea (1):
>   libcaccard: initial commit after coding style fixes
> 
>  Makefile                     |    6 +-
>  Makefile.objs                |    6 +
>  Makefile.target              |    2 +
>  configure                    |   36 ++
>  docs/ccid.txt                |  125 ++++
>  docs/libcaccard.txt          |  482 +++++++++++++++
>  hw/ccid-card-emulated.c      |  501 ++++++++++++++++
>  hw/ccid-card-passthru.c      |  277 +++++++++
>  hw/ccid.h                    |   34 ++
>  hw/usb-ccid.c                | 1345 ++++++++++++++++++++++++++++++++++++++++++
>  libcaccard/Makefile          |   18 +
>  libcaccard/cac.c             |  411 +++++++++++++
>  libcaccard/cac.h             |   20 +
>  libcaccard/card_7816.c       |  780 ++++++++++++++++++++++++
>  libcaccard/card_7816.h       |   60 ++
>  libcaccard/card_7816t.h      |  163 +++++
>  libcaccard/config.h          |   81 +++
>  libcaccard/event.c           |  112 ++++
>  libcaccard/eventt.h          |   28 +
>  libcaccard/link_test.c       |   20 +
>  libcaccard/mutex.h           |   59 ++
>  libcaccard/passthru.c        |  608 +++++++++++++++++++
>  libcaccard/passthru.h        |   50 ++
>  libcaccard/vcard.c           |  350 +++++++++++
>  libcaccard/vcard.h           |   85 +++
>  libcaccard/vcard_emul.h      |   59 ++
>  libcaccard/vcard_emul_nss.c  | 1147 +++++++++++++++++++++++++++++++++++
>  libcaccard/vcard_emul_type.c |   60 ++
>  libcaccard/vcard_emul_type.h |   29 +
>  libcaccard/vcardt.h          |   66 ++
>  libcaccard/vevent.h          |   26 +
>  libcaccard/vreader.c         |  515 ++++++++++++++++
>  libcaccard/vreader.h         |   53 ++
>  libcaccard/vreadert.h        |   23 +
>  libcaccard/vscard_common.h   |  130 ++++
>  libcaccard/vscclient.c       |  710 ++++++++++++++++++++++
>  36 files changed, 8475 insertions(+), 2 deletions(-)
>  create mode 100644 docs/ccid.txt
>  create mode 100644 docs/libcaccard.txt
>  create mode 100644 hw/ccid-card-emulated.c
>  create mode 100644 hw/ccid-card-passthru.c
>  create mode 100644 hw/ccid.h
>  create mode 100644 hw/usb-ccid.c
>  create mode 100644 libcaccard/Makefile
>  create mode 100644 libcaccard/cac.c
>  create mode 100644 libcaccard/cac.h
>  create mode 100644 libcaccard/card_7816.c
>  create mode 100644 libcaccard/card_7816.h
>  create mode 100644 libcaccard/card_7816t.h
>  create mode 100644 libcaccard/config.h
>  create mode 100644 libcaccard/event.c
>  create mode 100644 libcaccard/eventt.h
>  create mode 100644 libcaccard/link_test.c
>  create mode 100644 libcaccard/mutex.h
>  create mode 100644 libcaccard/passthru.c
>  create mode 100644 libcaccard/passthru.h
>  create mode 100644 libcaccard/vcard.c
>  create mode 100644 libcaccard/vcard.h
>  create mode 100644 libcaccard/vcard_emul.h
>  create mode 100644 libcaccard/vcard_emul_nss.c
>  create mode 100644 libcaccard/vcard_emul_type.c
>  create mode 100644 libcaccard/vcard_emul_type.h
>  create mode 100644 libcaccard/vcardt.h
>  create mode 100644 libcaccard/vevent.h
>  create mode 100644 libcaccard/vreader.c
>  create mode 100644 libcaccard/vreader.h
>  create mode 100644 libcaccard/vreadert.h
>  create mode 100644 libcaccard/vscard_common.h
>  create mode 100644 libcaccard/vscclient.c
> 
> -- 
> 1.7.3.2
> 
> 

  parent reply	other threads:[~2010-12-07 10:24 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-11-28 14:46 [Qemu-devel] [PATCH 0/6] usb-ccid (v8) Alon Levy
2010-11-28 14:46 ` [Qemu-devel] [PATCH 1/6] usb-ccid: add CCID bus Alon Levy
2010-11-28 14:46 ` [Qemu-devel] [PATCH 2/6] ccid: add passthru card device Alon Levy
2010-11-28 14:46 ` [Qemu-devel] [PATCH 3/6] libcaccard: initial commit after coding style fixes Alon Levy
2010-11-28 14:46 ` [Qemu-devel] [PATCH 4/6] libcaccard: update configure to build and use internal libcaccard Alon Levy
2010-11-28 14:46 ` [Qemu-devel] [PATCH 5/6] ccid: add ccid-card-emulated device (v2) Alon Levy
2010-11-28 14:46 ` [Qemu-devel] [PATCH 6/6] ccid: add docs Alon Levy
2010-12-07 10:20 ` Alon Levy [this message]
2010-12-11  9:43   ` [Qemu-devel] [PATCH 0/6] usb-ccid (v8) Blue Swirl
2010-12-11 15:33     ` Hans de Goede
2010-12-11 15:55       ` Blue Swirl
2010-12-12 11:35         ` Alon Levy
2010-12-12 15:07           ` Blue Swirl
2010-12-12 16:24             ` Alon Levy

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=20101207102019.GC31414@playa.tlv.redhat.com \
    --to=alevy@redhat.com \
    --cc=blauwirbel@gmail.com \
    --cc=qemu-devel@nongnu.org \
    /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).