Re: [PATCH] sd: fixup capacity calculation for 4k drives

2016-03-21 Thread Martin K. Petersen
> "Hannes" == Hannes Reinecke writes: Hannes> in sd_read_capacity() the sdkp->capacity field changes its Hannes> meaning: after the call to read_capacity_XX() it carries the Hannes> _unscaled_ values, making the comparison between the original Hannes> value and the new value

Re: [PATCH] megaraid_sas: fix missing { }, nullify kbuff_arr[i] only when necessary

2016-03-21 Thread Martin K. Petersen
> "Colin" == Colin King writes: Colin> From: Colin Ian King Fix missing { } Colin> on if statement, this change will nullify kbuff_arr[i] only where Colin> necessary as the code intended. Colin> detected using static analysis with smatch

Re: [PATCH] mpt3sas: Don't overreach ioc->reply_post[] during initialization

2016-03-21 Thread Martin K. Petersen
> "Calvin" == Calvin Owens writes: Calvin> In _base_make_ioc_operational(), we walk ioc->reply_queue_list Calvin> and pull a pointer out of successive elements of Calvin> ioc->reply_post[] for each entry in that list if RDPQ is Calvin> enabled. Calvin> Since the code

Re: [PATCH 1/3] aacraid: add missing curly braces

2016-03-21 Thread Martin K. Petersen
> "Arnd" == Arnd Bergmann writes: Arnd> gcc-6 warns about obviously wrong indentation for newly added code Arnd> in aac_slave_configure(): Applied to 4.6/scsi-fixes. -- Martin K. Petersen Oracle Linux Engineering -- To unsubscribe from this list: send the line

Re: [PATCHv3] scsi: disable automatic target scan

2016-03-21 Thread Laurence Oberman
Hello Hannes Please share latest scripts and an example of how you are using them. I have some scripts from last November, that you posted but I am sure they have changed. If not then I will modify them as appropriate, just let me know. I have added the patches and booted the system set to

FOR YOUR INFORMATION DEAR BENEFICIARY,

2016-03-21 Thread David Kedogo
FOR YOUR INFORMATION DEAR BENEFICIARY, Your Over-due ATM Card payment of $1.5MUSD by the UN Office have deposited to the ATM MASTER CARD OFFICE. All you have to do now is to contact the Office Manager Dr. peter Akupa Boni at: (peterakupa...@gmail.com) and Phone number: +229 61 31 07 78 , he will

Re: [BUG] mpt2sas: driver init fails on kernel >=4.2 for 9211-8i IT

2016-03-21 Thread Matthias Prager
Am 21.03.2016 um 16:52 schrieb Matthias Prager: > Hi Sreekanth, > > thanks for digging into this issue. Regarding the 4.5.0 after 4.1.20 > boot statement, I will try to express myself better: > > I first started the system with the 4.1.20 kernel. Then I issued an > 'init 6' warm-reboot and chose

Re: [BUG] mpt2sas: driver init fails on kernel >=4.2 for 9211-8i IT

2016-03-21 Thread Matthias Prager
Hi Sreekanth, thanks for digging into this issue. Regarding the 4.5.0 after 4.1.20 boot statement, I will try to express myself better: I first started the system with the 4.1.20 kernel. Then I issued an 'init 6' warm-reboot and chose to boot the 4.5.0 kernel. This procedure often works (i.e.

Re: scst_vdisk: Resize-fsync() race fails IO

2016-03-21 Thread Sebastian Parschauer
On 18.03.2016 13:08, Sebastian Parschauer wrote: > Hi, > > we are live resizing LVM LVs below SCST/SRP but we only increase the size. > > Writing to /sys/kernel/scst_tgt/devices/$DEV/resync_size with a new > device size races against fsync() from the SRP initiator so that IO is > failed.

Re: [PATCH RFC 1/2] scatterlist: add mempool based chained SG alloc/free api

2016-03-21 Thread Christoph Hellwig
On Sun, Mar 20, 2016 at 11:55:17PM -0700, Ming Lin wrote: > On Wed, 2016-03-16 at 09:23 +0100, Christoph Hellwig wrote: > > >  > > We can defintively kill this one. > > We want to support different size of pools. > How can we kill this one? > > Or did you mean we just create a single pool with

Re: [PATCH] uas: Limit qdepth at the scsi-host level

2016-03-21 Thread Christoph Hellwig
On Sat, Mar 19, 2016 at 09:59:12AM +0100, Hans de Goede wrote: > Commit 64d513ac31bd ("scsi: use host wide tags by default") causes > the scsi-core to queue more cmnds then we can handle on devices with > multiple LUNs, limit the qdepth at the scsi-host level instead of > per slave to fix this. >

Re: [PATCH] uas: Limit qdepth at the scsi-host level

2016-03-21 Thread Christoph Hellwig
On Sat, Mar 19, 2016 at 10:06:16AM -0700, James Bottomley wrote: > Help me understand this bug a bit more. Are you saying that the commit > you identify is causing the block layer to queue more commands than > you've set the per-lun limit to? In which case we have a serious > problem for more

Re: [PATCH] sd: fixup capacity calculation for 4k drives

2016-03-21 Thread Christoph Hellwig
On Mon, Mar 21, 2016 at 01:27:29PM +0100, Hannes Reinecke wrote: > in sd_read_capacity() the sdkp->capacity field changes its meaning: > after the call to read_capacity_XX() it carries the _unscaled_ values, > making the comparison between the original value and the new value > always false for

Re: [BUG] mpt2sas: driver init fails on kernel >=4.2 for 9211-8i IT

2016-03-21 Thread Sreekanth Reddy
Hi Matthias, Thanks for providing the logs. In these logs, I am not observing any such a prints which should be suspected for this issue. Can you please try once by setting mpt3sas driver's "msix_disable" module parameter to one. Also, can you please elaborate below statement for me, I am not

[PATCH] sd: fixup capacity calculation for 4k drives

2016-03-21 Thread Hannes Reinecke
in sd_read_capacity() the sdkp->capacity field changes its meaning: after the call to read_capacity_XX() it carries the _unscaled_ values, making the comparison between the original value and the new value always false for drives with a sector size != 512. So introduce a 'new_capacity' carrying

Re: [BUG] mpt2sas: driver init fails on kernel >=4.2 for 9211-8i IT

2016-03-21 Thread Matthias Prager
Hi Sreekanth, thank you for replying so quickly. Here are the logs you requested (kernel 4.5.0): > [2.083177] mpt3sas version 09.102.00.00 loaded > [2.083757] mpt2sas_cm0: mpt3sas_base_attach > [2.083956] mpt2sas_cm0: mpt3sas_base_map_resources > [2.084708] mpt2sas_cm0: 32 BIT

Re: [PATCH] mpt3sas: Don't overreach ioc->reply_post[] during initialization

2016-03-21 Thread Johannes Thumshirn
On Freitag, 18. März 2016 12:45:42 CET Calvin Owens wrote: > In _base_make_ioc_operational(), we walk ioc->reply_queue_list and pull > a pointer out of successive elements of ioc->reply_post[] for each entry > in that list if RDPQ is enabled. > > Since the code pulls the pointer for the next

Re: [PATCHv3] scsi: disable automatic target scan

2016-03-21 Thread Hannes Reinecke
On 03/21/2016 02:24 AM, Bart Van Assche wrote: > On 03/19/16 08:18, Hannes Reinecke wrote: >> On 03/18/2016 10:56 PM, Bart Van Assche wrote: >>> On 03/17/2016 12:39 AM, Hannes Reinecke wrote: On larger installations it is useful to disable automatic LUN scanning, and only add the

Re: [PATCH RFC 1/2] scatterlist: add mempool based chained SG alloc/free api

2016-03-21 Thread Ming Lin
On Wed, 2016-03-16 at 09:23 +0100, Christoph Hellwig wrote: > >  > We can defintively kill this one. We want to support different size of pools. How can we kill this one? Or did you mean we just create a single pool with size SG_CHUNK_SIZE? > > > +static __init int sg_mempool_init(void) > > +{