On Wed, Jul 08, 2015 at 06:00:03PM +0300, Sagi Grimberg wrote:
In fixed size sense format the information field is a four byte
field.
This looks correct to me,
Reviewed-by: Christoph Hellwig h...@lst.de
But: this will truncate 32bit sector numbers. Maybe we need to
enable descriptor format
Thanks for this, I'm sending a v2 shortly.
On Friday 07/03 at 09:00 -0700, Christoph Hellwig wrote:
On Mon, Jun 08, 2015 at 08:50:55PM -0700, Calvin Owens wrote:
This refactors the fw_event code to use the new refcount.
I spent some time looking over this code because it's so convoluted.
On Friday 07/03 at 08:38 -0700, Christoph Hellwig wrote:
+struct _sas_device *
+mpt2sas_scsih_sas_device_get_by_sas_address_nolock(struct MPT2SAS_ADAPTER
*ioc,
+u64 sas_address)
Any chance to use a shorter name for this function? E.g.
__mpt2sas_get_sdev_by_addr ?
Will do.
On Friday 07/03 at 09:02 -0700, Christoph Hellwig wrote:
On Mon, Jun 08, 2015 at 08:50:56PM -0700, Calvin Owens wrote:
Since the fw_event deletes itself from the list, cleanup_queue() can
walk onto garbage pointers or walk off into freed memory.
This refactors the code in
The fw_event_work struct is concurrently referenced at shutdown, so
add a refcount to protect it, and refactor the code to use it.
Additionally, refactor _scsih_fw_event_cleanup_queue() such that it
no longer iterates over the list without holding the lock, since
_firmware_event_work()
Hello all,
This patchset attempts to address problems we've been having with
panics due to memory corruption from the mpt2sas driver.
Thanks,
Calvin
Patches in this series:
[PATCH 1/2] mpt2sas: Refcount sas_device objects and fix unsafe list usage
[PATCH 2/2] mpt2sas: Refcount fw_events and fix
These objects can be referenced concurrently throughout the driver, we
need a way to make sure threads can't delete them out from under each
other. This patch adds the refcount, and refactors the code to use it.
Additionally, we cannot iterate over the sas_device_list without
holding the lock, or
Hi Brian,
Thanks for reviewing. Comments inline below.
-matt
On Jul 10, 2015, at 2:49 PM, Brian King wrote:
On 06/19/2015 05:37 PM, Matthew R. Ochs wrote:
cfg-init_state = INIT_STATE_NONE;
cfg-dev = pdev;
+cfg-last_lun_index[0] = 0;
+cfg-last_lun_index[1] = 0;
Hello Hannes Reinecke,
The patch b6ff1b14cdf4: [SCSI] scsi_dh: Update EMC handler from Jul
17, 2008, leads to the following static checker warning:
drivers/scsi/device_handler/scsi_dh_emc.c:252 parse_sp_model()
warn: buffer overflow 'buffer' 252 = 255
9 matches
Mail list logo