diff options
| author |   Stephen M. Cameron <scameron@beardog.cce.hp.com> | 2012-01-19 14:00:48 -0600 |
|---|---|---|
| committer |   James Bottomley <JBottomley@Parallels.com> | 2012-02-19 08:08:55 -0600 |
| commit | d66ae08bad182e9a87859e120e61cfd51e402ed8 (patch) | |
| tree | 1f7c7eca29f268cfef6927d4f503752c0a337511 /drivers/scsi/hpsa.h | |
| parent | [SCSI] hpsa: fix per device memory leak on driver unload (diff) | |
| download | linux-dev-d66ae08bad182e9a87859e120e61cfd51e402ed8.tar.xz linux-dev-d66ae08bad182e9a87859e120e61cfd51e402ed8.zip | |
[SCSI] hpsa: removed unneeded structure member max_sg_entries and fix badly named constant MAXSGENTRIES
We had both h->max_sg_entries and h->maxsgentries in the per controller
structure which is terribly confusing. max_sg_entries was really
just a constant, 32, which defines how big the "block fetch table"
is, which is as large as the max number of SG elements embedded
within a command (excluding SG elements in chain blocks).
MAXSGENTRIES was the constant used to denote the max number of SG
elements embedded within a command, also a poor name.
So renamed MAXSGENTREIS to SG_ENTRIES_IN_CMD, and removed
h->max_sg_entries and replaced it with SG_ENTRIES_IN_CMD.
h->maxsgentries is unchanged, and is the maximum number of sg
elements the controller will support in a command, including
those in chain blocks, minus 1 for the chain block pointer..
Signed-off-by: Stephen M. Cameron <scameron@beardog.cce.hp.com>
Signed-off-by: James Bottomley <JBottomley@Parallels.com>
Diffstat (limited to 'drivers/scsi/hpsa.h')
| -rw-r--r-- | drivers/scsi/hpsa.h | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/drivers/scsi/hpsa.h b/drivers/scsi/hpsa.h index 91edafb8c7e6..b4b97ebcb47a 100644 --- a/drivers/scsi/hpsa.h +++ b/drivers/scsi/hpsa.h @@ -58,7 +58,6 @@ struct ctlr_info { unsigned long paddr; int nr_cmds; /* Number of commands allowed on this controller */ struct CfgTable __iomem *cfgtable; - int max_sg_entries; int interrupts_enabled; int major; int max_commands; |