Re: [PATCH] ata: do not hard code limit in ata_set_lba_range_entries()

2016-08-23 Thread Shaun Tancheff
On Mon, Aug 22, 2016 at 3:53 PM, Tom Yan wrote: > On 22 August 2016 at 20:32, Shaun Tancheff wrote: >> On Mon, Aug 22, 2016 at 3:07 PM, Tom Yan wrote: >>> I don't see how that's possible. count / n_block will always be >>> smaller than 65535 * ATA_MAX_TRIM_RNUM(64) = 4194240. Not to mention >>>

Re: [PATCH] ata: do not hard code limit in ata_set_lba_range_entries()

2016-08-23 Thread Tom Yan
On 23 August 2016 at 07:30, Shaun Tancheff wrote: > > But the "humanized" limit is the one you just added and proceeded to > change ata_set_lba_range_entries(). You changed from a buffer size > to use "num" instead and now you want to remove the protection > entirely? I am not sure about what yo

Re: [PATCH] ata: do not hard code limit in ata_set_lba_range_entries()

2016-08-23 Thread Tom Yan
On 23 August 2016 at 08:37, Tom Yan wrote: > On 23 August 2016 at 07:30, Shaun Tancheff wrote: >> >> But the "humanized" limit is the one you just added and proceeded to >> change ata_set_lba_range_entries(). You changed from a buffer size >> to use "num" instead and now you want to remove the p

Re: [PATCH] ata: do not hard code limit in ata_set_lba_range_entries()

2016-08-23 Thread Tom Yan
On 23 August 2016 at 09:18, Shaun Tancheff wrote: > On Tue, Aug 23, 2016 at 3:37 AM, Tom Yan wrote: >> On 23 August 2016 at 07:30, Shaun Tancheff wrote: > >> If we really want/need to avoid hitting some real buffer limit (e.g. >> maximum length of scatter/gather list?), then we should in some wa

RE: Observing Softlockup's while running heavy IOs

2016-08-23 Thread Kashyap Desai
> -Original Message- > From: Elliott, Robert (Persistent Memory) [mailto:elli...@hpe.com] > Sent: Saturday, August 20, 2016 2:58 AM > To: Sreekanth Reddy > Cc: linux-scsi@vger.kernel.org; linux-ker...@vger.kernel.org; > irqbala...@lists.infradead.org; Kashyap Desai; Sathya Prakash Veerichet

Re: [PATCH] ata: do not hard code limit in ata_set_lba_range_entries()

2016-08-23 Thread Shaun Tancheff
On Tue, Aug 23, 2016 at 3:37 AM, Tom Yan wrote: > On 23 August 2016 at 07:30, Shaun Tancheff wrote: > If we really want/need to avoid hitting some real buffer limit (e.g. > maximum length of scatter/gather list?), then we should in some way > check n_block against that. If it is too large we the

Re: [PATCH] ata: do not hard code limit in ata_set_lba_range_entries()

2016-08-23 Thread Tom Yan
Wait a minute. I think you missed or misunderstood something when you listen to someone's opinion in that we should switch to sglist buffer. I think the danger people referred to is exactly what is revealed when the ugly code is removed in this commit (it doesn't mean that the code should be kept t

Re: [PATCH] ata: do not hard code limit in ata_set_lba_range_entries()

2016-08-23 Thread Shaun Tancheff
On Tue, Aug 23, 2016 at 5:17 AM, Tom Yan wrote: > Wait a minute. I think you missed or misunderstood something when you > listen to someone's opinion in that we should switch to sglist buffer. No, I think I can trust Christoph Hellwig > I think the danger people referred to is exactly what is r

Re: [PATCH] ata: do not hard code limit in ata_set_lba_range_entries()

2016-08-23 Thread Tom Yan
On 23 August 2016 at 10:45, Shaun Tancheff wrote: > On Tue, Aug 23, 2016 at 5:17 AM, Tom Yan wrote: >> Wait a minute. I think you missed or misunderstood something when you >> listen to someone's opinion in that we should switch to sglist buffer. > > No, I think I can trust Christoph Hellwig I

Bestätigung deiner Email Adresse

2016-08-23 Thread Newsletter
Hallo! Wir freuen uns, dass du dich für unseren kostenlosen Newsletter entschieden hast. Ab deiner Eintragung bekommst du von uns regelmäßig, komplett kostenfrei, alle aktuellen News zu dem von dir gewählten Thema auf deine E-Mailadresse zugestellt. Schließe deine Eintragung ab indem du au

Re: [GIT PULL] [PATCH v4 00/26] Delete CURRENT_TIME and CURRENT_TIME_SEC macros

2016-08-23 Thread Arnd Bergmann
On Monday, August 15, 2016 6:23:12 PM CEST Greg KH wrote: > On Sat, Aug 13, 2016 at 03:48:12PM -0700, Deepa Dinamani wrote: > > The series is aimed at getting rid of CURRENT_TIME and CURRENT_TIME_SEC > > macros. > > The macros are not y2038 safe. There is no plan to transition them into > > being

Re: [PATCH 5/6] target/tcm_fc: Update debugging statements to match libfc usage

2016-08-23 Thread Bart Van Assche
On 08/22/2016 01:54 AM, Hannes Reinecke wrote: + char *reason = "no session created"; Please use const char* instead of char* in the above declaration. Thanks, Bart. -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majord...@vger.kerne

Re: Oops when completing request on the wrong queue

2016-08-23 Thread Gabriel Krisman Bertazi
Gabriel Krisman Bertazi writes: >> Can you share what you ran to online/offline CPUs? I can't reproduce >> this here. > > I was using the ppc64_cpu tool, which shouldn't do nothing more than > write to sysfs. but I just reproduced it with the script below. > > Note that this is ppc64le. I don't

Re: Oops when completing request on the wrong queue

2016-08-23 Thread Jens Axboe
On 08/23/2016 02:54 PM, Gabriel Krisman Bertazi wrote: Gabriel Krisman Bertazi writes: Can you share what you ran to online/offline CPUs? I can't reproduce this here. I was using the ppc64_cpu tool, which shouldn't do nothing more than write to sysfs. but I just reproduced it with the scrip

Re: Oops when completing request on the wrong queue

2016-08-23 Thread Jens Axboe
On 08/23/2016 03:11 PM, Jens Axboe wrote: On 08/23/2016 02:54 PM, Gabriel Krisman Bertazi wrote: Gabriel Krisman Bertazi writes: Can you share what you ran to online/offline CPUs? I can't reproduce this here. I was using the ppc64_cpu tool, which shouldn't do nothing more than write to sysf

Re: Oops when completing request on the wrong queue

2016-08-23 Thread Keith Busch
On Tue, Aug 23, 2016 at 03:14:23PM -0600, Jens Axboe wrote: > On 08/23/2016 03:11 PM, Jens Axboe wrote: > >My workload looks similar to yours, in that it's high depth and with a > >lot of jobs to keep most CPUs loaded. My bash script is different than > >yours, I'll try that and see if it helps her

Re: linux-next: Tree for Jun 6 (scsi_debug.c)

2016-08-23 Thread Masanari Iida
This one still exist on linus's tree (as of v4.8-rc3). CALLscripts/checksyscalls.sh CHK kernel/config_data.h Building modules, stage 2. MODPOST 720 modules ERROR: "ip_compute_csum" [drivers/scsi/scsi_debug.ko] undefined! scripts/Makefile.modpost:91: recipe for target '__modpost' fai

Re: [PATCH 4/6] cxlflash: Transition to application close model

2016-08-23 Thread Martin K. Petersen
> "Matthew" == Matthew R Ochs writes: Matthew> Caching the adapter file descriptor and performing a close on Matthew> behalf of an application is a poor design. This is due to the Matthew> fact that once a file descriptor in installed, it is free to be Matthew> altered without the knowledge o

Re: [PATCH v2] fcoe: provide translation table between Ethernet and FC port speeds

2016-08-23 Thread Martin K. Petersen
> "Johannes" == Johannes Thumshirn writes: Johannes> Provide a translation table between Ethernet and FC port Johannes> speeds so odd speeds (from a Ethernet POV) like 8 Gbit are Johannes> correctly mapped to sysfs and open-fcoe's fcoeadm. Applied to 4.9/scsi-queue. -- Martin K. Petersen

Re: [PATCH] megaraid_sas: Use memdup_user() rather than duplicating its implementation

2016-08-23 Thread Martin K. Petersen
> "SF" == SF Markus Elfring writes: SF> Reuse existing functionality from memdup_user() instead of keeping SF> duplicate source code. Applied to 4.9/scsi-queue. -- Martin K. Petersen Oracle Linux Engineering -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in t

Re: [PATCH v2 00/29] be2iscsi: driver update 11.2.0.0

2016-08-23 Thread Martin K. Petersen
> "Jitendra" == Jitendra Bhivare writes: Jitendra Bhivare (29): be2iscsi: Fix to use correct configuration values be2iscsi: Replace _bh version for mcc_lock spinlock be2iscsi: Reduce driver load/unload time be2iscsi: Set and return right iface v4/v6 states be2iscsi: Fix gateway APIs

Re: [PATCH v2 2/4] On Discard either do Reset WP or Write Same

2016-08-23 Thread Shaun Tancheff
On Mon, Aug 22, 2016 at 8:25 PM, Damien Le Moal wrote: > > Shaun, > > On 8/23/16 09:22, Shaun Tancheff wrote: >> On Mon, Aug 22, 2016 at 6:57 PM, Damien Le Moal >> wrote: >> Also you may note that in my patch to get Host Aware working >> with the zone cache I do not include the runt zone in the