These patches are intended for rhel6-branch and master. Bug #528386 is meant to improve the dasdfmt support we had from RHEL-5. I have divided the work in to two parts: 1) Functional. Detect unformatted DASDs, offer the user to option to format them or bail, perform the format and display progress. 2) Parallel formatting. IBM wants parallel formatting support for sites that have thousands of DASDs. I wanted to separate that work from these patches. I changed execWithPulseProgress() in iutil.py to execWithCallback(). The execWithPulseProgress() function is now implemented using execWithCallback. You can see how I use execWithCallback in the new dasd.py for displaying progress as captured from the dasdfmt command. I did introduce storage/dasd.py, which brings in a DASD class. The whole dasdfmt operation needs to occur before we initialize the storage code in anaconda. So like we have the iscsi and ZFCP classes and call startup() on those before initializing the storage code, that's what the DASD class is meant to do. It also leaves a convenient place for future DASD patches that need to deal with system-wide configuration files such as a possible /etc/dasd.conf or something like that. Lastly, compared to RHEL-5, the dasdfmt question is asked once. We used to present a Yes/Skip dialog for each unformatted DASD, which is simply unusable. I now build a list of all unformatted DASDs and tell the user how many were found and ask if you want them formatted. I did kick around the idea of presenting a checklist where users could uncheck DASDs they did not want anaconda to format and use, but really, we give users plenty of ways to skip or mask out devices with all of the linuxrc.s390 stuff, I felt doing it here was pointless. By the time anaconda runs, we should be able to use all of the DASDs we see. The next patch set will be for the parallel formatting with dasdfmt. Progress bar code in this set has been tested on both the iw and textw code. Works fine on both. -- David Cantrell <dcantrell@xxxxxxxxxx> Red Hat / Honolulu, HI _______________________________________________ Anaconda-devel-list mailing list Anaconda-devel-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/anaconda-devel-list