On 25/06/25 15:08, dan.j.williams@xxxxxxxxx wrote:
dan.j.williams@ wrote:
Gustavo A. R. Silva wrote:
[..]
I think it would be a pretty small and direct replacement:
TRAILING_OVERLAP(struct nd_cmd_pkg, pkg, nd_payload,
struct nd_intel_get_security_state cmd;
) nd_cmd = {
...
Yes, this works. Hopefully, maintainers will comment on this and let us
know what they prefer. :)
Hey Gustavo, apologies for the latency here. I think TRAILING_OVERLAP()
looks lovely for this if only because I can read that and have an idea
what it means vs wondering what this _offset_to_fam is about and needing
to read the comment.
If you can get me that patch on top of the TRAILING_OVERLAP() branch I
can test it out and ack it to let it do in through the KSPP tree.
Just to move this along, I gave this conversion a try and all looks good
here. So feel free to fold this in and add:
Acked-by: Dan Williams <dan.j.williams@xxxxxxxxx>
Tested-by: Dan Williams <dan.j.williams@xxxxxxxxx>
...and take it through the KSPP tree with the TRAILING_OVERLAP() merge.
Thank you, Dan! :)
-Gustavo
