Subject: Re: [PATCH] parted: don't reload partition table on every command

From: "Brian C. Lane" <[email-masked]>
Date: Thu, 17 Apr 2014 15:13:07 -0700

On Sat, Mar 29, 2014 at 01:50:12PM -0400, Phillip Susi wrote:
> gpt was using a static local variable to suppress repeatedly reporting
> an error if you chose to ignore it.  This is incorrect as the variable is
> global to all disks, and ignoring the error on one should not suppress its
> reporting on another.  Moving the flag to the PedDisk object made it
> effectively useless because parted was destroying the PedDisk and reloading
> the partition table on every command.
> 
> Parted has been reworked to cache the PedDisk once loaded, and only discard
> it when changing disks, or creating a new disklabel.
> ---
>  NEWS                     |   5 +
>  libparted/labels/gpt.c   |   7 +-
>  parted/command.c         |   6 +-
>  parted/command.h         |   6 +-
>  parted/parted.c          | 291 +++++++++++++++++++++++------------------------
>  parted/ui.c              |   8 +-
>  parted/ui.h              |   8 +-
>  tests/t3300-palo-prep.sh |   8 +-
>  8 files changed, 170 insertions(+), 169 deletions(-)

This makes t0283 start failing, it stops showing the error about
overlapping partitions.

-- 
Brian C. Lane | Anaconda Team | IRC: bcl #anaconda | Port Orchard, WA (PST8PDT)