On 3/25/11 4:53 PM, Drew Fisher wrote:

Webrev location:
http://cr.opensolaris.org/~drewfish/cud_ti_2/

Due to Alok's untimely departure, I do not have his existing webrev directory to do an incremental webrev. If this is problematic, let me know and I'll work with Glenn to get Alok's machine back so I can try to spin an incremental webrev.


Thanks to Dave for pointing out how awesome wget is.

Here's an incremental webrev of the changes between the first and second code reviews.

I think.

http://cr.opensolaris.org/~drewfish/cud_ti_2_incr/


-Drew


The major changes between round 1 and round 2 are:
- how disk geometry is figured for non VTOC labeled disks. In debugging instantiation issues with Karen's installer, we found a pretty large bug in libdiskmgt (shocking, I know!). Sanjay is still in the process of examining what is wrong in libdiskmgt and how to correctly fix the issue.
- how slices are created on the disk.
- extremely basic GPT label detection and support. We are *not* yet ready to handle full creation of a GPT labeled disk with slices, but discovery now correctly finds them and adds them to the DOC. Consider GPT support in TI/TD to be read-only.

**NOTE**
I desperately need people to look at the ctypes code. I was unable to find reviewers for the first pass of the code. I know it's complicated and I am more than willing to sit on the phone with people to explain it if need be.

I would love for people to get comments by April 1st. Fool's Day jokes/comments are acceptable and encouraged.

Thanks,
- Drew, Alok and Jean

(a) usr/src/lib/install_target/td.py
    usr/src/lib/install_target/test_td.py
    usr/src/lib/install_target/vdevs.py
    usr/src/lib/install_target/size.py
   usr/src/lib/install_target/test/test_zpool_vdevs.py

b) usr/src/lib/install_target/ti.py
   usr/src/lib/install_target/physical.py
   usr/src/lib/install_target/logical.py
   usr/src/lib/install_target/test/test_target_instantiation.py
   usr/src/lib/install_target/test/ti_full.py

c) usr/src/lib/install_target/shadow/*
   usr/src/lib/install_target/test/test_shadow_list.py

d) usr/src/lib/install_target/libadm/*
   usr/src/lib/install_target/libbe/*
   usr/src/lib/install_target/libdevinfo/*
   usr/src/lib/install_target/libdiskmgt/*
   usr/src/lib/install_target/libnvpair/*

e) Makefiles, DC changes, MP test changes, engine and
   errorsvc changes, packaging
_______________________________________________
caiman-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/caiman-discuss
_______________________________________________
caiman-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/caiman-discuss

Reply via email to