[PATCH v2 for 6.5.0] domain_conf: Remove zPCI validation from formatter
Andrea Bolognani
abologna at redhat.com
Mon Jun 29 11:07:33 UTC 2020
On Mon, 2020-06-29 at 12:57 +0200, Michal Privoznik wrote:
> In 076591009ad a validation code was added to
> virDomainDeviceInfoFormat() which reports an error if zPCI
> address entered in was incomplete. But, there are two problems
> with this approach.
>
> The first problem is the placement of the code - it doesn't
> belong into XML formatter rather than XML validator.
>
> The second one is that at the point of formatting XML the post
> parse callback has run and thus filled in required info.
> Therefore this check can never do something useful and instead of
> moving it into validator, it's removed completely.
>
> Signed-off-by: Michal Privoznik <mprivozn at redhat.com>
> ---
>
> This is v2 of:
>
> https://www.redhat.com/archives/libvir-list/2020-June/msg01279.html
>
> diff to v1:
> - Instead of moving the check to validator remove it completely (per
> discussion in v1)
>
> src/conf/domain_conf.c | 4 ----
> 1 file changed, 4 deletions(-)
Thanks!
Reviewed-by: Andrea Bolognani <abologna at redhat.com>
and safe for freeze.
--
Andrea Bolognani / Red Hat / Virtualization
More information about the libvir-list
mailing list