Hi Sundar,

On Wed, 2007-03-28 at 10:09 -0700, Sundar Yamunachari wrote:
> Niall,
> 
> Niall Power wrote:
> > More comments/questions after having spent more timing examining the
> > disk layout modelling:)
> >
> > 4.1.1:
> > The data model assumes that only one VTOC (DiskSlices_t) can exist
> > on disk. On X86, the DiskSlice_t is associated with a parition.  Will the 
> > assumption on X86 that only one partition can contain a VTOC hold true
> > in the future? Will ZFS make this irrelevant?
> >
> > --------------------------------------------------------------------------------------
> > typedef struct {
> >         char                     *diskName;                      /* Disk 
> > Name for look up */
> >         PartitionInfo_t          pinfo[FD_NUMPART];              /* fdisk 
> > partitions */
> > } DiskParts_t;
> >
> > How is the pinfo[FD_NUMPART] array ordered? Is it ordered by partition ID 
> > or by
> > physical partition order on the disk?
> >   
> The partitions are listed based on the partition id. Partition numbers 
> are set based on the order of creation.
> >  
> > --------------------------------------------------------------------------------------
> > typedef struct {
> >      int                    partitionId; /* fdisk id (1-4) */
> >      int                    partitionOrder; /* Order in the disk */
> >      int                    partitionType; /* Solaris/linux swap/X86boot */
> >      om_content_type_t      contentType; /* Solaris/Linux */
> >      long                   partitionSize; /* Size in GB */
> >      boolean_t              active;      /* Is the partition active */
> >
> > } PartitionInfo_t;
> >
> > There is no indication of of where the partition offset starts or ends on 
> > the disk. It forces us to 
> > assume that partition at order 0 begins at offset 0 and, partition at order 
> > 1 begins immediately
> > after where partition at order 0 ends and so on. This seems like a 
> > dangerous assumption.
> > We discussed this briefly during the work week and about how the GUI would 
> > handle it. I believe
> > we said the GUI, due to it's design would not support it (it doesn't convey 
> > start and end points of
> > partitions to the user) and so it would issue a warning and force a 
> > destructive repartitioning of the disk. 
> > But from the data modelling perspective, partition start and end offsets 
> > seem like important data to capture.
> >   
> I was thinking of not exposing disk information like starting cylinder 
> and offset. I was hoping that we can get away by providing the 
> partitionOrder information. It looks like we need to provide starting 
> cylinder and offset. I will add this information and remove 
> partitionOrder from the PartitionInfo_t.

>From then GUI's perspective, we don't want to be dealing with intimate
disk geometry details (Cylinders, heads, sectors etc.). Is there a way
to represent the offset in MB just like partition sizes? Also, I don't
think there is a conflict between parititionOrder and offset. Can we
leave paritionOrder in, since it seems like a nice convenience to have?

Niall.

> 
> - Sundar
> > Thanks,
> > Niall.
> > --
> > This message posted from opensolaris.org
> > _______________________________________________
> > caiman-discuss mailing list
> > caiman-discuss at opensolaris.org
> > http://opensolaris.org/mailman/listinfo/caiman-discuss
> >   


Reply via email to