[libvirt] [PATCH 2/2] qemu: Figure out nodeset bitmap size correctly

Peter Krempa pkrempa at redhat.com
Thu Apr 19 14:40:34 UTC 2018


On Thu, Apr 19, 2018 at 16:37:23 +0200, Andrea Bolognani wrote:
> On Thu, 2018-04-19 at 16:15 +0200, Peter Krempa wrote:
> > On Thu, Apr 19, 2018 at 15:45:52 +0200, Andrea Bolognani wrote:
> > > Mh, that's trickier than I initially though, because
> > > virBitmapParseSeparator() calls virReportError() itself on parse
> > > failure, and changing doesn't sound feasible.
> > 
> > I think logging of the error is okay, we just probably should reset it.
> > 
> > At any rate this patch can be pushed as-is ... well after it passes
> > build checks ;)
> 
> Are you referring to the syntax-check failure? I posted a v2 that
> fixes that, but also changes 1/2, without which 2/2 can't work.

No, I was referring to making that whole function pass even when the
numad hints can't be parsed.

I'll do a review of v2 in a bit.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20180419/0b1d73c3/attachment-0001.sig>


More information about the libvir-list mailing list