From: Jens Axboe <axboe@suse.de>
To: James Bottomley <James.Bottomley@SteelEye.com>
Cc: Mike Christie <michaelc@cs.wisc.edu>,
SCSI Mailing List <linux-scsi@vger.kernel.org>
Subject: Re: [RFC] Add user scatter gather I/O to SG_IO
Date: Mon, 13 Jun 2005 08:55:30 +0200 [thread overview]
Message-ID: <20050613065529.GF1414@suse.de> (raw)
In-Reply-To: <1118447969.5031.44.camel@mulgrave>
On Fri, Jun 10 2005, James Bottomley wrote:
> diff --git a/drivers/block/scsi_ioctl.c b/drivers/block/scsi_ioctl.c
> --- a/drivers/block/scsi_ioctl.c
> +++ b/drivers/block/scsi_ioctl.c
> @@ -231,9 +231,6 @@ static int sg_io(struct file *file, requ
> if (verify_command(file, cmd))
> return -EPERM;
>
> - /*
> - * we'll do that later
> - */
> if (hdr->iovec_count)
> return -EOPNOTSUPP;
Didn't you want to remove more than the comment here?
> diff --git a/drivers/scsi/scsi_error.c b/drivers/scsi/scsi_error.c
> --- a/drivers/scsi/scsi_error.c
> +++ b/drivers/scsi/scsi_error.c
> @@ -1230,6 +1230,9 @@ static void scsi_eh_offline_sdevs(struct
> scmd->device->channel,
> scmd->device->id,
> scmd->device->lun);
> + /* try to reset the bus and the card to a sane state */
> + scsi_try_bus_reset(scmd);
> + scsi_try_host_reset(scmd);
> scsi_device_set_state(scmd->device, SDEV_OFFLINE);
> if (scsi_eh_eflags_chk(scmd, SCSI_EH_CANCEL_CMD)) {
Different patch?
Other than that I think it looks good, thanks. I'll get this added asap.
--
Jens Axboe
next prev parent reply other threads:[~2005-06-13 6:54 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2005-06-10 23:59 [RFC] Add user scatter gather I/O to SG_IO James Bottomley
2005-06-12 22:19 ` Mike Christie
2005-06-13 6:55 ` Jens Axboe [this message]
2005-06-13 13:48 ` James Bottomley
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20050613065529.GF1414@suse.de \
--to=axboe@suse.de \
--cc=James.Bottomley@SteelEye.com \
--cc=linux-scsi@vger.kernel.org \
--cc=michaelc@cs.wisc.edu \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.