Re: Bypass block layer and Fill SCSI lower layer driver queue

2013-09-26 Thread Vladislav Bolkhovitin
Douglas Gilbert, on 09/18/2013 07:07 AM wrote: > On 13-09-18 03:58 AM, Jack Wang wrote: >> On 09/18/2013 08:41 AM, Alireza Haghdoost wrote: >>> Hi >>> >>> I am working on a high throughput and low latency application which >>> does not tolerate block layer overhead to send IO request directly to >>

Re: Race condition between "read CFQ stats" and "block device shutdown"

2013-09-26 Thread Hannes Reinecke
On 09/26/2013 06:23 PM, Anatol Pomozov wrote: > Hi > > On Thu, Sep 26, 2013 at 7:18 AM, Hannes Reinecke wrote: >> On 09/26/2013 03:54 PM, Tejun Heo wrote: >>> Hello, (cc'ing linux-scsi) >>> >>> On Wed, Sep 25, 2013 at 01:37:51PM -0700, Anatol Pomozov wrote: Hi On Wed, Sep 4, 2013 a

Re: [PATCH 31/51] DMA-API: media: omap3isp: use dma_coerce_mask_and_coherent()

2013-09-26 Thread Laurent Pinchart
Hi Russell, Thank you for the patch. On Thursday 19 September 2013 22:56:02 Russell King wrote: > The code sequence: > isp->raw_dmamask = DMA_BIT_MASK(32); > isp->dev->dma_mask = &isp->raw_dmamask; > isp->dev->coherent_dma_mask = DMA_BIT_MASK(32); > bypasses the architectures ch

Re: [PATCH 00/51] DMA mask changes

2013-09-26 Thread Rafał Miłecki
2013/9/19 Russell King - ARM Linux : > This email is only being sent to the mailing lists in question, not to > anyone personally. The list of individuals is far to great to do that. > I'm hoping no mailing lists reject the patches based on the number of > recipients. Huh, I think it was enough t

RE: SCSI's heuristics for enabling WRITE SAME still need work [was: dm mpath: disable WRITE SAME if it fails]

2013-09-26 Thread Saxena, Sumit
>-Original Message- >From: Douglas Gilbert [mailto:dgilb...@interlog.com] >Sent: Thursday, September 26, 2013 9:17 PM >To: Martin K. Petersen; Bernd Schubert >Cc: Mike Snitzer; Hannes Reinecke; emi...@redhat.com; device-mapper >development; linux-scsi@vger.kernel.org; Saxena, Sumit >Subje

[PATCH] sg: fix memory leaks

2013-09-26 Thread Douglas Gilbert
This patch plugs two memory leaks in the sg driver. The first was reported by Vegard Nossum and the second was found by checking for the same pattern. ChangeLog: - close two heap memory leaks. The first is an error path; the second is associated with getting debug information from pr

Re: Race condition between "read CFQ stats" and "block device shutdown"

2013-09-26 Thread Tejun Heo
Hello, On Thu, Sep 26, 2013 at 09:23:19AM -0700, Anatol Pomozov wrote: > We use modified version of "sbull" block device driver from GKH book. > We use it for testing block device startup/shutdown path + CFQ > manipulation. > > The sbull driver uses function > blk_init_queue(..., &dev->qlock); >

Re: Race condition between "read CFQ stats" and "block device shutdown"

2013-09-26 Thread Anatol Pomozov
Hi On Thu, Sep 26, 2013 at 7:18 AM, Hannes Reinecke wrote: > On 09/26/2013 03:54 PM, Tejun Heo wrote: >> Hello, (cc'ing linux-scsi) >> >> On Wed, Sep 25, 2013 at 01:37:51PM -0700, Anatol Pomozov wrote: >>> Hi >>> >>> On Wed, Sep 4, 2013 at 9:07 AM, Tejun Heo wrote: Hello, On Wed,

[PATCH RESEND] iscsi_tcp: consider session state in iscsi_sw_sk_state_check

2013-09-26 Thread Chris Leech
It seems some iSCSI targets (including the Linux kernel target) close the TCP connection from the target side immediately after processing a session logout. When a TCP FIN comes in right after the iSCSI logout response, iscsi_sw_sk_state_check sees the local socket as not yet being in CLOSE_WAIT o

Re: SCSI's heuristics for enabling WRITE SAME still need work [was: dm mpath: disable WRITE SAME if it fails]

2013-09-26 Thread Douglas Gilbert
On 13-09-26 10:42 AM, Martin K. Petersen wrote: "Bernd" == Bernd Schubert writes: Bernd, Bernd> Both types of systems we have in-house neither block limits vpd Bernd> nor READ_CAP16 return anything that would indicate discard is Bernd> supported. But UNMAP and WRITE SAME unmap(*) just work fi

Re: SCSI's heuristics for enabling WRITE SAME still need work [was: dm mpath: disable WRITE SAME if it fails]

2013-09-26 Thread Bernd Schubert
On 09/26/2013 04:42 PM, Martin K. Petersen wrote: "Bernd" == Bernd Schubert writes: Bernd, Bernd> Both types of systems we have in-house neither block limits vpd Bernd> nor READ_CAP16 return anything that would indicate discard is Bernd> supported. But UNMAP and WRITE SAME unmap(*) just work

Re: SCSI's heuristics for enabling WRITE SAME still need work [was: dm mpath: disable WRITE SAME if it fails]

2013-09-26 Thread Martin K. Petersen
> "Bernd" == Bernd Schubert writes: Bernd, Bernd> Both types of systems we have in-house neither block limits vpd Bernd> nor READ_CAP16 return anything that would indicate discard is Bernd> supported. But UNMAP and WRITE SAME unmap(*) just work fine. I have a collection of different SSDs in

Re: Race condition between "read CFQ stats" and "block device shutdown"

2013-09-26 Thread Tejun Heo
Hey, Hannes. On Thu, Sep 26, 2013 at 04:18:34PM +0200, Hannes Reinecke wrote: > However, none of the modern devices should be using this lock > switching, so I would just ignore it. > EG SCSI most definitely doesn't use it. The kernel is crashing from it, so I don't think ignoring is an acceptabl

Re: Race condition between "read CFQ stats" and "block device shutdown"

2013-09-26 Thread Hannes Reinecke
On 09/26/2013 03:54 PM, Tejun Heo wrote: > Hello, (cc'ing linux-scsi) > > On Wed, Sep 25, 2013 at 01:37:51PM -0700, Anatol Pomozov wrote: >> Hi >> >> On Wed, Sep 4, 2013 at 9:07 AM, Tejun Heo wrote: >>> Hello, >>> >>> On Wed, Sep 04, 2013 at 08:45:33AM -0700, Anatol Pomozov wrote: I am not a

Re: Race condition between "read CFQ stats" and "block device shutdown"

2013-09-26 Thread Tejun Heo
Hello, (cc'ing linux-scsi) On Wed, Sep 25, 2013 at 01:37:51PM -0700, Anatol Pomozov wrote: > Hi > > On Wed, Sep 4, 2013 at 9:07 AM, Tejun Heo wrote: > > Hello, > > > > On Wed, Sep 04, 2013 at 08:45:33AM -0700, Anatol Pomozov wrote: > >> I am not an expect in block code, so I have a few questions

Re: esp_scsi QTAG in FAS216 (was Re: [PATCH 0/2] Experimental Amiga Zorro ESP driver)

2013-09-26 Thread Michael Schmitz
Tuomas, I might still have a copy of the manual somewhere from way back, if you haven't found it anywhere. Would be on an old disk or even hardcopy in storage, so please confirm you still need it. Cheers, Michael Am 13.09.2013 um 03:36 schrieb Tuomas Vainikka: On 09/11/2013 05:48 PM, G

Re: SCSI's heuristics for enabling WRITE SAME still need work [was: dm mpath: disable WRITE SAME if it fails]

2013-09-26 Thread Bernd Schubert
On 09/26/2013 07:39 AM, Douglas Gilbert wrote: On 13-09-25 08:44 PM, Martin K. Petersen wrote: "Bernd" == Bernd Schubert writes: Hey Bernd, Bernd> I'm afraid we have another problem. I'm currently working on to Bernd> get discard working for our LSI2008 HBAs with attached sata-SSDs Bernd> an

Re: [PATCH 39/51] DMA-API: others: use dma_set_coherent_mask()

2013-09-26 Thread Archit Taneja
Hi, On Friday 20 September 2013 04:41 AM, Russell King wrote: The correct way for a driver to specify the coherent DMA mask is not to directly access the field in the struct device, but to use dma_set_coherent_mask(). Only arch and bus code should access this member directly. Convert all direc

RE: [PATCH V2 05/10] pm80xx: Phy settings support for motherboard controller.

2013-09-26 Thread Sangeetha Gnanasekaran
Ok. We will add the device list and send updated patch. -Original Message- From: Jack Wang [mailto:xjtu...@gmail.com] Sent: Thursday, September 26, 2013 2:43 PM To: Sangeetha Gnanasekaran Cc: Anand Kumar Santhanam; linux-scsi@vger.kernel.org; Nikith Ganigarakoppal; Viswas G Subject: Re: [

Re: [PATCH V2 05/10] pm80xx: Phy settings support for motherboard controller.

2013-09-26 Thread Jack Wang
On 09/26/2013 11:04 AM, Sangeetha Gnanasekaran wrote: > Phy settings support is only for Motherboard controller. Firmware will > only initialize default values for all PHY in case of motherboard > controller. Hence, this is mandatory to add this support in all > motherboard controllers to configure

RE: [PATCH V2 08/10] pm80xx: Queue rotation logic for inbound and outbound queues

2013-09-26 Thread Sangeetha Gnanasekaran
We have increased performance by setting inbound to 16 and outbound to 4. We will send patch for it. -Original Message- From: Jack Wang [mailto:xjtu...@gmail.com] Sent: Thursday, September 26, 2013 12:38 PM To: Anand Kumar Santhanam Cc: linux-scsi@vger.kernel.org; Sangeetha Gnanasekaran;

RE: [PATCH V2 01/10] pm80xx: Device id changes to support series 8 controllers.

2013-09-26 Thread Sangeetha Gnanasekaran
We will modify and send updated patch. -Original Message- From: Jack Wang [mailto:xjtu...@gmail.com] Sent: Thursday, September 26, 2013 12:10 PM To: Anand Kumar Santhanam Cc: linux-scsi@vger.kernel.org; Sangeetha Gnanasekaran; Nikith Ganigarakoppal; Viswas G Subject: Re: [PATCH V2 01/10]

RE: [PATCH V2 09/10] pm80xx: 4G boundary fix

2013-09-26 Thread Sangeetha Gnanasekaran
We will add description and resend the updated patch. -Original Message- From: Jack Wang [mailto:xjtu...@gmail.com] Sent: Thursday, September 26, 2013 12:40 PM To: Anand Kumar Santhanam Cc: linux-scsi@vger.kernel.org; Sangeetha Gnanasekaran; Nikith Ganigarakoppal; Viswas G Subject: Re: [P

RE: [PATCH V2 06/10] pm80xx: Print SAS address of IO failed device.

2013-09-26 Thread Sangeetha Gnanasekaran
We will resend the updated patch with this change. -Original Message- From: Jack Wang [mailto:xjtu...@gmail.com] Sent: Thursday, September 26, 2013 12:33 PM To: Anand Kumar Santhanam Cc: linux-scsi@vger.kernel.org; Sangeetha Gnanasekaran; Nikith Ganigarakoppal; Viswas G Subject: Re: [PATC

RE: [PATCH V2 05/10] pm80xx: Phy settings support for motherboard controller.

2013-09-26 Thread Sangeetha Gnanasekaran
Phy settings support is only for Motherboard controller. Firmware will only initialize default values for all PHY in case of motherboard controller. Hence, this is mandatory to add this support in all motherboard controllers to configure their own settings. -Original Message- From: Jack W

Re: [PATCH V2 10/10] pm80xx: Firmware logging support

2013-09-26 Thread Jack Wang
On 09/26/2013 07:43 AM, Anand wrote: > From ab1b030500a835eacda3d86e5570366099b21311 Mon Sep 17 00:00:00 2001 > From: Anand Kumar Santhanam > Date: Wed, 4 Sep 2013 12:57:00 +0530 > Subject: [PATCH V2 10/10] pm80xx: Firmware logging support. > > Supports below logging facilities, > Inbound outboun

Re: [PATCH V2 09/10] pm80xx: 4G boundary fix

2013-09-26 Thread Jack Wang
On 09/26/2013 07:42 AM, Anand wrote: > From 4715339743d45a6ef862bc0f5d5ec404b4667f94 Mon Sep 17 00:00:00 2001 > From: Anand Kumar Santhanam > Date: Wed, 18 Sep 2013 11:14:54 +0530 > Subject: [PATCH V2 09/10] pm80xx: 4G boundary fix. > > Signed-off-by: anandkumar.santha...@pmcs.com > Please give

Re: [PATCH V2 08/10] pm80xx: Queue rotation logic for inbound and outbound queues

2013-09-26 Thread Jack Wang
On 09/26/2013 07:40 AM, Anand wrote: > From 678d8085ace7c471dc140420c41dc4ad70300d60 Mon Sep 17 00:00:00 2001 > From: Anand Kumar Santhanam > Date: Wed, 18 Sep 2013 11:12:59 +0530 > Subject: [PATCH V2 08/10] pm80xx: Queue rotation logic for inbound and > outbound queues. > > Signed-off-by: anand

Re: [PATCH V2 07/10] pm80xx: Set device state response logic fix.

2013-09-26 Thread Jack Wang
On 09/26/2013 07:39 AM, Anand wrote: > From edf018a942e0da4da12bad21e5bf0d48621a18de Mon Sep 17 00:00:00 2001 > From: Anand Kumar Santhanam > Date: Tue, 17 Sep 2013 16:58:10 +0530 > Subject: [PATCH V2 07/10] pm80xx: Set device state response logic fix. > > Signed-off-by: anandkumar.santha...@pmcs

Re: [PATCH V2 06/10] pm80xx: Print SAS address of IO failed device.

2013-09-26 Thread Jack Wang
On 09/26/2013 07:37 AM, Anand wrote: > From 5ddec5ef3033b4fee08dcdc7ba8b434425453e9d Mon Sep 17 00:00:00 2001 > From: Anand Kumar Santhanam > Date: Tue, 17 Sep 2013 16:47:21 +0530 > Subject: [PATCH V2 06/10] pm80xx: Print SAS address of IO failed device. > > Signed-off-by: anandkumar.santha...@pm

Re: [PATCH v2] ufs: adjust queue settings to PRDT limitations

2013-09-26 Thread Subhash Jadavani
Looks good to me. Reviewed-by: Subhash Jadavani On 9/25/2013 7:17 PM, Akinobu Mita wrote: The data byte count field of PRDT indicates the length of data block which is a segment of data transfer for SCSI commands. The value of this field shall have Dword granularity and the the maximum of len