Dave, Thanks for the review. Comments inline.
Jean Dave Miner wrote: > Jean McCormack wrote: >> I have posted the functional specification for the Install support >> for extended partitions and slice modification at: >> >> http://www.opensolaris.org/os/project/caiman/Extended_Partition_Support/EP_Func_Spec.txt/ >> >> >> >> Please review and provide comments by Wednesday, July 29, at 5pm >> Mountain. >> > > General: please version and date specs for reference purposes. Will do. I'll make this spec V1. > > 1.1 - Would it be more accurate to say that the project will be > integrated in two phases (rather than "broken into")? (Three, if you > count that libdiskmgt work goes into ON, not the Caiman project > source) Also should clarify that the resizing support is destructive > resize, right? Correct. > > 2.3 Any more specifics on test drivers that need development or > modification? Yeah. I should at list them here. > > 2.4 Can you list specific test suites in DIY that will be run? For OpenSolaris the Caiman-Install test suite For Nevada I was going to have them do everything. > > 3 - Suddenly use the term "logical disk" here when "logical partition" > is used everywhere else. Missed that instance. I'll change it. > > 5.3 Not clear to me if the second paragraph here is specifying > details of the first one, or something else? > Clarifying details. Possibly too detailed and I should remove it? > 6.x - There seems to be at least one missing scenario, in which no > extended partition already exists and one is created. > Added. > Dave
