Re: [PATCH 2/2] virt: vmgenid: add support for generation counter
From: Greg KH
Date: Wed Aug 03 2022 - 11:32:36 EST
On Wed, Aug 03, 2022 at 05:21:27PM +0200, bchalios@xxxxxxxxx wrote:
> + /* Backwards compatibility. If CTRA is not there we just don't expose
> + * the char device
Backwards compatibility with what?
> + */
> + ret = parse_vmgenid_address(device, "CTRA", &state->gen_cntr_addr);
> + if (ret)
> + return 0;
> +
> + state->next_counter = devm_memremap(&device->dev, state->gen_cntr_addr,
> + sizeof(u32), MEMREMAP_WB);
> + if (IS_ERR(state->next_counter))
> + return 0;
This too is an error, you can not return with "all is good", right?
Once you try to create this device because the address is present, you
can't just give up and succeed no matter what went wrong, that seems
incorrect.
thanks,
greg k-h