Code Review for do-not-assume-curzone

Prepared by:Dan McDonald on 2019-Sep-19 13:31 -0400 EDT
Workspace:/export/home/danmcd/ws/nfs-zone (at e85394a28977)
Compare against: /export/home/danmcd/ws/nfs-zone (explicit revision db5be39c0cd7)
Summary of changes: 60 lines changed: 52 ins; 0 del; 8 mod; 20986 unchg
Patch of changes: do-not-assume-curzone.patch

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New Patch Raw usr/src/uts/common/fs/nfs/nfs3_srv.c

curzone reality check and teardown changes to use the RIGHT zone
3 lines changed: 3 ins; 0 del; 0 mod; 4451 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New Patch Raw usr/src/uts/common/fs/nfs/nfs4_srv_ns.c

curzone reality check and teardown changes to use the RIGHT zone
7 lines changed: 5 ins; 0 del; 2 mod; 1267 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New Patch Raw usr/src/uts/common/fs/nfs/nfs4_state.c

curzone reality check and teardown changes to use the RIGHT zone
3 lines changed: 2 ins; 0 del; 1 mod; 4125 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New Patch Raw usr/src/uts/common/fs/nfs/nfs_auth.c

curzone reality check and teardown changes to use the RIGHT zone
1 line changed: 1 ins; 0 del; 0 mod; 1572 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New Patch Raw usr/src/uts/common/fs/nfs/nfs_export.c

Bad assertions
nfs_export_zone_init() can't assume called in zone-context.
curzone reality check and teardown changes to use the RIGHT zone
19 lines changed: 17 ins; 0 del; 2 mod; 2872 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New Patch Raw usr/src/uts/common/fs/nfs/nfs_srv.c

curzone reality check and teardown changes to use the RIGHT zone
4 lines changed: 4 ins; 0 del; 0 mod; 3153 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New Patch Raw usr/src/uts/common/klm/nlm_impl.c

curzone reality check and teardown changes to use the RIGHT zone
7 lines changed: 6 ins; 0 del; 1 mod; 2821 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New Patch Raw usr/src/uts/common/nfs/export.h

curzone reality check and teardown changes to use the RIGHT zone
16 lines changed: 14 ins; 0 del; 2 mod; 725 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.