Code Review for 6740

Prepared by:Dan McDonald (danmcd) on 2016-Apr-01 09:12 -0400 EDT
Workspace:/export/home/danmcd/ws/6740 (at 7f7916e510e4)
Compare against: origin/master (/export/home/danmcd/ws/illumos-gate at 915c848a88eb)
Summary of changes: 72 lines changed: 54 ins; 0 del; 18 mod; 8325 unchg
Patch of changes: 6740.patch

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New Patch Raw usr/src/uts/common/io/comstar/port/iscsit/iscsit.c

6740 ISCSI_OP_SCSI_RSP needs support
Reviewed by: Rob Gittins <rob.gittins@nexenta.com>
Reviewed by: Dan McDonald <danmcd@omniti.com>
21 lines changed: 17 ins; 0 del; 4 mod; 3426 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New Patch Raw usr/src/uts/common/io/comstar/port/iscsit/iscsit.h

6740 ISCSI_OP_SCSI_RSP needs support
Reviewed by: Rob Gittins <rob.gittins@nexenta.com>
Reviewed by: Dan McDonald <danmcd@omniti.com>
1 line changed: 1 ins; 0 del; 0 mod; 866 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New Patch Raw usr/src/uts/common/io/comstar/port/iscsit/iscsit_tgt.c

6740 ISCSI_OP_SCSI_RSP needs support
Reviewed by: Rob Gittins <rob.gittins@nexenta.com>
Reviewed by: Dan McDonald <danmcd@omniti.com>
2 lines changed: 0 ins; 0 del; 2 mod; 2107 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New Patch Raw usr/src/uts/common/io/idm/idm_conn_sm.c

6742 Freed and reused idm_conn_t buffer leads to system panic.
Reviewed by: Rob Gittins <rob.gittins@nexenta.com>
Reviewed by: Steve Ping <steve.ping@nexenta.com>
Reviewed by: Dan McDonald <danmcd@omniti.com>
Reviewed by: Yuri Pankov <yuri.pankov@nexenta.com>
21 lines changed: 9 ins; 0 del; 12 mod; 1645 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New Patch Raw usr/src/uts/common/sys/idm/idm_conn_sm.h

6742 Freed and reused idm_conn_t buffer leads to system panic.
Reviewed by: Rob Gittins <rob.gittins@nexenta.com>
Reviewed by: Steve Ping <steve.ping@nexenta.com>
Reviewed by: Dan McDonald <danmcd@omniti.com>
Reviewed by: Yuri Pankov <yuri.pankov@nexenta.com>
27 lines changed: 27 ins; 0 del; 0 mod; 281 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.