[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Xen-devel] [Qemu-devel] [PATCH] chardev: Convert IOReadHandler to read an unsigned number of bytes
- To: Philippe Mathieu-Daudé <philmd@xxxxxxxxxx>, Prasad J Pandit <ppandit@xxxxxxxxxx>
- From: Paolo Bonzini <pbonzini@xxxxxxxxxx>
- Date: Thu, 11 Oct 2018 17:14:11 +0200
- Cc: Peter Maydell <peter.maydell@xxxxxxxxxx>, Li Zhijian <lizhijian@xxxxxxxxxxxxxx>, "Michael S. Tsirkin" <mst@xxxxxxxxxx>, Jan Kiszka <jan.kiszka@xxxxxxxxxxx>, Jason Wang <jasowang@xxxxxxxxxx>, Palmer Dabbelt <palmer@xxxxxxxxxx>, qemu-devel@xxxxxxxxxx, Alexander Graf <agraf@xxxxxxx>, Max Filippov <jcmvbkbc@xxxxxxxxx>, Michael Clark <mjc@xxxxxxxxxx>, Gerd Hoffmann <kraxel@xxxxxxxxxx>, "Edgar E. Iglesias" <edgar.iglesias@xxxxxxxxx>, Stefano Stabellini <sstabellini@xxxxxxxxxx>, Alberto Garcia <berto@xxxxxxxxxx>, Sagar Karandikar <sagark@xxxxxxxxxxxxxxxxx>, Markus Armbruster <armbru@xxxxxxxxxx>, Christian Borntraeger <borntraeger@xxxxxxxxxx>, Antony Pavlov <antonynpavlov@xxxxxxxxx>, Anthony Perard <anthony.perard@xxxxxxxxxx>, Marc-André Lureau <marcandre.lureau@xxxxxxxxxx>, Samuel Thibault <samuel.thibault@xxxxxxxxxxxx>, Laurent Vivier <lvivier@xxxxxxxxxx>, Corey Minyard <minyard@xxxxxxx>, Amit Shah <amit@xxxxxxxxxx>, Alistair Francis <alistair@xxxxxxxxxxxxx>, "Dr. David Alan Gilbert" <dgilbert@xxxxxxxxxx>, Fabien Chouteau <chouteau@xxxxxxxxxxx>, "open list:S390 Virtio-ccw" <qemu-s390x@xxxxxxxxxx>, "open list:PXA2XX" <qemu-arm@xxxxxxxxxx>, Peter Chubb <peter.chubb@xxxxxxxxxxxx>, Zhang Chen <zhangckid@xxxxxxxxx>, "open list:X86" <xen-devel@xxxxxxxxxxxxxxxxxxxx>, David Gibson <david@xxxxxxxxxxxxxxxxxxxxx>, Igor Mitsyanko <i.mitsyanko@xxxxxxxxx>, Bastian Koppelmann <kbastian@xxxxxxxxxxxxxxxxxxxxx>, Cornelia Huck <cohuck@xxxxxxxxxx>, Thomas Huth <huth@xxxxxxxxxxxxx>, Michael Walle <michael@xxxxxxxx>, "open list:sPAPR" <qemu-ppc@xxxxxxxxxx>
- Delivery-date: Thu, 11 Oct 2018 15:14:28 +0000
- List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
On 11/10/2018 15:14, Philippe Mathieu-Daudé wrote:
> The number of bytes can not be negative nor zero.
>
> Fixed 2 format string:
> - hw/char/spapr_vty.c
> - hw/usb/ccid-card-passthru.c
>
> Signed-off-by: Philippe Mathieu-Daudé <philmd@xxxxxxxxxx>
> ---
> See: https://lists.gnu.org/archive/html/qemu-devel/2018-10/msg02212.html
Not that easy; you need to adjust qemu_chr_be_can_write,
qemu_chr_be_write_impl, qemu_chr_be_write (which is where the callbacks
are invoked).
On the other hand, fd_chr_read_poll is not an IOCanReadHandler, and this
patch therefore probably doesn't compile?
Paolo
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel
|
