On 08.05.2025 15:20, Stewart Hildebrand wrote:
> --- a/xen/common/rangeset.c
> +++ b/xen/common/rangeset.c
> @@ -397,6 +397,18 @@ int rangeset_merge(struct rangeset *r1, struct rangeset
> *r2)
> return rangeset_report_ranges(r2, 0, ~0UL, merge, r1);
> }
>
> +static int cf_check subtract(unsigned long s, unsigned long e, void *data)
> +{
> + struct rangeset *r = data;
> +
> + return rangeset_remove_range(r, s, e);
> +}
> +
> +int rangeset_subtract(struct rangeset *r1, struct rangeset *r2)
> +{
> + return rangeset_report_ranges(r2, 0, ~0UL, subtract, r1);
> +}
I understand this was committed already, but I don't understand why: This
introduces a Misra rule 2.1 violation aiui. The rule isn't tagged as clean
yet, but it was accepted and hence I thought we would strive towards not
introducing new violations. What's the deal?
Jan
