Re: [PATCH] scsi/ibmvscsi: add module alias for ibmvscsic

2012-07-30 Thread Benjamin Herrenschmidt
On Mon, 2012-07-30 at 21:06 +0200, Olaf Hering wrote: > > So while this would work, I do wonder however whether we could > instead > > fix it by simplifying the whole thing as follow since iSeries is now > > gone and so we don't need split backends anymore: > > > > scsi/ibmvscsi: Remove backend ab

Re: [PATCH v4 0/7] ZPODD patches

2012-07-30 Thread Aaron Lu
Hi James, Any chance of these patches get merged into 3.6? Thanks, Aaron On 07/27/2012 05:00 PM, Aaron Lu wrote: v4: Rebase on top of Linus' tree, due to this, the problem of a missing flag in v3 is gone; Add a new function scsi_autopm_put_device_autosuspend to first mark last busy for the dev

[GIT PULL] tcm_vhost: Initial merge of vhost level target fabric driver

2012-07-30 Thread Nicholas A. Bellinger
Hi Linus, Here is the PULL request for the initial merge of tcm_vhost based on RFC-v5 code with MST's ACK appended to the initial merge commit. As promised, the commit is available from two different branches for you to consider merging as for-3.6 code. The 'for-next-merge' branch based on mainli

[PATCH] tcm_vhost: Post-merge review changes requested by MST

2012-07-30 Thread Nicholas A. Bellinger
From: Nicholas Bellinger This patch contains the post RFC-v5 (post-merge) changes, this includes: - Add locking comment - Move vhost_scsi_complete_cmd ahead of TFO callbacks in order to drop forward declarations - Drop extra '!= NULL' usage in vhost_scsi_complete_cmd_work() - Change vhost_scsi

Re: [PATCH SCSI] sr: check_events() ignore GET_EVENT when TUR says otherwise

2012-07-30 Thread Markus Rathgeb
> I have 3.5 here and see the loop too, but it stops after a couple of seconds: > > first: KERNEL[15636.368034] > last: KERNEL[15638.879796] Ah, sorry then. The udev messages reported by the udev monitor reported up to 20 seconds of udev (userspace) events. I did not filter kernel reported message

Re: [PATCH SCSI] sr: check_events() ignore GET_EVENT when TUR says otherwise

2012-07-30 Thread Kay Sievers
On Mon, Jul 30, 2012 at 9:06 PM, Markus Rathgeb wrote: > Hello! > > Do you remember? > I get it again, is there something I could do? > > I attached a file that contains the "udevadm monitor" log. > > I am using a v3.5.0 linux kernel (containing some gentoo patches). > Should I test it with a vani

Re: [PATCH SCSI] sr: check_events() ignore GET_EVENT when TUR says otherwise

2012-07-30 Thread Markus Rathgeb
Hello! Do you remember? I get it again, is there something I could do? I attached a file that contains the "udevadm monitor" log. I am using a v3.5.0 linux kernel (containing some gentoo patches). Should I test it with a vanilla one? Here the versions of some packages (kernel, udev, systemd): s

Re: [PATCH] scsi/ibmvscsi: add module alias for ibmvscsic

2012-07-30 Thread Olaf Hering
On Mon, Jul 30, Benjamin Herrenschmidt wrote: > On Wed, 2012-07-18 at 18:49 +0200, o...@aepfle.de wrote: > > From: Olaf Hering > > > > The driver is named ibmvscsic, at runtime it its name is advertised as > > ibmvscsi. For this reason mkinitrd wont pickup the driver properly. > > Reported by IB

Re: [patch 1/2] [SCSI] ipr: missing unlock before a return

2012-07-30 Thread Brian King
On 07/30/2012 03:16 AM, Dan Carpenter wrote: > We recently changed the locking in this function, but this return was > missed. It needs an unlock and the IRQs need to be restored. > > Signed-off-by: Dan Carpenter > --- Thanks for catching this. Acked-by: Brian King -- Brian King Power Linux

Re: [patch 2/2] [SCSI] ipr: remove an unneeded check

2012-07-30 Thread Brian King
On 07/30/2012 03:18 AM, Dan Carpenter wrote: > "rc" is always zero here, so there is no need to check. > > Signed-off-by: Dan Carpenter Thanks! Acked-by: Brian King -- Brian King Power Linux I/O IBM Linux Technology Center -- To unsubscribe from this list: send the line "unsubscribe lin

Re: virtio(-scsi) vs. chained sg_lists (was Re: [PATCH] scsi: virtio-scsi: Fix address translation failure of HighMem pages used by sg list)

2012-07-30 Thread Boaz Harrosh
On 07/30/2012 10:12 AM, Paolo Bonzini wrote: > Il 30/07/2012 01:50, Rusty Russell ha scritto: >>> Also, being the first user of chained scatterlist doesn't exactly give >>> me warm fuzzies. >> >> We're far from the first user: they've been in the kernel for well over >> 7 years. They were introdu

[patch 2/2] [SCSI] ipr: remove an unneeded check

2012-07-30 Thread Dan Carpenter
"rc" is always zero here, so there is no need to check. Signed-off-by: Dan Carpenter diff --git a/drivers/scsi/ipr.c b/drivers/scsi/ipr.c index 7a5ccb2c..fc49f17 100644 --- a/drivers/scsi/ipr.c +++ b/drivers/scsi/ipr.c @@ -5847,7 +5847,7 @@ static int ipr_queuecommand(struct Scsi_Host *shost,

[patch 1/2] [SCSI] ipr: missing unlock before a return

2012-07-30 Thread Dan Carpenter
We recently changed the locking in this function, but this return was missed. It needs an unlock and the IRQs need to be restored. Signed-off-by: Dan Carpenter --- Applies to linux-next. diff --git a/drivers/scsi/ipr.c b/drivers/scsi/ipr.c index 07b14ba..7a5ccb2c 100644 --- a/drivers/scsi/ipr.c

Re: [PATCH v3] scsi: virtio-scsi: Fix address translation failure of HighMem pages used by sg list

2012-07-30 Thread Paolo Bonzini
Il 30/07/2012 08:25, Wang Sen ha scritto: > When using the commands below to write some data to a virtio-scsi LUN of the > QEMU guest(32-bit) with 1G physical memory(qemu -m 1024), the qemu will crash. > > # sudo mkfs.ext4 /dev/sdb (/dev/sdb is the virtio-scsi LUN.) > # sudo mount

Re: virtio(-scsi) vs. chained sg_lists (was Re: [PATCH] scsi: virtio-scsi: Fix address translation failure of HighMem pages used by sg list)

2012-07-30 Thread Paolo Bonzini
Il 30/07/2012 01:50, Rusty Russell ha scritto: >> Also, being the first user of chained scatterlist doesn't exactly give >> me warm fuzzies. > > We're far from the first user: they've been in the kernel for well over > 7 years. They were introduced for the block layer, but they tended to > ignore