On Wed, Apr 12, 2017 at 01:05:10PM -0400, David Miller wrote:
> From: "Michael S. Tsirkin" <m...@redhat.com>
> Date: Wed, 12 Apr 2017 19:43:20 +0300
>
> > On Wed, Apr 12, 2017 at 05:08:24PM +0100, Stefan Hajnoczi wrote:
> >> +struct af_vsockmon_hdr {
> >> + __le64 src_cid;
> >> + __le64 dst_cid;
> >> + __le32 src_port;
> >> + __le32 dst_port;
> >> + __le16 op; /* enum af_vsockmon_op */
> >> + __le16 transport; /* enum af_vsockmon_transport */
> >> + __le16 len; /* Transport header length */
> >> +} __attribute__((packed));
> >
> > I'd add a 2 byte padding here and then you do not need to pack it.
>
> Indeed, use of the packed attribute is strongly discouraged.Thanks will fix. Stefan
signature.asc
Description: PGP signature
