Re: [PATCH 4/4] conf: Fix generating addresses for SCSI hostdev

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Jan 04, 2018 at 07:24:30AM -0500, John Ferlan wrote:

Jan -

ping on the question from my response to your review...

On 12/20/2017 01:33 PM, John Ferlan wrote:


On 12/20/2017 07:38 AM, Ján Tomko wrote:

[...]


Easier to read as:
for (next_unit = -1; next_unit < -1; controller++)
   next_unit = virDomainControllerSCSINextUnit(def, max_unit, controller);


Not functionally the same comparisons... Caused
hostdev-scsi-autogen-address to fail... There's 11 hostdevs and only 1
controller. We never get controller==1 from that for loop.

I can change @ret above to be @next_unit though


Is changing to use next_unit enough?

Yes.

Or did you have some other easier
to read loop that actually works that you'd prefer to see?


I do not.

Jan

Attachment: signature.asc
Description: Digital signature

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]
  Powered by Linux