Re: [PATCH 0/5] Modify ida_* users to use ida_simple_*

2015-10-05 Thread Tejun Heo
On Thu, Oct 01, 2015 at 11:59:04AM -0700, Lee Duncan wrote: > The ida index management routines are used in several > driver modules to manage allocation and release of > index values. Reviewing the way in which the > ida routines were called, together with the small > number of such clients, led

Re: [PATCH 0/5] Modify ida_* users to use ida_simple_*

2015-10-05 Thread James Bottomley
On Mon, 2015-10-05 at 13:44 -0400, Tejun Heo wrote: > On Thu, Oct 01, 2015 at 11:59:04AM -0700, Lee Duncan wrote: > > The ida index management routines are used in several > > driver modules to manage allocation and release of > > index values. Reviewing the way in which the > > ida routines were

[PATCH for v4.3-rc] scsi_dh: fix use-after-free when removing scsi device

2015-10-05 Thread Junichi Nomura
The commit 1bab0de0274f ("dm-mpath, scsi_dh: don't let dm detach device handlers") removed reference counting of attached scsi device handler. As a result, handler data is freed immediately via scsi_dh->detach() in the context of scsi_remove_device() where activation request can be still in

Re: [PATCH v3 13/13] scsi: ufs: Add exynos ufs platform data

2015-10-05 Thread Alim Akhtar
CCing Rob Herring, Hi Arnd, On 10/01/2015 04:59 PM, Arnd Bergmann wrote: On Thursday 01 October 2015 18:46:34 kbuild test robot wrote: [auto build test results on v4.3-rc3 -- if it's inappropriate base, please ignore] config: x86_64-allmodconfig (attached as .config) reproduce: git

Re: [RFT v3] eata: Convert eata driver as normal PCI and platform device drivers

2015-10-05 Thread Arthur Marsh
Jiang Liu wrote on 03/10/15 17:41: If I do a normal boot which includes eata being loaded, the disk attached to the DPT2044W controller having its filesystems checked and mounted, then attempt a kexec reboot, I get the reboot pausing after the "synchronizing SCSI cache" messages as before.

[PATCH 0/2] scsi: reduce CONFIG_SCSI_CONSTANTS=y impact by 8k

2015-10-05 Thread Rasmus Villemoes
This reduces the impact of choosing CONFIG_SCSI_CONSTANTS by about 8KB. 2dd951ecd511 ("scsi: Conditionally compile in constants.c") updated the Kconfig help text from 12KB to 75KB. The 12K predated git so was certainly outdated. But I'm not sure where the 75K comes from; using size(1) on a

[PATCH 2/2] scsi: reduce CONFIG_SCSI_CONSTANTS=y impact by 8k

2015-10-05 Thread Rasmus Villemoes
On 64 bit, struct error_info has 6 bytes of padding, which amounts to over 4k of wasted space in the additional[] array. We could easily get rid of that by instead using separate arrays for the codes and the pointers. However, we can do even better than that and save an additional 6 bytes per

[PATCH 1/2] scsi: move Additional Sense Codes to separate file

2015-10-05 Thread Rasmus Villemoes
This is a purely mechanical move of the list of additional sense codes to a separate file, in preparation for reducing the impact of choosing CONFIG_SCSI_CONSTANTS=y by about 8k.. Signed-off-by: Rasmus Villemoes --- drivers/scsi/constants.c | 839

Re: [PATCH] 3w-9xxx: don't unmap bounce buffered commands

2015-10-05 Thread adam radford
On Sat, Oct 3, 2015 at 10:16 AM, Christoph Hellwig wrote: > 3w controller don't dma map small single SGL entry commands but instead > bounce buffer them. Add a helper to identify these commands and don't > call scsi_dma_unmap for them. > > Based on an earlier patch from James

Re: RFC: reduce CONFIG_SCSI_CONSTANTS impact by 4k

2015-10-05 Thread Julian Calaby
Hi Rasmus, On Sun, Oct 4, 2015 at 9:09 AM, Rasmus Villemoes wrote: > Subject: [PATCH 2/2] scsi: reduce CONFIG_SCSI_CONSTANTS=y impact by 8k > > On 64 bit, struct error_info has 6 bytes of padding, which amounts to > over 4k of wasted space in the additional[] array. We

RE: [PATCH 1/1] storvsc: Don't set the SRB_FLAGS_QUEUE_ACTION_ENABLE flag

2015-10-05 Thread KY Srinivasan
> -Original Message- > From: James Bottomley [mailto:james.bottom...@hansenpartnership.com] > Sent: Monday, August 31, 2015 7:02 AM > To: KY Srinivasan > Cc: gre...@linuxfoundation.org; linux-ker...@vger.kernel.org; > de...@linuxdriverproject.org; oher...@suse.com; >

[PATCHv2 0/1] Update SCSI hosts to use idr for host number mgmt

2015-10-05 Thread Lee Duncan
This patch updates the SCSI hosts module to use the idr index-management routines to manage its host_no index instead of using an ATOMIC integer. This means that host numbers can now be reclaimed and re-used. It also updates the hosts module to use the idr routine idr_find() to lookup hosts based

[PATCHv2 1/1] SCSI: update hosts module to use idr index management

2015-10-05 Thread Lee Duncan
Update the SCSI hosts module to use idr to manage its host_no index instead of an ATOMIC integer. This also allows using idr_find() to look up the SCSI host structure given the host number. This means that the SCSI host number will now be reclaimable. Signed-off-by: Lee Duncan

Re: [PATCH v5 33/34] cxlflash: Fix to avoid leaving dangling interrupt resources

2015-10-05 Thread Andrew Donnellan
On 02/10/15 01:58, Matthew R. Ochs wrote: When running with an unsupported AFU, the cxlflash driver fails the probe. When the driver is removed, the following Oops is encountered on a show_interrupts() thread: Call Trace: [c01fba5a7a10] [0003] 0x3 (unreliable) [c01fba5a7a60]

Inheritance

2015-10-05 Thread Stephen Woolfe
I am Stephen Woolfe,the Legal Attorney to late Engr. Mark with assets value placed at £10.9 Million.Contact me for more information.Email: stephenwoo...@qq.com -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majord...@vger.kernel.org More

Re: [PATCH] isci: fix two comment typos

2015-10-05 Thread Artur Paszkiewicz
On 10/04/2015 10:53 AM, Geliang Tang wrote: > Just fix two typos in the code comment. > > Signed-off-by: Geliang Tang > --- > drivers/scsi/isci/request.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/scsi/isci/request.c

[PATCH v2 1/3] scsi: drop unlikely behind BUG_ON()

2015-10-05 Thread Geliang Tang
(1) For !CONFIG_BUG cases, the bug call is a no-op, so we couldn't care less and the change is ok. (2) ppc and mips, which HAVE_ARCH_BUG_ON, do not rely on branch predictions as it seems to be pointless[1] and thus callers should not be trying to push an optimization in the first place. (3) For

Re: [PATCH v3 13/13] scsi: ufs: Add exynos ufs platform data

2015-10-05 Thread Alim Akhtar
Hi Rob, On Mon, Oct 5, 2015 at 7:41 PM, Rob Herring wrote: > On Mon, Oct 5, 2015 at 4:06 AM, Arnd Bergmann wrote: >> On Monday 05 October 2015 13:44:29 Alim Akhtar wrote: >>> CCing Rob Herring, >>> >>> Hi Arnd, >>> >>> On 10/01/2015 04:59 PM, Arnd Bergmann wrote:

Re: [PATCH 1/2] scsi: move Additional Sense Codes to separate file

2015-10-05 Thread Bart Van Assche
On 10/05/15 02:26, Rasmus Villemoes wrote: - {0x041A, "Logical unit not ready, start stop unit command in " -"progress"}, - {0x041B, "Logical unit not ready, sanitize in progress"}, - {0x041C, "Logical unit not ready, additional power use not yet " -"granted"},

Re: [PATCH 2/2] scsi: reduce CONFIG_SCSI_CONSTANTS=y impact by 8k

2015-10-05 Thread Bart Van Assche
On 10/05/15 02:26, Rasmus Villemoes wrote: struct error_info { unsigned short code12; /* 0x0302 looks better than 0x03,0x02 */ - const char * text; + unsigned short size; }; Had you considered to use the type uint16_t instead of unsigned short ? Bart. -- To

Re: [PATCH v3 13/13] scsi: ufs: Add exynos ufs platform data

2015-10-05 Thread Arnd Bergmann
On Monday 05 October 2015 09:11:33 Rob Herring wrote: > On Mon, Oct 5, 2015 at 4:06 AM, Arnd Bergmann wrote: > > On Monday 05 October 2015 13:44:29 Alim Akhtar wrote: > >> > >> On 10/01/2015 04:59 PM, Arnd Bergmann wrote: > >> > On Thursday 01 October 2015 18:46:34 kbuild test

Re: [PATCH v3 13/13] scsi: ufs: Add exynos ufs platform data

2015-10-05 Thread Rob Herring
On Mon, Oct 5, 2015 at 4:06 AM, Arnd Bergmann wrote: > On Monday 05 October 2015 13:44:29 Alim Akhtar wrote: >> CCing Rob Herring, >> >> Hi Arnd, >> >> On 10/01/2015 04:59 PM, Arnd Bergmann wrote: >> > On Thursday 01 October 2015 18:46:34 kbuild test robot wrote: >> >> [auto build