diff options
author | Vasu Dev <vasu.dev@intel.com> | 2011-09-28 00:37:57 -0400 |
---|---|---|
committer | James Bottomley <JBottomley@Parallels.com> | 2011-10-02 13:55:07 -0400 |
commit | ed26cfece6177310ad8f5896651a3187ce02711c (patch) | |
tree | 2997e4c698ce773d1fb6867a6daea0cfdd264737 /drivers/scsi/libfc | |
parent | 5c609ff937a91cf09b1f8d7612cd00bc046c27f7 (diff) |
[SCSI] libfc: cache align struct fc_fcp_pkt fields
Re-arrange its fields to avoid padding and have better
cacheline alignments.
Removed not used start_time, end_time and last_pkt_time
fields.
This all reduced this struct size to 448 from 480 and
that also reduced one cacheline on x86_64 beside
eliminating 8 pads. However kept logical fields together.
Signed-off-by: Vasu Dev <vasu.dev@intel.com>
Tested-by: Ross Brattain <ross.b.brattain@intel.com>
Signed-off-by: Yi Zou <yi.zou@intel.com>
Signed-off-by: James Bottomley <JBottomley@Parallels.com>
Diffstat (limited to 'drivers/scsi/libfc')
-rw-r--r-- | drivers/scsi/libfc/fc_fcp.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/drivers/scsi/libfc/fc_fcp.c b/drivers/scsi/libfc/fc_fcp.c index afb63c843144..f725d282b641 100644 --- a/drivers/scsi/libfc/fc_fcp.c +++ b/drivers/scsi/libfc/fc_fcp.c | |||
@@ -759,7 +759,6 @@ static void fc_fcp_recv(struct fc_seq *seq, struct fc_frame *fp, void *arg) | |||
759 | goto out; | 759 | goto out; |
760 | if (fc_fcp_lock_pkt(fsp)) | 760 | if (fc_fcp_lock_pkt(fsp)) |
761 | goto out; | 761 | goto out; |
762 | fsp->last_pkt_time = jiffies; | ||
763 | 762 | ||
764 | if (fh->fh_type == FC_TYPE_BLS) { | 763 | if (fh->fh_type == FC_TYPE_BLS) { |
765 | fc_fcp_abts_resp(fsp, fp); | 764 | fc_fcp_abts_resp(fsp, fp); |
@@ -1148,7 +1147,6 @@ static int fc_fcp_cmd_send(struct fc_lport *lport, struct fc_fcp_pkt *fsp, | |||
1148 | rc = -1; | 1147 | rc = -1; |
1149 | goto unlock; | 1148 | goto unlock; |
1150 | } | 1149 | } |
1151 | fsp->last_pkt_time = jiffies; | ||
1152 | fsp->seq_ptr = seq; | 1150 | fsp->seq_ptr = seq; |
1153 | fc_fcp_pkt_hold(fsp); /* hold for fc_fcp_pkt_destroy */ | 1151 | fc_fcp_pkt_hold(fsp); /* hold for fc_fcp_pkt_destroy */ |
1154 | 1152 | ||