All of lore.kernel.org
 help / color / mirror / Atom feed
From: Dave Olien <dmo@osdl.org>
To: device-mapper development <dm-devel@redhat.com>
Cc: Alasdair G Kergon <agk@redhat.com>
Subject: Re: New -udm?
Date: Sun, 10 Apr 2005 18:14:44 -0700	[thread overview]
Message-ID: <20050411011444.GA3748@osdl.org> (raw)
In-Reply-To: <4259B895.4070600@us.ibm.com>


You're correct.  I'll rewrite it on Thursday this week.
I'll use the same methods Lars used in the dm-emc.c


On Sun, Apr 10, 2005 at 04:36:53PM -0700, Mike Christie wrote:
> christophe varoqui wrote:
> >On dim, 2005-04-10 at 21:29 +0200, Lars Marowsky-Bree wrote:
> >
> >>Hi Alasdair,
> >>
> >>I'm resync'ing against -udm and would like to pull all recent fixes if
> >>possible. Do you have anything more recent than the 2.6.11-rc3-udm2?
> >>
> >>
> >>Sincerely,
> >>   Lars Marowsky-Brée <lmb@suse.de>
> >>
> >There is this hwhandler from Dave Olien, that needs auditing, but is a
> >must-have for StorageWorks and FSC FibreCat users.
> >
> >http://christophe.varoqui.free.fr/multipath-tools/dm-scsi-start.patch
> >
> >Would be good to have in an enterprise distro ...
> >
> >Regards,
> 
> 
> 
> +static void sstart_pg_init(struct hw_handler *hwh, unsigned bypassed,
> +			struct path *path)
> +{
> +	struct block_device *bdev = path->dev->bdev;
> +	struct gendisk *disk = bdev->bd_disk;
> +	struct scsi_device *sdev = scsi_disk(disk)->device;
> +	int result;
> +
> +	result = scsi_ioctl(sdev, SCSI_IOCTL_START_UNIT, NULL);
> 
> 
> 
> is scsi_ioctl depreciated for this use? Besides that I think it 
> allocates memory with GFP_KERNEL which I do not think you want here, and 
> you will want some way to gaurantee you can make forward progress.
> 
> 
> +	dm_pg_init_complete(path, result?MP_FAIL_PATH:0);
> +}
> +
> 
> 
> --
> dm-devel mailing list
> dm-devel@redhat.com
> https://www.redhat.com/mailman/listinfo/dm-devel

  reply	other threads:[~2005-04-11  1:14 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-04-10 19:29 New -udm? Lars Marowsky-Bree
2005-04-10 20:25 ` christophe varoqui
2005-04-10 23:36   ` Dave Olien
2005-04-10 23:36   ` Mike Christie
2005-04-11  1:14     ` Dave Olien [this message]
2005-04-11  8:46       ` Lars Marowsky-Bree
2005-04-11  9:27         ` Mike Christie
2005-04-11  9:34           ` Mike Christie
2005-04-11  9:56           ` Mike Christie
2005-04-11 11:43           ` Lars Marowsky-Bree
2005-04-11 11:53             ` Mike Christie
  -- strict thread matches above, loose matches on Subject: below --
2005-04-11 17:36 goggin, edward
2005-04-11 18:25 ` Mike Christie
2005-04-11 18:26   ` Mike Christie
2005-04-11 18:31   ` Mike Christie
2005-04-11 22:56 ` Mike Christie

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=20050411011444.GA3748@osdl.org \
    --to=dmo@osdl.org \
    --cc=agk@redhat.com \
    --cc=dm-devel@redhat.com \
    /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.