From: Jeff Garzik <jeff@garzik.org>
To: linux-scsi@vger.kernel.org, Andrew Morton <akpm@osdl.org>,
LKML <linux-kernel@vger.kernel.org>
Subject: [PATCH] SCSI/qla2xxx: handle sysfs errors
Date: Wed, 11 Oct 2006 21:45:38 -0400 [thread overview]
Message-ID: <20061012014538.GA12894@havoc.gtf.org> (raw)
Signed-off-by: Jeff Garzik <jeff@garzik.org>
---
drivers/scsi/qla2xxx/qla_attr.c | 52 +++++++++++++++++++++++++++++++---------
drivers/scsi/qla2xxx/qla_gbl.h | 4 ---
drivers/scsi/qla2xxx/qla_os.c | 7 ++++-
diff --git a/drivers/scsi/qla2xxx/qla_attr.c b/drivers/scsi/qla2xxx/qla_attr.c
index ee75a71..54e4a60 100644
--- a/drivers/scsi/qla2xxx/qla_attr.c
+++ b/drivers/scsi/qla2xxx/qla_attr.c
@@ -379,22 +379,52 @@ static struct bin_attribute sysfs_sfp_at
.read = qla2x00_sysfs_read_sfp,
};
-void
+int
qla2x00_alloc_sysfs_attr(scsi_qla_host_t *ha)
{
struct Scsi_Host *host = ha->host;
-
- sysfs_create_bin_file(&host->shost_gendev.kobj, &sysfs_fw_dump_attr);
- sysfs_create_bin_file(&host->shost_gendev.kobj, &sysfs_nvram_attr);
- sysfs_create_bin_file(&host->shost_gendev.kobj, &sysfs_optrom_attr);
- sysfs_create_bin_file(&host->shost_gendev.kobj,
- &sysfs_optrom_ctl_attr);
+ int rc;
+
+ rc = sysfs_create_bin_file(&host->shost_gendev.kobj,
+ &sysfs_fw_dump_attr);
+ if (rc) goto err;
+ rc = sysfs_create_bin_file(&host->shost_gendev.kobj,
+ &sysfs_nvram_attr);
+ if (rc) goto err_fwd;
+ rc = sysfs_create_bin_file(&host->shost_gendev.kobj,
+ &sysfs_optrom_attr);
+ if (rc) goto err_nvr;
+ rc = sysfs_create_bin_file(&host->shost_gendev.kobj,
+ &sysfs_optrom_ctl_attr);
+ if (rc) goto err_optrom;
if (IS_QLA24XX(ha) || IS_QLA54XX(ha)) {
- sysfs_create_bin_file(&host->shost_gendev.kobj,
- &sysfs_vpd_attr);
- sysfs_create_bin_file(&host->shost_gendev.kobj,
- &sysfs_sfp_attr);
+ rc = sysfs_create_bin_file(&host->shost_gendev.kobj,
+ &sysfs_vpd_attr);
+ if (rc) goto err_opt_ctl;
+ rc = sysfs_create_bin_file(&host->shost_gendev.kobj,
+ &sysfs_sfp_attr);
+ if (rc) goto err_vpd;
}
+
+ return 0;
+
+err_vpd:
+ sysfs_remove_bin_file(&host->shost_gendev.kobj,
+ &sysfs_vpd_attr);
+err_opt_ctl:
+ sysfs_remove_bin_file(&host->shost_gendev.kobj,
+ &sysfs_optrom_ctl_attr);
+err_optrom:
+ sysfs_remove_bin_file(&host->shost_gendev.kobj,
+ &sysfs_optrom_attr);
+err_nvr:
+ sysfs_remove_bin_file(&host->shost_gendev.kobj,
+ &sysfs_nvram_attr);
+err_fwd:
+ sysfs_remove_bin_file(&host->shost_gendev.kobj,
+ &sysfs_fw_dump_attr);
+err:
+ return rc;
}
void
diff --git a/drivers/scsi/qla2xxx/qla_gbl.h b/drivers/scsi/qla2xxx/qla_gbl.h
index 7da6983..60ade19 100644
--- a/drivers/scsi/qla2xxx/qla_gbl.h
+++ b/drivers/scsi/qla2xxx/qla_gbl.h
@@ -296,9 +296,7 @@ struct class_device_attribute;
extern struct class_device_attribute *qla2x00_host_attrs[];
struct fc_function_template;
extern struct fc_function_template qla2xxx_transport_functions;
-extern void qla2x00_alloc_sysfs_attr(scsi_qla_host_t *);
+extern int qla2x00_alloc_sysfs_attr(scsi_qla_host_t *);
extern void qla2x00_free_sysfs_attr(scsi_qla_host_t *);
extern void qla2x00_init_host_attr(scsi_qla_host_t *);
-extern void qla2x00_alloc_sysfs_attr(scsi_qla_host_t *);
-extern void qla2x00_free_sysfs_attr(scsi_qla_host_t *);
#endif /* _QLA_GBL_H */
diff --git a/drivers/scsi/qla2xxx/qla_os.c b/drivers/scsi/qla2xxx/qla_os.c
index 3f20d76..d70162b 100644
--- a/drivers/scsi/qla2xxx/qla_os.c
+++ b/drivers/scsi/qla2xxx/qla_os.c
@@ -1638,7 +1638,9 @@ qla2x00_probe_one(struct pci_dev *pdev,
if (ret)
goto probe_failed;
- qla2x00_alloc_sysfs_attr(ha);
+ ret = qla2x00_alloc_sysfs_attr(ha);
+ if (ret)
+ goto probe_failed_rmhost;
qla2x00_init_host_attr(ha);
@@ -1658,6 +1660,9 @@ qla2x00_probe_one(struct pci_dev *pdev,
return 0;
+probe_failed_rmhost:
+ scsi_remove_host(host);
+
probe_failed:
qla2x00_free_device(ha);
next reply other threads:[~2006-10-12 1:45 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-10-12 1:45 Jeff Garzik [this message]
2006-10-12 16:57 ` [PATCH] SCSI/qla2xxx: handle sysfs errors Andrew Vasquez
2006-10-12 17:20 ` Jeff Garzik
2006-10-12 17:31 ` Andrew Vasquez
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=20061012014538.GA12894@havoc.gtf.org \
--to=jeff@garzik.org \
--cc=akpm@osdl.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-scsi@vger.kernel.org \
/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