[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Embedded-pv-devel] [PATCH v14] This is the ABI for the two halves of a para-virtualized sound driver to communicate with each to other.
- To: Jan Beulich <JBeulich@xxxxxxxx>
- From: Oleksandr Andrushchenko <andr2000@xxxxxxxxx>
- Date: Tue, 29 Nov 2016 18:55:32 +0200
- Cc: lars.kurth@xxxxxxxxxx, iurii.konovalenko@xxxxxxxxxxxxxxx, konrad.wilk@xxxxxxxxxx, vlad.babchuk@xxxxxxxxx, tim@xxxxxxx, dario.faggioli@xxxxxxxxxx, ian.jackson@xxxxxxxxxxxxx, al1img@xxxxxxxxx, andrii.anisov@xxxxxxxxx, olekstysh@xxxxxxxxx, embedded-pv-devel@xxxxxxxxxxxxxxxxxxxx, julien.grall@xxxxxxx, david.vrabel@xxxxxxxxxx, xen-devel@xxxxxxxxxxxxxxxxxxxx, oleksandr.dmytryshyn@xxxxxxxxxxxxxxx, joculator@xxxxxxxxx
- Delivery-date: Tue, 29 Nov 2016 16:55:38 +0000
- List-id: <embedded-pv-devel.lists.xenproject.org>
On 11/29/2016 06:09 PM, Jan Beulich wrote:
On 29.11.16 at 16:24, <andr2000@xxxxxxxxx> wrote:
+struct xensnd_req {
+ uint16_t id;
+ uint8_t operation;
+ uint8_t stream_idx;
+ uint32_t reserved;
+ union {
+ struct xensnd_open_req open;
+ struct xensnd_rw_req rw;
+ uint8_t padding[24];
+ } op;
+};
+
+struct xensnd_resp {
+ uint16_t id;
+ uint8_t operation;
+ uint8_t stream_idx;
+ int8_t status;
+ uint8_t padding[27];
Other than in _req, this ought to be "reserved" imo.
IMO, this is a classic padding, why reserved?
Jan
_______________________________________________
Embedded-pv-devel mailing list
Embedded-pv-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/cgi-bin/mailman/listinfo/embedded-pv-devel
|