[libvirt] [PATCH 1/4] virsh-domain: use correct base for virStrToLong_ui

Pavel Hrdina phrdina at redhat.com
Tue Oct 27 14:26:59 UTC 2015


On Mon, Oct 26, 2015 at 09:48:20AM -0400, Laine Stump wrote:
> On 10/26/2015 05:53 AM, Pavel Hrdina wrote:
> > The current situation is not ideal, it's not documented anywhere and for users
> > you can only try the command and see what happens.  Yes, it can and probably
> > will break some scripts for some users, but I think we should make it somehow
> > consistent and document it properly how to format it.
> >
> > The output of nodedev-dumpxml should be definitely fixed to print the PCI
> > address using only hex numbers.
> >
> > For the parsing part, this code is currently used only for 'attach-disk' and
> > the 'attach-interface' will be a new functionality and we can easily restrict
> > the format of PCI address to be provided only in hex numbers with '0x' prefix.
> 
> /me wakes up Monday morning, reads this paragraph, and realizes that he 
> should have looked at the entire patch before commenting :-/
> 
> Yeah, forget everything I said. I hadn't scrolled all the way to the 
> bottom of the patch, but just assumed from the commit message that it 
> was going to fix the base at 16 for the XML parser too. Pretty stupid of 
> me - sorry for all the noise over nothing.
> 
> Definitely any time someone is writing a unified PCI address 
> (DDDD:BB:SS.F) I think we can safely assume/require they are using hex. 
> Really, I'm surprised this didn't trip anyone up before now.
> 
> ACK.

Thanks :) pushed now

Pavel

> 
> --
> libvir-list mailing list
> libvir-list at redhat.com
> https://www.redhat.com/mailman/listinfo/libvir-list




More information about the libvir-list mailing list