On Fri, Oct 08, 2021 at 10:04:12AM +0000, Arvin Schnell wrote: > > Hi, > > here is a patch to add the swap flag for the DASD label. So far > only the partition type on disk was set to swap if a new > partition was created with file system "linux-swap". With the > patch the swap flag can now also be seen and set like for other > labels. > > Additionally, parts of the patch [1] are included. That fixes > reading the flags from disk. So far the RAID and LVM flag was > only read when no file system signatures was found on the > partition. > > Also, as with my recent patch for GPT [2] the individual flags > "raid" and "lvm" are removed and instead the "system" variable is > used directly.
This is pretty similar to the changes to msdos, the concerns I have about it are that it changes the behavior of the *_set_system function, and read_dasd and I prefer to get the opinion of someone from IBM on this before merging it. If merged as is, with the change in *_set_system behavior, it will of course fit right in with the msdos changes and bumping the API. Brian -- Brian C. Lane (PST8PDT) - weldr.io - lorax - parted - pykickstart
