On 13.06.2023 18:52, Andrew Cooper wrote:
> @@ -1062,9 +1081,13 @@ map_grant_ref(
> ((act->domid != ld->domain_id) ||
> (act->pin & GNTPIN_incr2oflow_mask(pin_incr)) ||
> (act->is_sub_page)) )
> - PIN_FAIL(act_release_out, GNTST_general_error,
> + {
> + gdprintk(XENLOG_WARNING,
> "Bad domain (%d != %d), or risk of counter overflow %08x,
> or subpage %d\n",
> act->domid, ld->domain_id, act->pin, act->is_sub_page);
> + rc = GNTST_general_error;
> + goto act_release_out;
Nit: One too deep indentation on these last two lines above. With that
adjusted:
Reviewed-by: Jan Beulich <[email protected]>
Jan
