mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: "Yinghai Lu" <yhlu.kernel@gmail.com>
To: sekharan@us.ibm.com
Cc: "Ingo Molnar" <mingo@elte.hu>,
	"James Bottomley" <James.Bottomley@hansenpartnership.com>,
	"Mark Salyzyn" <mark_salyzyn@adaptec.com>,
	"Linux Kernel Mailing List" <linux-kernel@vger.kernel.org>,
	Linux-Scsi <linux-scsi@vger.kernel.org>
Subject: Re: rdac and aac
Date: Wed, 16 Jul 2008 12:52:12 -0700	[thread overview]
Message-ID: <86802c440807161252j54fee5d3tc572574f083f4735@mail.gmail.com> (raw)
In-Reply-To: <1216233961.27507.225.camel@chandra-ubuntu>

On Wed, Jul 16, 2008 at 11:46 AM, Chandra Seetharaman
<sekharan@us.ibm.com> wrote:
> Please test with the attached patch and see if it works.
>
> regards,
>
> chandra
>
> ---------------------
>
> Signed-off-by: Chandra Seetharaman <sekharan@us.ibm.com>
> --
>
> Index: linux2.6.26-git3/drivers/scsi/device_handler/scsi_dh_rdac.c
> ===================================================================
> --- linux2.6.26-git3.orig/drivers/scsi/device_handler/scsi_dh_rdac.c
> +++ linux2.6.26-git3/drivers/scsi/device_handler/scsi_dh_rdac.c
> @@ -608,12 +608,17 @@ static int rdac_bus_notify(struct notifi
>                            unsigned long action, void *data)
>  {
>        struct device *dev = data;
> -       struct scsi_device *sdev = to_scsi_device(dev);
> +       struct scsi_device *sdev;
>        struct scsi_dh_data *scsi_dh_data;
>        struct rdac_dh_data *h;
>        int i, found = 0;
>        unsigned long flags;
>
> +       if (!scsi_is_sdev_device(dev))
> +               return 0;
> +
> +       sdev = to_scsi_device(dev);
> +
>        if (action == BUS_NOTIFY_ADD_DEVICE) {
>                for (i = 0; rdac_dev_list[i].vendor; i++) {
>                        if (!strncmp(sdev->vendor, rdac_dev_list[i].vendor,
> Index: linux2.6.26-git3/drivers/scsi/device_handler/scsi_dh_emc.c
> ===================================================================
> --- linux2.6.26-git3.orig/drivers/scsi/device_handler/scsi_dh_emc.c
> +++ linux2.6.26-git3/drivers/scsi/device_handler/scsi_dh_emc.c
> @@ -416,12 +416,17 @@ static int clariion_bus_notify(struct no
>                                unsigned long action, void *data)
>  {
>        struct device *dev = data;
> -       struct scsi_device *sdev = to_scsi_device(dev);
> +       struct scsi_device *sdev;
>        struct scsi_dh_data *scsi_dh_data;
>        struct clariion_dh_data *h;
>        int i, found = 0;
>        unsigned long flags;
>
> +       if (!scsi_is_sdev_device(dev))
> +               return 0;
> +
> +       sdev = to_scsi_device(dev);
> +
>        if (action == BUS_NOTIFY_ADD_DEVICE) {
>                for (i = 0; clariion_dev_list[i].vendor; i++) {
>                        if (!strncmp(sdev->vendor, clariion_dev_list[i].vendor,
> Index: linux2.6.26-git3/drivers/scsi/device_handler/scsi_dh_hp_sw.c
> ===================================================================
> --- linux2.6.26-git3.orig/drivers/scsi/device_handler/scsi_dh_hp_sw.c
> +++ linux2.6.26-git3/drivers/scsi/device_handler/scsi_dh_hp_sw.c
> @@ -131,11 +131,16 @@ static int hp_sw_bus_notify(struct notif
>                            unsigned long action, void *data)
>  {
>        struct device *dev = data;
> -       struct scsi_device *sdev = to_scsi_device(dev);
> +       struct scsi_device *sdev;
>        struct scsi_dh_data *scsi_dh_data;
>        int i, found = 0;
>        unsigned long flags;
>
> +       if (!scsi_is_sdev_device(dev))
> +               return 0;
> +
> +       sdev = to_scsi_device(dev);
> +
>        if (action == BUS_NOTIFY_ADD_DEVICE) {
>                for (i = 0; hp_sw_dh_data_list[i].vendor; i++) {
>                        if (!strncmp(sdev->vendor, hp_sw_dh_data_list[i].vendor,

it works. Thanks

YH

      reply	other threads:[~2008-07-16 19:52 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-07-16 17:51 Yinghai Lu
2008-07-16 18:17 ` James Bottomley
2008-07-16 18:46 ` Chandra Seetharaman
2008-07-16 19:52   ` Yinghai Lu [this message]

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=86802c440807161252j54fee5d3tc572574f083f4735@mail.gmail.com \
    --to=yhlu.kernel@gmail.com \
    --cc=James.Bottomley@hansenpartnership.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-scsi@vger.kernel.org \
    --cc=mark_salyzyn@adaptec.com \
    --cc=mingo@elte.hu \
    --cc=sekharan@us.ibm.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®