Review comments:

...

I guess this is really a manifestation of the fact that smp_sign should
> not be used as a space-handle in the first place, so what we really
> should do is add:
>         struct smp_space {
>                 uint64_t        first;
>                 uint64_t        len;
>                 uint64_t        last;
>                 struct smp_sign sign;
>         }


Changes done:

Added a smp_signspace struct that contains a sign and has ranges.

-- 
Martin Blix Grydeland
Varnish Software AS

Attachment: 0002-Create-and-use-a-smp_signspace-structure-to-have-ran.patch
Description: Binary data

_______________________________________________
varnish-dev mailing list
[email protected]
https://www.varnish-cache.org/lists/mailman/listinfo/varnish-dev

Reply via email to