Re: [Xen-devel] [PATCH v4 RESEND] public/io/netif.h: make control ring hash protocol more general

[Paul Durrant <Paul.Durrant@xxxxxxxxxx>]

> -----Original Message-----
> From: David Vrabel [mailto:david.vrabel@xxxxxxxxxx]
> Sent: 03 March 2016 16:26
> To: Paul Durrant; xen-devel@xxxxxxxxxxxxxxxxxxxx
> Cc: Wei Liu; Ian Jackson; David Vrabel; Jan Beulich
> Subject: Re: [Xen-devel] [PATCH v4 RESEND] public/io/netif.h: make control
> ring hash protocol more general
> 
> On 26/02/16 16:52, Paul Durrant wrote:
> > This patch modified the control ring protocol (of which there is
> > not yet an implementation) to make it more general. Most of the
> > concepts are not limited to toeplitz hashing so it's best not to
> > make them unnecessarily specific.
> >
> > Apart from changing the names of various definitions and modifying
> > comments, this patch:
> >
> > - Adds a new control message type to select a hash algorithm.
> > - Adds a reference implementation of the toeplitz hash.
> > - Changes the 'toeplitz' extra info fragment into a 'hash' extra
> >   info fragment and replaces the octet of padding with the index of
> >   the algorithm that was used to create the hash value.
> > - Relaxes the restriction that the mapping table has to be
> >   power-of-2 sized.
> >
> > The patch also fixes a few spelling typos noticed along the way.
> 
> Reviewed-by: David Vrabel <david.vrabel@xxxxxxxxxx>
> 
> David

Thanks David :-)

Jan, are you happy that this can be committed now?

  Paul
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel