[prev in list] [next in list] [prev in thread] [next in thread]
List: libguestfs
Subject: Re: [Libguestfs] [PATCH 3/3] sparsify: Ignore read-only LVs (RHBZ#1185561).
From: "Richard W.M. Jones" <rjones () redhat ! com>
Date: 2015-01-28 20:39:04
Message-ID: 20150128203904.GC17954 () redhat ! com
[Download RAW message or body]
On Wed, Jan 28, 2015 at 05:15:06PM +0100, Pino Toscano wrote:
> This looks to me that it would go through all the LVs, even RW ones,
> when is_read_only_lv is invoked, right?
>
> Considering that we get a list of all the LVs anyway when doing:
>
> > + let is_read_only_lv = is_read_only_lv g in
>
> wouldn't it be better to just get the list of UUIDs of RO LVs, and
> looking for 'fs' in that? Considering that in most of the cases LVs
> are RW, the list with RO LVs should be small if not empty, and thus
> save checks.
Yup, makes sense. I'll rework the patch.
Rich.
--
Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones
Read my programming and virtualization blog: http://rwmj.wordpress.com
Fedora Windows cross-compiler. Compile Windows programs, test, and
build Windows installers. Over 100 libraries supported.
http://fedoraproject.org/wiki/MinGW
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic