Re: T10 WCE interpretation in Linux & device level access

2013-04-26 Thread Paolo Bonzini
Il 25/04/2013 03:32, Martin K. Petersen ha scritto: > I'm ok with your patch. And a strong believer in not altering the > SYNCHRONIZE CACHE behavior that's been rigorously tested in the field by > adding SYNC_NV to the mix. SYNC_NV is absolutely necessary for targets that (a) have both volatile an

Re: [PATCH RESEND^2] sd: fix infinite kernel/udev loop on non-removable Medium Not Present

2013-04-26 Thread James Bottomley
On Fri, 2013-04-26 at 11:39 -0500, Steven J. Magnani wrote: > Commit eface65c336eff420d70beb0fb6787a732e05ffb (2.6.38) altered > set_media_not_present() in a way that prevents the sd driver from > remembering that a non-removable device has reported "Medium Not Present". > This condition can occur

Re: [PATCH RESEND^2] sd: fix infinite kernel/udev loop on non-removable Medium Not Present

2013-04-26 Thread Greg KH
On Fri, Apr 26, 2013 at 11:39:33AM -0500, Steven J. Magnani wrote: > Commit eface65c336eff420d70beb0fb6787a732e05ffb (2.6.38) altered > set_media_not_present() in a way that prevents the sd driver from > remembering that a non-removable device has reported "Medium Not Present". > This condition can

Re: [PATCH v2 1/4] Wrap crc_t10dif function all to use crypto transform framework

2013-04-26 Thread Tim Chen
On Fri, 2013-04-26 at 20:52 +0800, Herbert Xu wrote: > On Thu, Apr 25, 2013 at 10:28:30AM -0700, Tim Chen wrote: > > > > @@ -51,6 +54,98 @@ static const __u16 t10_dif_crc_table[256] = { > > 0xF0D8, 0x7B6F, 0x6C01, 0xE7B6, 0x42DD, 0xC96A, 0xDE04, 0x55B3 > > }; > > > > +#ifdef CONFIG_CRYPTO_CR

[PATCH RESEND^2] sd: fix infinite kernel/udev loop on non-removable Medium Not Present

2013-04-26 Thread Steven J. Magnani
Commit eface65c336eff420d70beb0fb6787a732e05ffb (2.6.38) altered set_media_not_present() in a way that prevents the sd driver from remembering that a non-removable device has reported "Medium Not Present". This condition can occur on hotplug of a (i.e.) USB Mass Storage device whose medium is offli

Re: [PATCH RESEND 0/9] zfcp features and bugfixes for 3.10 merge window

2013-04-26 Thread James Bottomley
On Fri, 2013-04-26 at 17:44 +0200, Steffen Maier wrote: > On 04/26/2013 05:10 PM, James Bottomley wrote: > > On Fri, 2013-04-26 at 16:13 +0200, Steffen Maier wrote: > >> James, > >> > >> here is a series of zfcp features and bugfixes. > >> We would very much appreciate if you could integrate them f

Re: [PATCH RESEND 0/9] zfcp features and bugfixes for 3.10 merge window

2013-04-26 Thread Steffen Maier
On 04/26/2013 05:10 PM, James Bottomley wrote: On Fri, 2013-04-26 at 16:13 +0200, Steffen Maier wrote: James, here is a series of zfcp features and bugfixes. We would very much appreciate if you could integrate them for the upcoming merge window preparing kernel v3.10. The patches apply on top

[PATCH RESEND 9/9] zfcp: status read buffers on first adapter open with link down

2013-04-26 Thread Steffen Maier
Commit 64deb6efdc5504ce97b5c1c6f281fffbc150bd93 "[SCSI] zfcp: Use status_read_buf_num provided by FCP channel" started using a value returned by the channel but only evaluated the value if the fabric link is up. Commit 8d88cf3f3b9af4713642caeb221b6d6a42019001 "[SCSI] zfcp: Update status read mempoo

[PATCH RESEND 6/9] zfcp: block queue limits with data router

2013-04-26 Thread Steffen Maier
Commit 86a9668a8d29ea711613e1cb37efa68e7c4db564 "[SCSI] zfcp: support for hardware data router" reduced the initial block queue limits in the scsi_host_template to the absolute minimum and adjusted them later on. However, the adjustment was too late for the BSG devices of Scsi_Host and fc_host. Th

[PATCH RESEND 5/9] zfcp: fix adapter (re)open recovery while link to SAN is down

2013-04-26 Thread Steffen Maier
From: Daniel Hansel FCP device remains in status ERP_FAILED when device is switched online or adapter recovery is triggered while link to SAN is down. When Exchange Configuration Data command returns the FSF status FSF_EXCHANGE_CONFIG_DATA_INCOMPLETE it aborts the exchange process. The only ret

RE: [nhor...@tuxdriver.com: [PATCH] libcxgbi: supress warning when we request to much space from kmalloc]

2013-04-26 Thread Karen Xie
-Original Message- From: Neil Horman [mailto:nhor...@tuxdriver.com] Sent: Friday, April 26, 2013 4:02 AM To: Karen Xie Subject: [nhor...@tuxdriver.com: [PATCH] libcxgbi: supress warning when we request to much space from kmalloc] The patch looks fine to me. Thanks, Karen - Forwarde

Re: [PATCH RESEND 0/9] zfcp features and bugfixes for 3.10 merge window

2013-04-26 Thread James Bottomley
On Fri, 2013-04-26 at 16:13 +0200, Steffen Maier wrote: > James, > > here is a series of zfcp features and bugfixes. > We would very much appreciate if you could integrate them for the > upcoming merge window preparing kernel v3.10. > The patches apply on top of the current misc branch of your scs

PATCH] scsi_debug: Fix endianess in partition table

2013-04-26 Thread Martin Peschke
James, could you pick up this fix, preferably for 3.10, please? I am not the only one you has run into the issue: https://nazar.karan.org/blob/distro! parted.git/8780c767126938173f49dfbcd4360813932f7756/SOURCES! disable-t9020.patch Thanks, Martin Both start_sect and nr_sects in struct partiti

[PATCH RESEND 7/9] zfcp: module parameter dbflevel for early debugging

2013-04-26 Thread Steffen Maier
So far, we could only increase the s390dbf log level after an FCP device has been initially set online for it to create the dbf entries required to adjust the level. Introduce zfcp.dbflevel as counterpart to the already existing zfcp.dbfsize to enable debugging of e.g. setting an FCP device online

[PATCH RESEND 1/9] zfcp: cfdc fops add owner

2013-04-26 Thread Steffen Maier
From: Sebastian Ott Set the owner member of zfcp_cfdc_fops, to ensure that the caller of these functions holds a module reference. Signed-off-by: Sebastian Ott Signed-off-by: Steffen Maier --- drivers/s390/scsi/zfcp_cfdc.c |2 ++ 1 file changed, 2 insertions(+) --- a/drivers/s390/scsi/zf

[PATCH RESEND 3/9] zfcp: cleanup unit sysfs attribute usage

2013-04-26 Thread Steffen Maier
From: Sebastian Ott Let the driver core handle device attribute creation and removal. This will simplify the code and eliminates races between attribute availability and userspace notification via uevents. Reviewed-by: Steffen Maier Signed-off-by: Sebastian Ott Signed-off-by: Martin Schwidefsk

[PATCH RESEND 4/9] zfcp: remove unused device_unregister wrapper

2013-04-26 Thread Steffen Maier
From: Sebastian Ott Remove the now unused function zfcp_device_unregister since all users have been converted to use device_unregister directly. Reviewed-by: Steffen Maier Signed-off-by: Sebastian Ott Signed-off-by: Martin Schwidefsky Signed-off-by: Steffen Maier --- drivers/s390/scsi/zfcp_

[PATCH RESEND 8/9] zfcp: remove access control tables interface

2013-04-26 Thread Steffen Maier
From: Martin Peschke This patch removes an interface that was used to manage access control tables within the HBA. The patch consequently removes the handling for conditions related to those access control tables, too. That initiator-based access control feature was only needed until the introdu

[PATCH RESEND 2/9] zfcp: cleanup port sysfs attribute usage

2013-04-26 Thread Steffen Maier
From: Sebastian Ott Let the driver core handle device attribute creation and removal. This will simplify the code and eliminates races between attribute availability and userspace notification via uevents. Reviewed-by: Steffen Maier Signed-off-by: Sebastian Ott Signed-off-by: Martin Schwidefsk

[PATCH RESEND 0/9] zfcp features and bugfixes for 3.10 merge window

2013-04-26 Thread Steffen Maier
James, here is a series of zfcp features and bugfixes. We would very much appreciate if you could integrate them for the upcoming merge window preparing kernel v3.10. The patches apply on top of the current misc branch of your scsi.git. Apparently, I did something wrong on git-send-email so the m

Re: [PATCH 2/2] dm mpath: attach scsi_dh during table resume

2013-04-26 Thread Mike Snitzer
On Fri, Apr 26 2013 at 2:05am -0400, Hannes Reinecke wrote: > On 04/25/2013 05:31 PM, Mike Snitzer wrote: > > On Thu, Apr 25 2013 at 10:50am -0400, > > Mikulas Patocka wrote: > > > >> > >> > >> On Thu, 25 Apr 2013, Mike Snitzer wrote: > >> > >>> On Thu, Apr 25 2013 at 9:48am -0400, > >>> Miku

Re: [PATCH v2 1/4] Wrap crc_t10dif function all to use crypto transform framework

2013-04-26 Thread Herbert Xu
On Thu, Apr 25, 2013 at 10:28:30AM -0700, Tim Chen wrote: > > @@ -51,6 +54,98 @@ static const __u16 t10_dif_crc_table[256] = { > 0xF0D8, 0x7B6F, 0x6C01, 0xE7B6, 0x42DD, 0xC96A, 0xDE04, 0x55B3 > }; > > +#ifdef CONFIG_CRYPTO_CRCT10DIF No need for ifdefs, just make it unconditional like the

Re: [PATCH v2 0/9] fix max discard sectors limit

2013-04-26 Thread Hannes Reinecke
On 04/20/2013 09:50 PM, James Bottomley wrote: > On Sat, 2013-04-20 at 01:40 +0900, Namjae Jeon wrote: >> From: Namjae Jeon >> >> linux-v3.8-rc1 and later support for plug for blkdev_issue_discard with >> commit 0cfbcafcae8b7364b5fa96c2b26ccde7a3a296a9 >> (block: add plug for blkdev_issue_discard