Re: [PATCH] Recheck if a partition should be ignored after getting its disk (#518971)

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

 



On Wed, 2009-09-02 at 16:49 +0200, Hans de Goede wrote:
> Recheck if a partition should be ignored after getting its disk (if its
> disk was not scanned yet). Once we've the disk, we need to recheck if the
> partition should be ignored again, as this catches cases when we scan the
> partition before the disk and we have for example a disk which is part of
> a BIOS raid set (and thus we should ignore the raw partitions).

Looks good.

> ---
>  storage/devicetree.py |    8 ++++++++
>  1 files changed, 8 insertions(+), 0 deletions(-)
> 
> diff --git a/storage/devicetree.py b/storage/devicetree.py
> index f4596de..9248cd6 100644
> --- a/storage/devicetree.py
> +++ b/storage/devicetree.py
> @@ -1100,6 +1100,14 @@ class DeviceTree(object):
>                  log.error("failure scanning device %s" % disk_name)
>                  return
>  
> +            # Now that we've the disk, check if this partition should be
> +            # ignored again. This catches cases when we scan the partition
> +            # before the disk and we have for example a disk which is part of
> +            # a BIOS raid set (and thus we should ignore the raw partitions)
> +            if self.isIgnored(info):
> +                log.debug("ignoring %s (%s)" % (name, sysfs_path))
> +                return
> +
>          # Check that the disk has partitions. If it does not, we must have
>          # reinitialized the disklabel.
>          if not getattr(disk.format, "partitions", None):

_______________________________________________
Anaconda-devel-list mailing list
Anaconda-devel-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/anaconda-devel-list

[Index of Archives]     [Kickstart]     [Fedora Users]     [Fedora Legacy List]     [Fedora Maintainers]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [Yosemite Photos]     [KDE Users]     [Fedora Tools]
  Powered by Linux