On 03/22/2016 04:34 AM, Nitesh Konkar wrote: > If the pool creation thread happens to detect the luns > in the scsi target,the size parameters will be populated > as part of the refresh called from storagePoolCreate(). > The commit <4a85bf3e2f> added additional refresh for > SCSI which should also have these values initialised > to zero, otherwise the values would appear exactly double. > A separate refresh would correct the values, but without > an explicit pool-refresh, the allocation/capacity would > appear exactly double after pool-create. > --- > src/storage/storage_backend_scsi.c | 3 +++ > 1 file changed, 3 insertions(+) > Your commit history is a bit off - commit id '4a85bf3e2f' was for the virStorageVolPoolRefreshThread not the virStoragePoolFCRefreshThread. Two very different threads and problems! In any case, not clearing the pool allocation, capacity, and available values in the FCRefreshThread was an oversight, but then again - the refreshPool ran and would clear them. However, it seems "timing" has changed since the original commit (there are storage state driver and FindLUs processing changes) and perhaps there is "enough time" for the luns to appear. I adjusted the commit to use commit id '512b874' and adjusted the commit description a bit and pushed. Thanks for catching this! John -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list