Re: SCSI core patches for kernel 3.12

2013-10-02 Thread Christoph Hellwig
On Wed, Oct 02, 2013 at 03:15:20PM +, James Bottomley wrote: > And I thought the world was ending when Linus started top posting ... I think it's quite usual style if you ask a bytestander to comment on a whole mail - there's not specific context to quote in that case. > You mean apart from t

Re: SCSI core patches for kernel 3.12

2013-10-02 Thread James Bottomley
And I thought the world was ending when Linus started top posting ... On Wed, 2013-10-02 at 00:40 -0700, Christoph Hellwig wrote: > James, and comments on this? You mean apart from the fact that it's a long series which seems to self confess to providing not much value and therefore not rating it

Re: SCSI core patches for kernel 3.12

2013-10-02 Thread Christoph Hellwig
James, and comments on this? On Tue, Aug 20, 2013 at 02:05:45PM +0200, Bart Van Assche wrote: > This patch series consists of four patches that address device > removal issues and three patches that improve performance of the > SCSI mid-layer. > > The individual patches are. > 0001-Introduce-scsi

Re: SCSI core patches for kernel 3.12

2013-08-21 Thread Nicholas A. Bellinger
On Tue, 2013-08-20 at 23:48 -0700, Christoph Hellwig wrote: > On Tue, Aug 20, 2013 at 01:38:04PM -0700, Nicholas A. Bellinger wrote: > > , I only care about the performance against upstream code, so that > > would mean scsi_debug here. Typically the onus of demonstrating a > > performance improvem

Re: SCSI core patches for kernel 3.12

2013-08-20 Thread Christoph Hellwig
On Tue, Aug 20, 2013 at 01:38:04PM -0700, Nicholas A. Bellinger wrote: > , I only care about the performance against upstream code, so that > would mean scsi_debug here. Typically the onus of demonstrating a > performance improvement is on the patch submitter (eg: not the > reviewer). Do you real

Re: SCSI core patches for kernel 3.12

2013-08-20 Thread Nicholas A. Bellinger
On Tue, 2013-08-20 at 20:00 +0200, Bart Van Assche wrote: > On 08/20/13 19:04, Nicholas A. Bellinger wrote: > > On Tue, 2013-08-20 at 18:15 +0200, Bart Van Assche wrote: > >> On 08/20/13 18:11, Nicholas A. Bellinger wrote: > >>> On Tue, 2013-08-20 at 14:05 +0200, Bart Van Assche wrote: > This

Re: SCSI core patches for kernel 3.12

2013-08-20 Thread Bart Van Assche
On 08/20/13 19:04, Nicholas A. Bellinger wrote: On Tue, 2013-08-20 at 18:15 +0200, Bart Van Assche wrote: On 08/20/13 18:11, Nicholas A. Bellinger wrote: On Tue, 2013-08-20 at 14:05 +0200, Bart Van Assche wrote: This patch series consists of four patches that address device removal issues and

Re: SCSI core patches for kernel 3.12

2013-08-20 Thread Nicholas A. Bellinger
On Tue, 2013-08-20 at 18:15 +0200, Bart Van Assche wrote: > On 08/20/13 18:11, Nicholas A. Bellinger wrote: > > On Tue, 2013-08-20 at 14:05 +0200, Bart Van Assche wrote: > >> This patch series consists of four patches that address device removal > >> issues and three patches that improve performanc

Re: SCSI core patches for kernel 3.12

2013-08-20 Thread Bart Van Assche
On 08/20/13 18:11, Nicholas A. Bellinger wrote: On Tue, 2013-08-20 at 14:05 +0200, Bart Van Assche wrote: This patch series consists of four patches that address device removal issues and three patches that improve performance of the SCSI mid-layer. Perhaps it would be useful to know what the

Re: SCSI core patches for kernel 3.12

2013-08-20 Thread Nicholas A. Bellinger
On Tue, 2013-08-20 at 14:05 +0200, Bart Van Assche wrote: > This patch series consists of four patches that address device removal > issues and three patches that improve performance of the SCSI mid-layer. > Perhaps it would be useful to know what the performance improvement actually is..? Eg:

SCSI core patches for kernel 3.12

2013-08-20 Thread Bart Van Assche
This patch series consists of four patches that address device removal issues and three patches that improve performance of the SCSI mid-layer. The individual patches are. 0001-Introduce-scsi_device_being_removed.patch 0002-Rework-scsi_internal_device_unblock.patch 0003-Avoid-re-enabling-I-O-aft