Skip to content

Commit

Permalink
Change U16 to U32 due to atomic_inc_32_nv
Browse files Browse the repository at this point in the history
After run a long time with QAT compression, the variable "inst_num"
is overflow by "atomic_inc_32_nv", which causes its neighbor
variable overwritten. Change its definition from U16 to U32.

Reviewed-by: Brian Behlendorf <[email protected]>
Signed-off-by: Weigang Li <[email protected]>
Closes #6051
  • Loading branch information
wli5 authored and behlendorf committed Apr 26, 2017
1 parent 603a178 commit 7a25f08
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions module/zfs/qat_compress.c
Original file line number Diff line number Diff line change
Expand Up @@ -104,8 +104,8 @@ static kstat_t *qat_ksp;
static CpaInstanceHandle dc_inst_handles[MAX_INSTANCES];
static CpaDcSessionHandle session_handles[MAX_INSTANCES];
static CpaBufferList **buffer_array[MAX_INSTANCES];
static Cpa16U num_inst = 0;
static Cpa16U inst_num = 0;
static Cpa32U num_inst = 0;
static Cpa32U inst_num = 0;
static boolean_t qat_init_done = B_FALSE;
int zfs_qat_disable = 0;

Expand Down

0 comments on commit 7a25f08

Please sign in to comment.