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

Re: [dm-devel] [PATCH 0/2] block: online resize of disk partitions



-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On 2/13/2012 2:30 PM, Vivek Goyal wrote:
> Hi,
> 
> Following patch adds support for online resizing of a partition.
> This patch is based on previously posted patches by Phillip Susi.
> 
> There are two patches. Out of which one is kernel patch and other
> one is util-linux patch to add support of a user space utility
> "resizepart" to allow resizing the partition.
> 
> This ioctl only resizes the partition size in kenrel and does not
> change the size on disk. A user needs to make sure that
> corresponding changes are made to disk data structures also using
> fdisk(or partx), if changes are to be retained across reboot.
> 
> I made some changes according to the feedback received last time.
> Following are changes since the version Phillip posted.
> 
> - RESIZE ioctl ignores the partition "start" and does not expect
> user to specify one. Caller needs to just specify "device",
> "partition number" and "size" of new partition.

I would prefer that the start argument not be left undefined.  I
checked it to make sure it was unchanged as a simple sanity check to
guard against user space being goofy, as well as to allow for the
possibility of extending the interface in the future to allow changing
the start as well as the length.

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2.0.17 (MingW32)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/

iQEcBAEBAgAGBQJPOYH+AAoJEJrBOlT6nu75rvwIALjEMrUZPEvwYzH9MI9wHmGg
lZzeBTSalqEOWL/hM0Ge6o1Of1adjeEyB7riXi7Fbb4Z8U55oBkAyKcRD/fdm0EA
8SyfMnGnMqZjh6VthIZMw43HbNwHojXeu2ri4ygeUORMIrX8tmZLAePw6iuOWcX6
qxOZsJOz3CdoGy2yc362AuIQACKH6IfFimP81NH5uUA01DIawWpes3EuN+KcxOQP
hlIeixVPzlNrXuDY/m35xlXNaXcTUqTYFJ30vvVn0/MHylAbfCXKx6P19jvPHTt9
j0OyBNVnsb/BCKsbSW89lfHoVPMeYC6Kz2u8d1AX5byCxcYcznqclA6S93/UUms=
=ZABC
-----END PGP SIGNATURE-----


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