On Wed, Sep 25, 2019 at 09:46:13AM +0100, Richard W.M. Jones wrote:
On Tue, Sep 24, 2019 at 05:24:34PM -0500, Eric Blake wrote:
> On 9/24/19 4:07 PM, Richard W.M. Jones wrote:
> > /* New-style handshake server reply when using NBD_OPT_EXPORT_NAME.
> > * Modern clients use NBD_OPT_GO instead of this.
> > @@ -167,7 +167,7 @@ struct nbd_new_handshake_finish {
> > uint64_t exportsize;
> > uint16_t eflags; /* per-export flags */
> > char zeroes[124]; /* must be sent as zero bytes */
> > -} __attribute__((packed));
> > +} NBD_ATTRIBUTE_PACKED;
>
> Redundant type.
Indeed, we don't actually use this anywhere (for obvious reasons of
course). As I was only synchronizing the two headers I didn't notice.
I'll remove it instead.
Oh I spoke too soon - we do use it in nbdkit.
Rich.
--
Richard Jones, Virtualization Group, Red Hat
http://people.redhat.com/~rjones
Read my programming and virtualization blog:
http://rwmj.wordpress.com
libguestfs lets you edit virtual machines. Supports shell scripting,
bindings from many languages.
http://libguestfs.org