Re: PATCH: 14/16: disk partition backend

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

 



"Daniel P. Berrange" <berrange@xxxxxxxxxx> wrote:
> On Fri, Feb 15, 2008 at 07:52:37PM +0100, Jim Meyering wrote:
>> "Daniel P. Berrange" <berrange@xxxxxxxxxx> wrote:
...
>> Shouldn't size be 1 larger?  i.e.,
>>
>>         unsigned long long size =
>>             dev->freeExtents[i].end -
>>             dev->freeExtents[i].start + 1;
>
> I don't think so - the +1  adjustment is already done in the libvirt_parthelper
> program, but I'll double-check

Yes, you're right:
(from that same patch):

+        /* We do +1 on geom.end, because we want end of the last sector
+         * in bytes, not the last sector number

--
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]