Code Review for il_3544-lsa

Prepared by:Richard Lowe (richlowe) on 2013-Feb-24 21:11 -0500 EST
Workspace:/builds/work/illumos/illumos-git (at 057a4120a220)
Compare against: illumos/master (http://github.com/illumos/illumos-gate.git at 77372cb0f35e)
Summary of changes: 908 lines changed: 795 ins; 54 del; 59 mod; 5161 unchg
Patch of changes: il_3544-lsa.patch
Printable review: il_3544-lsa.pdf

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New Patch Raw exception_lists/packaging

saveargs: Implement as libsaveargs
8 lines changed: 8 ins; 0 del; 0 mod; 975 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New Patch Raw usr/src/cmd/mdb/intel/amd64/Makefile.kmdb

saveargs: Implement as libsaveargs
10 lines changed: 0 ins; 9 del; 1 mod; 33 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New Patch Raw usr/src/cmd/mdb/intel/amd64/kmdb/Makefile

saveargs: Implement as libsaveargs
1 line changed: 1 ins; 0 del; 0 mod; 104 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New Patch Raw usr/src/cmd/mdb/intel/amd64/mdb/Makefile

saveargs: Implement as libsaveargs
saveargs: let disasm do the lifting
3544 save-args matcher could be considerably more robust
3545 save-args matcher should accept saves maybe out-of-order
Reviewed by: Joshua M. Clulow <josh@sysmgr.org>
10 lines changed: 0 ins; 8 del; 2 mod; 51 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New Patch Raw usr/src/lib/Makefile

saveargs: Implement as libsaveargs
6 lines changed: 3 ins; 0 del; 3 mod; 692 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New Patch Raw usr/src/lib/libdisasm/common/libdisasm.h

saveargs: let disasm do the lifting
3 lines changed: 1 ins; 2 del; 0 mod; 80 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New Patch Raw usr/src/lib/libdisasm/common/mapfile-vers

saveargs: let disasm do the lifting
1 line changed: 1 ins; 0 del; 0 mod; 55 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New Patch Raw usr/src/lib/libdisasm/i386/dis_i386.c

saveargs: let disasm do the lifting
11 lines changed: 9 ins; 2 del; 0 mod; 245 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New Patch Raw usr/src/lib/libdisasm/sparc/dis_sparc.c

saveargs: Implement as libsaveargs
saveargs: let disasm do the lifting
10 lines changed: 7 ins; 3 del; 0 mod; 336 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New Patch Raw usr/src/lib/libproc/Makefile.com

saveargs: let disasm do the lifting
3544 save-args matcher could be considerably more robust
3545 save-args matcher should accept saves maybe out-of-order
Reviewed by: Joshua M. Clulow <josh@sysmgr.org>
7 lines changed: 0 ins; 7 del; 0 mod; 122 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New Patch Raw usr/src/lib/libproc/amd64/Makefile

saveargs: Implement as libsaveargs
saveargs: let disasm do the lifting
3 lines changed: 1 ins; 2 del; 0 mod; 34 unchg

------ ------ ------ ------ ------ --- New Patch Raw usr/src/lib/libsaveargs/Makefile

saveargs: Implement as libsaveargs
65 lines changed: 65 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ ------ --- New Patch Raw usr/src/lib/libsaveargs/Makefile.com

saveargs: Implement as libsaveargs
99 lines changed: 99 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ ------ --- New Patch Raw usr/src/lib/libsaveargs/Makefile.targ

saveargs: Implement as libsaveargs
80 lines changed: 80 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ ------ --- New Patch Raw usr/src/lib/libsaveargs/amd64/Makefile

saveargs: Implement as libsaveargs
34 lines changed: 34 ins; 0 del; 0 mod; 0 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New Patch Raw usr/src/lib/libsaveargs/amd64/saveargs.c (renamed and modified from usr/src/common/saveargs/saveargs.c)

saveargs: Implement as libsaveargs
171 lines changed: 124 ins; 18 del; 29 mod; 221 unchg

------ ------ ------ ------ ------ --- New Patch Raw usr/src/lib/libsaveargs/common/linktest_stand.c

saveargs: Implement as libsaveargs
17 lines changed: 17 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ ------ --- New Patch Raw usr/src/lib/libsaveargs/common/llib-lsaveargs

saveargs: Implement as libsaveargs
15 lines changed: 15 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ ------ --- New Patch Raw usr/src/lib/libsaveargs/common/mapfile-vers

saveargs: Implement as libsaveargs
37 lines changed: 37 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ ------ Old New Patch Raw usr/src/lib/libsaveargs/common/saveargs.h (renamed from usr/src/common/saveargs/saveargs.h)

saveargs: Implement as libsaveargs
0 lines changed: 0 ins; 0 del; 0 mod; 52 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New Patch Raw usr/src/lib/libsaveargs/tests/README (renamed and modified from usr/src/common/saveargs/tests/README)

saveargs: Implement as libsaveargs
5 lines changed: 5 ins; 0 del; 0 mod; 9 unchg

------ ------ ------ ------ ------ --- New Patch Raw usr/src/lib/libsaveargs/tests/dump/Makefile

saveargs: Implement as libsaveargs
39 lines changed: 39 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ ------ --- New Patch Raw usr/src/lib/libsaveargs/tests/dump/dump.c

saveargs: Implement as libsaveargs
152 lines changed: 152 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ ------ Old New Patch Raw usr/src/lib/libsaveargs/tests/functional/Makefile (renamed from usr/src/common/saveargs/tests/functional/Makefile)

saveargs: Implement as libsaveargs
0 lines changed: 0 ins; 0 del; 0 mod; 45 unchg

------ ------ ------ ------ ------ Old New Patch Raw usr/src/lib/libsaveargs/tests/functional/align.c (renamed from usr/src/common/saveargs/tests/functional/align.c)

saveargs: Implement as libsaveargs
0 lines changed: 0 ins; 0 del; 0 mod; 32 unchg

------ ------ ------ ------ ------ Old New Patch Raw usr/src/lib/libsaveargs/tests/functional/basic.c (renamed from usr/src/common/saveargs/tests/functional/basic.c)

saveargs: Implement as libsaveargs
0 lines changed: 0 ins; 0 del; 0 mod; 32 unchg

------ ------ ------ ------ ------ Old New Patch Raw usr/src/lib/libsaveargs/tests/functional/big-struct-ret-and-spill.c (renamed from usr/src/common/saveargs/tests/functional/big-struct-ret-and-spill.c)

saveargs: Implement as libsaveargs
0 lines changed: 0 ins; 0 del; 0 mod; 38 unchg

------ ------ ------ ------ ------ Old New Patch Raw usr/src/lib/libsaveargs/tests/functional/big-struct-ret.c (renamed from usr/src/common/saveargs/tests/functional/big-struct-ret.c)

saveargs: Implement as libsaveargs
0 lines changed: 0 ins; 0 del; 0 mod; 38 unchg

------ ------ ------ ------ ------ Old New Patch Raw usr/src/lib/libsaveargs/tests/functional/small-struct-ret-and-spill.c (renamed from usr/src/common/saveargs/tests/functional/small-struct-ret-and-spill.c)

saveargs: Implement as libsaveargs
0 lines changed: 0 ins; 0 del; 0 mod; 37 unchg

------ ------ ------ ------ ------ Old New Patch Raw usr/src/lib/libsaveargs/tests/functional/small-struct-ret.c (renamed from usr/src/common/saveargs/tests/functional/small-struct-ret.c)

saveargs: Implement as libsaveargs
0 lines changed: 0 ins; 0 del; 0 mod; 37 unchg

------ ------ ------ ------ ------ Old New Patch Raw usr/src/lib/libsaveargs/tests/functional/stack-spill.c (renamed from usr/src/common/saveargs/tests/functional/stack-spill.c)

saveargs: Implement as libsaveargs
0 lines changed: 0 ins; 0 del; 0 mod; 32 unchg

------ ------ ------ ------ ------ Old New Patch Raw usr/src/lib/libsaveargs/tests/functional/test.sh (renamed from usr/src/common/saveargs/tests/functional/test.sh)

saveargs: Implement as libsaveargs
0 lines changed: 0 ins; 0 del; 0 mod; 37 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New Patch Raw usr/src/lib/libsaveargs/tests/testmatch/Makefile (renamed and modified from usr/src/common/saveargs/tests/testmatch/Makefile)

saveargs: Implement as libsaveargs
8 lines changed: 2 ins; 3 del; 3 mod; 35 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New Patch Raw usr/src/lib/libsaveargs/tests/testmatch/data.s (renamed and modified from usr/src/common/saveargs/tests/testmatch/data.s)

saveargs: Implement as libsaveargs
58 lines changed: 58 ins; 0 del; 0 mod; 396 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New Patch Raw usr/src/lib/libsaveargs/tests/testmatch/testmatch.c (renamed and modified from usr/src/common/saveargs/tests/testmatch/testmatch.c)

saveargs: Implement as libsaveargs
57 lines changed: 36 ins; 0 del; 21 mod; 116 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New Patch Raw usr/src/pkg/manifests/system-library.mf

saveargs: Implement as libsaveargs
1 line changed: 1 ins; 0 del; 0 mod; 1277 unchg

This code review page was prepared using /opt/onbld/bin/webrev. Webrev is maintained by the illumos project. The latest version may be obtained here.