On 13/12/2018 16:17, Christoph Hellwig wrote:
> There is no such limitation in the protocol or implementation, so
> remove it.
>
> Signed-off-by: Christoph Hellwig
Reviewed-by: Juergen Gross
Juergen
with
> "fall through - Missed the backend's Closing state", which
> contains the "fall through" annotation at the beginnig of
> the code comment, which is what GCC is expecting to find.
>
> Signed-off-by: Gustavo A. R. Silva
Reviewed-by: Juergen Gross
Juergen
On 21/11/2018 10:41, Christoph Hellwig wrote:
> Hi all,
>
> you in the To list maintain or wrote SCSI drivers that set the
> DISABLE_CLUSTERING flag, which basically disable merges of any
> bio segments. We already have the actual max_segment size limit
> to say which length a segment should have
On 24/05/17 02:34, Bart Van Assche wrote:
> Since the SCSI core zeroes driver-private command data, remove
> that code from the xen-scsifront driver.
>
> Signed-off-by: Bart Van Assche
> Cc: Juergen Gross
> Cc: xen-de...@lists.xenproject.org
Reviewed-by: Juergen Gross
Thanks,
Juergen
On 22/04/17 03:21, Geliang Tang wrote:
> Use offset_in_page() macro instead of open-coding.
>
> Signed-off-by: Geliang Tang
Pushed to xen/tip for-linus-4.12
Thanks,
Juergen
On 25/04/17 00:15, Martin K. Petersen wrote:
>
> Juergen,
>
>> On 22/04/17 03:21, Geliang Tang wrote:
>>> Use offset_in_page() macro instead of open-coding.
>>>
>>> Signed-off-by: Geliang Tang
>>
>> Reviewed-by: Juergen Gross
>
> T
On 22/04/17 03:21, Geliang Tang wrote:
> Use offset_in_page() macro instead of open-coding.
>
> Signed-off-by: Geliang Tang
Reviewed-by: Juergen Gross
Thanks,
Juergen
On 02/12/16 07:15, Juergen Gross wrote:
> Instead of requesting a new slot on the ring to the backend early, do
> so only after all has been setup for the request to be sent. This
> makes error handling easier as we don't need to undo the request id
> allocation and rin
On 05/12/16 21:53, Dan Carpenter wrote:
> On Mon, Nov 21, 2016 at 07:01:36AM +0100, Juergen Gross wrote:
>> On 19/11/16 19:22, Quentin Lambert wrote:
>>> Most error branches following the call to kmalloc contain
>>> a call to kfree. This patch add these calls
On 05/12/16 16:32, Boris Ostrovsky wrote:
> On 12/02/2016 01:15 AM, Juergen Gross wrote:
>>
>> -static struct vscsiif_request *scsifront_pre_req(struct vscsifrnt_info
>> *info)
>> +static int scsifront_do_request(struct vscsifrnt_info *info,
>> +
Instead of requesting a new slot on the ring to the backend early, do
so only after all has been setup for the request to be sent. This
makes error handling easier as we don't need to undo the request id
allocation and ring slot allocation.
Suggested-by: Jan Beulich
Signed-off-by: Juergen
Instead of requesting a new slot on the ring to the backend early, do
so only after all has been setup for the request to be sent. This
makes error handling easier as we don't need to undo the request id
allocation and ring slot allocation.
Suggested-by: Jan Beulich
Signed-off-by: Juergen
Instead of requesting a new slot on the ring to the backend early, do
so only after all has been setup for the request to be sent. This
makes error handling easier as we don't need to undo the request id
allocation and ring slot allocation.
Suggested-by: Jan Beulich
Signed-off-by: Juergen
On 29/11/16 12:40, Jan Beulich wrote:
On 29.11.16 at 12:19, wrote:
>> On 29/11/16 12:14, Jan Beulich wrote:
>> On 29.11.16 at 11:50, wrote:
--- a/drivers/scsi/xen-scsifront.c
+++ b/drivers/scsi/xen-scsifront.c
@@ -184,8 +184,6 @@ static struct vscsiif_request
*scsifr
On 29/11/16 12:28, David Vrabel wrote:
> On 29/11/16 11:19, Juergen Gross wrote:
>> On 29/11/16 12:14, Jan Beulich wrote:
>>>>>> On 29.11.16 at 11:50, wrote:
>>>> --- a/drivers/scsi/xen-scsifront.c
>>>> +++ b/drivers/scsi/xen-scsifront.c
>
On 29/11/16 12:14, Jan Beulich wrote:
On 29.11.16 at 11:50, wrote:
>> --- a/drivers/scsi/xen-scsifront.c
>> +++ b/drivers/scsi/xen-scsifront.c
>> @@ -184,8 +184,6 @@ static struct vscsiif_request *scsifront_pre_req(struct
>> vscsifrnt_info *info)
>>
>> ring_req = RING_GET_REQUEST(&(in
st id in case of an error.
Signed-off-by: Juergen Gross
---
In case accepted I'll take this patch through the Xen tree as it is based
on another patch by lambert.quen...@gmail.com which is going through Xen, too.
---
drivers/scsi/xen-scsifront.c | 5 +++--
1 file changed, 3 insertions(+), 2
On 19/11/16 19:22, Quentin Lambert wrote:
> Most error branches following the call to kmalloc contain
> a call to kfree. This patch add these calls where they are
> missing.
>
> This issue was found with Hector.
>
> Signed-off-by: Quentin Lambert
Applied to xen/tip.git for-linus-4.10
Juergen
On 22/11/16 04:40, Martin K. Petersen wrote:
>>>>>> "Juergen" == Juergen Gross writes:
>
> Juergen,
>
> Juergen> On 19/11/16 19:22, Quentin Lambert wrote:
>>> Most error branches following the call to kmalloc contain a call to
>>
s will need some more work, I'll do a
follow-on patch.
Reviewed-by: Juergen Gross
>
> ---
> drivers/scsi/xen-scsifront.c |1 +
> 1 file changed, 1 insertion(+)
>
> --- a/drivers/scsi/xen-scsifront.c
> +++ b/drivers/scsi/xen-scsifront.c
> @@ -627,6 +627,7 @@ stati
gned-off-by: Markus Elfring
Reviewed-by: Juergen Gross
Juergen
--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
using the Coccinelle software.
>
> Signed-off-by: Markus Elfring
Even if already given, here it is again:
Reviewed-by: Juergen Gross
Juergen
--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majord...@vger.ke
en if already given, here it is again:
Reviewed-by: Juergen Gross
Juergen
--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
On 20/07/16 07:10, SF Markus Elfring wrote:
> @@ -606,7 +606,7 @@ static void scsiback_device_action(struct
> vscsibk_pend *pending_req,
> tmr = kzalloc(sizeof(struct scsiback_tmr), GFP_KERNEL);
> if (!tmr) {
> target_put_sess_cmd(se_cmd);
> - goto err
On 19/07/16 16:56, SF Markus Elfring wrote:
>>> @@ -606,7 +606,7 @@ static void scsiback_device_action(struct vscsibk_pend
>>> *pending_req,
>>> tmr = kzalloc(sizeof(struct scsiback_tmr), GFP_KERNEL);
>>> if (!tmr) {
>>> target_put_sess_cmd(se_cmd);
>>> - goto err;
>>
On 16/07/16 22:23, SF Markus Elfring wrote:
> From: Markus Elfring
> Date: Sat, 16 Jul 2016 21:42:42 +0200
>
> The kfree() function was called in one case by the
> scsiback_device_action() function during error handling
> even if the passed variable "tmr" contained a null pointer.
>
> Adjust jum
On 16/07/16 22:24, SF Markus Elfring wrote:
> From: Markus Elfring
> Date: Sat, 16 Jul 2016 21:55:01 +0200
>
> Pass the constant "FAILED" in a function call directly instead of
> using an intialisation for a local variable.
>
> Signed-off-by: Markus Elfri
using the Coccinelle software.
>
> Signed-off-by: Markus Elfring
Reviewed-by: Juergen Gross
Juergen
--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
On 12/07/16 02:36, Bin Wu wrote:
> During scsi command queueing or exception handling, if prepairing
> fails, we need to reclaim the failed request. Otherwise, the garbage
> request will be pushed into the ring for the backend to work.
>
> Signed-off-by: Bin Wu
Reviewed-by
On 11/07/16 11:50, David Vrabel wrote:
> On 11/07/16 10:33, Juergen Gross wrote:
>> On 11/07/16 04:51, Bin Wu wrote:
>>> During scsi command queueing, if mapping data fails, we need to
>>> reclaim the failed request. Otherwise, the garbage request will
>>> be
On 11/07/16 04:51, Bin Wu wrote:
> During scsi command queueing, if mapping data fails, we need to
> reclaim the failed request. Otherwise, the garbage request will
> be pushed into the ring for the backend to work.
Well spotted. There is another instance of this problem in
scsifront_action_handle
On 07/07/16 10:01, Jan Beulich wrote:
> Only a positive return value indicates success.
>
> Signed-off-by: Jan Beulich
Acked-by: Juergen Gross
> ---
> drivers/xen/xen-scsiback.c |4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> --- 4.7-rc6-xenbus_sc
On 07/07/16 10:01, Jan Beulich wrote:
> Only a positive return value indicates success.
>
> Signed-off-by: Jan Beulich
Acked-by: Juergen Gross
> ---
> drivers/scsi/xen-scsifront.c |4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> --- 4.7-rc6-xenbus
patches and modified patch to check feature_flush/fua bits.
>
> Signed-off-by: Mike Christie
> Reviewed-by: Hannes Reinecke
> Cc: Juergen Gross
Acked-by: Juergen Gross
Juergen
--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to ma
On 15/04/16 12:40, mchri...@redhat.com wrote:
> From: Mike Christie
>
> The last patch added a REQ_OP_FLUSH for request_fn drivers
> and the next patch renames REQ_FLUSH to REQ_PREFLUSH which
> will be used by file systems and make_request_fn drivers so
> they can send a write/flush combo.
>
> T
On 03/02/16 06:03, Nicholas A. Bellinger wrote:
> Hi Juergen,
>
> On Tue, 2016-02-02 at 17:31 +0100, Juergen Gross wrote:
>> On 30/01/16 08:05, Nicholas A. Bellinger wrote:
>>> From: Nicholas Bellinger
>>>
>>> Cc: Juergen Gross
>>> Cc: Hanne
On 30/01/16 08:05, Nicholas A. Bellinger wrote:
> From: Nicholas Bellinger
>
> Cc: Juergen Gross
> Cc: Hannes Reinecke
> Cc: David Vrabel
> Signed-off-by: Nicholas Bellinger
Sorry, with your patches applied xen-scsiback isn't working any more.
I've tried multiple
On 27/01/16 07:28, Nicholas A. Bellinger wrote:
> On Tue, 2016-01-26 at 10:45 +0100, Juergen Gross wrote:
>> On 25/01/16 09:11, Nicholas A. Bellinger wrote:
>>> From: Nicholas Bellinger
>>>
>>> Cc: Juergen Gross
>>> Cc: Hannes Reinecke
>>> C
On 25/01/16 09:11, Nicholas A. Bellinger wrote:
> From: Nicholas Bellinger
>
> Cc: Juergen Gross
> Cc: Hannes Reinecke
> Cc: David Vrabel
> Signed-off-by: Nicholas Bellinger
> ---
> drivers/xen/xen-scsiback.c | 53
> +++--
On 25/01/16 09:10, Nicholas A. Bellinger wrote:
> From: Christoph Hellwig
>
> Cc: Christoph Hellwig
> Cc: Hannes Reinecke
> Cc: Chris Boot
> Cc: Sebastian Andrzej Siewior
> Cc: Andrzej Pietrasiewicz
> Cc: Juergen Gross
> Signed-off-by: Nicholas Bellinger
>
On 25/01/16 09:11, Nicholas A. Bellinger wrote:
> From: Nicholas Bellinger
>
> Cc: Juergen Gross
> Cc: Hannes Reinecke
> Cc: David Vrabel
> Signed-off-by: Nicholas Bellinger
> ---
> drivers/xen/xen-scsiback.c | 163
>
On 25/01/16 09:11, Nicholas A. Bellinger wrote:
> From: Nicholas Bellinger
>
> Cc: Juergen Gross
> Cc: Hannes Reinecke
> Cc: David Vrabel
> Signed-off-by: Nicholas Bellinger
Could you please rebase to current version? The patch won't apply.
Juergen
--
To unsubscri
On 22/01/16 13:34, Wei Liu wrote:
> The comment at the beginning of the file is the canonical source of
> licenses for this module. Currently it contains GPL and MIT license.
> Fix the code to reflect the reality.
>
> Signed-off-by: Wei Liu
Acked-by: Juergen Gross
> ---
Reviewed-by: Juergen Gross
---
drivers/xen/xen-scsiback.c | 75 ++
1 file changed, 36 insertions(+), 39 deletions(-)
diff --git a/drivers/xen/xen-scsiback.c b/drivers/xen/xen-scsiback.c
index 9faca6a..d0b0bc5 100644
--- a/drivers/xen/xen
On 03/06/2015 09:34 PM, Tao Chen wrote:
Add the {xen-pvscsi: } prefix in pr_fmt and remove DPRINTK, then
replace all DPRINTK with pr_debug.
Also fixed up some comments just as eliminate redundant whitespace
and format the code.
These will make the code easier to read.
Signed-off-by: Tao Chen
addressed you can have my:
Acked-by: Juergen Gross
Signed-off-by: Tao Chen
---
drivers/xen/xen-scsiback.c | 67 +++---
1 file changed, 34 insertions(+), 33 deletions(-)
diff --git a/drivers/xen/xen-scsiback.c b/drivers/xen/xen-scsiback.c
index 9faca6
On 02/07/2015 04:31 AM, Rudy Zhang wrote:
From: Tao Chen
Signed-off-by: Tao Chen
Are some white space fixes in comments really worth a patch?
Juergen
---
drivers/xen/xen-scsiback.c | 16
1 file changed, 8 insertions(+), 8 deletions(-)
diff --git a/drivers/xen/xen-scsi
callers which check the return value must let
'err' be 0.
And also need change the related comments for xenbus_switch_state().
Signed-off-by: Chen Gang
Acked-by: Juergen Gross
---
drivers/block/xen-blkback/xenbus.c | 9 ++---
drivers/net/xen-netback/xenbus.c | 5 +---
The pvscsi frontend driver requires the xenbus frontend driver. Reflect
this in Kconfig.
Signed-off-by: Juergen Gross
---
drivers/scsi/Kconfig | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/scsi/Kconfig b/drivers/scsi/Kconfig
index 9130df1..ff62dc1 100644
--- a/drivers/scsi/Kconfig
ifficult. :-)
Reviewed-by: Juergen Gross
Thanks for spotting this.
Juergen
---
drivers/scsi/xen-scsifront.c |3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/scsi/xen-scsifront.c b/drivers/scsi/xen-scsifront.c
index 7e88659..cc14c8d 100644
--- a/drivers/scsi/xen-scsifront
enter
Reviewed-by: Juergen Gross
Thanks,
Juergen
diff --git a/drivers/xen/xen-scsiback.c b/drivers/xen/xen-scsiback.c
index 7b565632..ad11258 100644
--- a/drivers/xen/xen-scsiback.c
+++ b/drivers/xen/xen-scsiback.c
@@ -1885,13 +1885,14 @@ scsiback_make_tpg(struct se_wwn
On 09/08/2014 01:15 PM, Dan Carpenter wrote:
This function is only called with a spin_lock held and IRQs disabled.
The allocation is not allowed to sleep and NOIO is not sufficient, it
has to be ATOMIC.
Signed-off-by: Dan Carpenter
Reviewed-by: Juergen Gross
Thanks,
Juergen
diff --git
style guide
- some minor code simplifications
- replace constants with defines
- remove not used defines
- add support for larger SG lists by putting them in a granted page
Signed-off-by: Juergen Gross
Xen related parts
Acked-by: David Vrabel
---
drivers/scsi/Kconfig |9 +
drivers
emulation, now handled by target core infrastructure
Signed-off-by: Juergen Gross
Xen related parts
Acked-by: David Vrabel
Reviewed-by: Nicholas Bellinger
---
drivers/xen/Kconfig|9 +
drivers/xen/Makefile |1 +
drivers/xen/xen-scsiback.c | 2124
Add myself as maintainer for the Xen pvSCSI stuff.
Signed-off-by: Juergen Gross
Acked-by: Konrad Rzeszutek Wilk
---
MAINTAINERS | 9 +
1 file changed, 9 insertions(+)
diff --git a/MAINTAINERS b/MAINTAINERS
index 1ff06de..0ae6238 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -10128,6
support for larger SG lists by putting them in grant page(s)
- add command abort capability
Juergen Gross (5):
xen/events: support threaded irqs for interdomain event channels
Add XEN pvSCSI protocol description
Introduce xen-scsifront module
Introduce XEN scsiback module
add xen pvscsi
for larger SG-lists by putting them in an own granted page
- remove stale definitions
Signed-off-by: Juergen Gross
---
include/xen/interface/io/vscsiif.h | 229 +
1 file changed, 229 insertions(+)
create mode 100644 include/xen/interface/io/vscsiif.h
diff
Export bind_interdomain_evtchn_to_irq() so drivers can use threaded
interrupt handlers with:
irq = bind_interdomain_evtchn_to_irq(remote_dom, remote_port);
if (irq < 0)
/* error */
ret = request_threaded_irq(...);
Signed-off-by: Juergen Gross
Acked-by: David Vrabel
Acked-by: Kon
On 08/26/2014 07:12 PM, David Vrabel wrote:
On 26/08/14 17:37, James Bottomley wrote:
On Tue, 2014-08-26 at 16:23 +0200, Juergen Gross wrote:
On 08/26/2014 04:14 PM, David Vrabel wrote:
On 18/08/14 10:31, jgr...@suse.com wrote:
From: Juergen Gross
Add myself as maintainer for the Xen
On 08/26/2014 04:14 PM, David Vrabel wrote:
On 18/08/14 10:31, jgr...@suse.com wrote:
From: Juergen Gross
Add myself as maintainer for the Xen pvSCSI stuff.
Signed-off-by: Juergen Gross
---
MAINTAINERS | 8
1 file changed, 8 insertions(+)
diff --git a/MAINTAINERS b/MAINTAINERS
On 08/23/2014 12:25 AM, Konrad Rzeszutek Wilk wrote:
--- /dev/null
+++ b/drivers/scsi/xen-scsifront.c
@@ -0,0 +1,1017 @@
+/*
+ * Xen SCSI frontend driver
+ *
+ * Copyright (c) 2008, FUJITSU Limited
+ *
+ * This program is free software; you can redistribute it and/or
+ * modify it under the terms
On 08/21/2014 09:26 PM, Konrad Rzeszutek Wilk wrote:
On Wed, Aug 20, 2014 at 04:01:57PM +0200, Juergen Gross wrote:
On 08/20/2014 03:25 PM, Konrad Rzeszutek Wilk wrote:
On Mon, Aug 18, 2014 at 11:31:47AM +0200, jgr...@suse.com wrote:
...
+struct vscsiif_request {
+ uint16_t rqid
On 08/20/2014 03:25 PM, Konrad Rzeszutek Wilk wrote:
On Mon, Aug 18, 2014 at 11:31:47AM +0200, jgr...@suse.com wrote:
From: Juergen Gross
Add the definition of pvSCSI protocol used between the pvSCSI frontend in a
XEN domU and the pvSCSI backend in a XEN driver domain (usually Dom0).
This
On 08/17/2014 04:38 AM, Nicholas A. Bellinger wrote:
On Thu, 2014-08-14 at 12:14 +0200, Juergen Gross wrote:
On 08/14/2014 10:53 AM, Juergen Gross wrote:
Nicholas,
just one more question (see below):
On 08/12/2014 11:13 PM, Nicholas A. Bellinger wrote:
Hi Juergen & Co,
Finally had a ch
On 08/14/2014 10:53 AM, Juergen Gross wrote:
Nicholas,
just one more question (see below):
On 08/12/2014 11:13 PM, Nicholas A. Bellinger wrote:
Hi Juergen & Co,
Finally had a chance to review this code. Comments are inline below..
On Fri, 2014-08-08 at 09:49 +0200, jgr...@suse.com w
Nicholas,
just one more question (see below):
On 08/12/2014 11:13 PM, Nicholas A. Bellinger wrote:
Hi Juergen & Co,
Finally had a chance to review this code. Comments are inline below..
On Fri, 2014-08-08 at 09:49 +0200, jgr...@suse.com wrote:
...
+ if (IS_ERR(tv_nexus->tvn_se_sess
On 08/13/2014 09:02 AM, Juergen Gross wrote:
On 08/12/2014 11:13 PM, Nicholas A. Bellinger wrote:
Hi Juergen & Co,
Finally had a chance to review this code. Comments are inline below..
+static struct se_node_acl *
+scsiback_alloc_fabric_acl(struct se_portal_group *se_tpg)
+{
+st
On 08/12/2014 11:13 PM, Nicholas A. Bellinger wrote:
Hi Juergen & Co,
Finally had a chance to review this code. Comments are inline below..
Thank you very much for your review!
On Fri, 2014-08-08 at 09:49 +0200, jgr...@suse.com wrote:
From: Juergen Gross
Introduces the XEN pv
On 08/12/2014 02:29 PM, Juergen Gross wrote:
On 08/11/2014 08:14 PM, Christoph Hellwig wrote:
+static void scsiback_notify_work(struct vscsibk_info *info)
+{
+info->waiting_reqs = 1;
+wake_up(&info->wq);
+}
+
+static irqreturn_t scsiback_intr(int irq, vo
On 08/11/2014 08:14 PM, Christoph Hellwig wrote:
+#include
__scsi_print_sense()
+#include
struct scsi_sense_hdr
+#include
SG_ALL
What do you need these for? Normally target drivers shouldn't need
these.
+struct vscsibk_emulate {
+ void (*pre_function)(struct vscsibk_pend
On 08/11/2014 07:50 PM, Christoph Hellwig wrote:
On Mon, Aug 11, 2014 at 12:27:29PM +0200, Juergen Gross wrote:
What do you mean with "unusual"? You mean transferring the EH action to
Dom0?
Yes. Note that hyperv tries something similar and they've run into
timeout issues, yo
On 08/11/2014 11:54 AM, Christoph Hellwig wrote:
+ BUG_ON(sc->cmd_len > VSCSIIF_MAX_COMMAND_SIZE);
+
+ if (sc->cmd_len)
I can't see how you can get a zero cmd_len here.
Ahh, thanks for spotting this. In a previous version it could be zero
in case of reset.
+static int scsifront_
On 08/04/2014 01:49 PM, Boaz Harrosh wrote:
On 08/04/2014 02:30 PM, jgr...@suse.com wrote:
From: Juergen Gross
If a scsi host driver specifies .cmd_len in it's scsi_host_template, a driver's
private command pool is needed. scsi_find_host_cmd_pool() will locate it, but
scsi_alloc_hos
On 08/04/2014 01:24 PM, jgr...@suse.com wrote:
From: Juergen Gross
If a scsi host driver specifies .cmd_len in it's scsi_host_template, a driver's
private command pool is needed. scsi_find_host_cmd_pool() will locate it, but
scsi_alloc_host_cmd_pool() isn't saving the pool addr
On 08/04/2014 01:08 PM, Christoph Hellwig wrote:
On Mon, Aug 04, 2014 at 06:26:09AM +0200, jgr...@suse.com wrote:
From: Juergen Gross
If a scsi host driver specifies .cmd_len in it's scsi_host_template, a driver's
private command pool is needed. scsi_find_host_cmd_pool() will loca
On 08/04/2014 12:10 PM, Jan Beulich wrote:
On 04.08.14 at 10:27, wrote:
+/* Requests from the frontend to the backend */
+
+/*
+ * Request a SCSI operation specified via a CDB in vscsiif_request.cmnd.
+ * The target is specified via channel, id and lun.
+ */
+#define VSCSIIF_ACT_SCSI_CDB
On 08/01/2014 10:24 PM, James Bottomley wrote:
On Fri, 2014-08-01 at 05:03 -0700, Christoph Hellwig wrote:
On Fri, Aug 01, 2014 at 08:27:05AM +0200, jgr...@suse.com wrote:
From: Juergen Gross
If a scsi host driver specifies .cmd_len in it's scsi_host_template, a driver's
private co
On 08/01/2014 02:08 PM, Christoph Hellwig wrote:
On Wed, Jul 30, 2014 at 06:53:59AM +0200, Juergen Gross wrote:
Hmm, I looked into scsi_add_device(). It seems as if the caller can't
distinguish between a new created and an already existing device. Am I
missing something?
That's righ
On 08/01/2014 09:05 AM, James Bottomley wrote:
On Fri, 2014-08-01 at 08:02 +0200, Juergen Gross wrote:
On 08/01/2014 07:41 AM, Juergen Gross wrote:
During test of Xen pvSCSI frontend module I found the following issue:
When unplugging a passed-through SCSI-device the SCSI Host is removed
On 08/01/2014 07:41 AM, Juergen Gross wrote:
During test of Xen pvSCSI frontend module I found the following issue:
When unplugging a passed-through SCSI-device the SCSI Host is removed.
When calling the final scsi_host_put() from the driver an Oops is
happening:
[ 219.816292] (file=drivers
During test of Xen pvSCSI frontend module I found the following issue:
When unplugging a passed-through SCSI-device the SCSI Host is removed.
When calling the final scsi_host_put() from the driver an Oops is
happening:
[ 219.816292] (file=drivers/scsi/xen-scsifront.c, line=808)
scsifront_remov
On 07/29/2014 04:57 PM, Juergen Gross wrote:
On 07/29/2014 03:53 PM, Christoph Hellwig wrote:
+switch (op) {
+case VSCSIFRONT_OP_ADD_LUN:
+if (device_state == XenbusStateInitialised) {
+sdev = scsi_device_lookup(info->host, chn,
On 07/29/2014 03:53 PM, Christoph Hellwig wrote:
+ * Patched to support >2TB drives
+ * 2010, Samuel Kvasnica, IMS Nanofabrication AG
+ */
This doesn't really belong into the top of the file comment and should
be moved to the patch description.
Okay.
+
+#include
not needed.
Okay.
+st
On 07/28/2014 04:28 PM, David Vrabel wrote:
On 25/07/14 12:37, jgr...@suse.com wrote:
From: Juergen Gross
Add the definition of pvSCSI protocol used between the pvSCSI frontend in a
XEN domU and the pvSCSI backend in a XEN driver domain (usually Dom0).
This header was originally provided by
On 07/25/2014 07:38 PM, Konrad Rzeszutek Wilk wrote:
On Fri, Jul 25, 2014 at 01:37:29PM +0200, jgr...@suse.com wrote:
This series adds XEN pvSCSI support. With pvSCSI it is possible to use physical
SCSI devices from a XEN domain.
The support consists of a backend in the privileged Domain-0 doin
On 06/28/2014 08:09 PM, Christoph Hellwig wrote:
On Fri, Jun 27, 2014 at 04:34:35PM +0200, jgr...@suse.com wrote:
From: Juergen Gross
Introduces the XEN pvSCSI backend. With pvSCSI it is possible for a XEN domU
to issue SCSI commands to a SCSI LUN assigned to that domU. The SCSI commands
are
On 06/30/2014 03:35 PM, David Vrabel wrote:
On 27/06/14 15:34, jgr...@suse.com wrote:
+/* .resume = scsifront_resume, */
Following on from the recent discussion on migration with front/backends
I thought I'd review how this new driver does it.
"New" is an attribute I wou
On 06/30/2014 01:02 PM, Jan Beulich wrote:
On 27.06.14 at 19:11, wrote:
On Fri, Jun 27, 2014 at 04:34:33PM +0200, jgr...@suse.com wrote:
+/*
+ * Maximum scatter/gather segments per request.
+ *
+ * Considering balance between allocating at least 16 "vscsiif_request"
+ * structures on one page
On 06/27/2014 07:11 PM, Konrad Rzeszutek Wilk wrote:
On Fri, Jun 27, 2014 at 04:34:33PM +0200, jgr...@suse.com wrote:
From: Juergen Gross
Add the definition of pvSCSI protocol used between the pvSCSI frontend in a
XEN domU and the pvSCSI backend in a XEN driver domain (usually Dom0).
This
89 matches
Mail list logo