lkml.org 
[lkml]   [2005]   [Dec]   [7]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    Date
    From
    SubjectRFC: ACPI/scsi/libata integration and hotswap
    Hi!

    The included patch does three things:

    1) It adds basic support for binding SCSI and SATA devices to ACPI
    device handles. At the moment this is limited to hosts, and in practice
    it's probably limited to SATA ones (ACPI doesn't spec how SCSI devices
    should appear in the DSDT, so I'm guessing that in general they don't).
    Given a host, you can DEVICE_ACPI_HANDLE(dev) it to get the handle to
    the ACPI device - this should be handy for implementing suspend
    functions, since the methods should be in a standard location underneath
    this.

    Support for binding the devices hasn't been implemented yet. I'm not
    entire sure where they should be bound (the target, presumably?), and I
    haven't actually got it to work...

    2) It adds support for attaching notification events to the host. These
    can be host-driver specific (they're just part of the host template).
    Whenever the hardware generates an event on that bus, the host will be
    called. I've added one of these to ata_piix (since that's what I have),
    which should really be implemented in the host and only call the generic
    one when appropriate. But still.

    3) Adds a generic libata notification handler that currently treats all
    notifications as drive removal/insertion. It then calls Lukasz
    Kosewski's hotplug code to remove/add the drive as appropriate.

    Most laptops generate ACPI notifications requesting bus rescans whenever
    a bay drive is inserted or removed. This handles the event
    appropriately. On my Dell d610, removing or plugging the drive results
    in it appearing or disappearing from /proc/scsi/scsi as appropriate.

    Patch:

    diff -u drivers/scsi/ata_piix.c /tmp/drivers/scsi/ata_piix.c
    --- drivers/scsi/ata_piix.c 2005-12-05 14:30:58 +0000
    +++ /tmp/drivers/scsi/ata_piix.c 2005-12-08 02:16:59 +0000
    @@ -148,6 +148,7 @@
    .ordered_flush = 1,
    .resume = ata_scsi_device_resume,
    .suspend = ata_scsi_device_suspend,
    + .acpi_notify = ata_acpi_notify,
    };

    static const struct ata_port_operations piix_pata_ops = {
    diff -u drivers/scsi/libata-core.c /tmp/drivers/scsi/libata-core.c
    --- drivers/scsi/libata-core.c 2005-12-08 02:27:02 +0000
    +++ /tmp/drivers/scsi/libata-core.c 2005-12-08 02:16:50 +0000
    @@ -50,6 +50,7 @@
    #include <linux/workqueue.h>
    #include <linux/jiffies.h>
    #include <linux/scatterlist.h>
    +#include <linux/acpi.h>
    #include <scsi/scsi.h>
    #include "scsi_priv.h"
    #include <scsi/scsi_cmnd.h>
    @@ -75,9 +76,11 @@
    unsigned int *xfer_shift_out);
    static void __ata_qc_complete(struct ata_queued_cmd *qc);
    static void ata_pio_error(struct ata_port *ap);
    +static int ata_bus_probe(struct ata_port *ap);

    static unsigned int ata_unique_id = 1;
    static struct workqueue_struct *ata_wq;
    +static struct workqueue_struct *ata_hotplug_wq;

    int atapi_enabled = 1;
    module_param(atapi_enabled, int, 0444);
    @@ -88,6 +91,17 @@
    MODULE_LICENSE("GPL");
    MODULE_VERSION(DRV_VERSION);

    +void ata_acpi_notify(acpi_handle device, u32 type, void *data) {
    + struct device *dev = acpi_get_physical_device(device);
    + struct Scsi_Host *shost = dev_to_shost(dev);
    + struct ata_port *ap = (struct ata_port *) &shost->hostdata[0];
    +
    + if (!ata_bus_probe(ap))
    + ata_hotplug_plug(ap);
    + else
    + ata_hotplug_unplug(ap);
    +}
    +
    /**
    * ata_tf_load_pio - send taskfile registers to host controller
    * @ap: Port to which output is sent
    diff -u drivers/scsi/scsi.c /tmp/drivers/scsi/scsi.c
    --- drivers/scsi/scsi.c 2005-12-04 16:48:07 +0000
    +++ /tmp/drivers/scsi/scsi.c 2005-12-08 02:16:28 +0000
    @@ -55,6 +55,7 @@
    #include <linux/interrupt.h>
    #include <linux/notifier.h>
    #include <linux/cpu.h>
    +#include <linux/acpi.h>

    #include <scsi/scsi.h>
    #include <scsi/scsi_cmnd.h>
    @@ -1305,6 +1306,48 @@
    #define unregister_scsi_cpu()
    #endif /* CONFIG_HOTPLUG_CPU */

    +#ifdef CONFIG_ACPI
    +static int scsi_acpi_find_device(struct device *dev, acpi_handle *handle)
    +{
    + return -ENODEV;
    +}
    +
    +void ata_acpi_notify(acpi_handle handle, u32 type, void *data);
    +
    +static int scsi_acpi_find_channel(struct device *dev, acpi_handle *handle)
    +{
    + int i;
    + struct Scsi_Host *shost;
    +
    + if (sscanf(dev->bus_id, "host%u", &i) != 1) {
    + return -ENODEV;
    + }
    +
    + *handle = acpi_get_child(DEVICE_ACPI_HANDLE(dev->parent), (acpi_integer) i);
    + if (!*handle)
    + return -ENODEV;
    +
    + shost = dev_to_shost(dev);
    +
    + if (shost->hostt->acpi_notify) {
    + acpi_install_notify_handler(*handle, ACPI_ALL_NOTIFY, &ata_acpi_notify, (void *)i);
    + }
    +
    + return 0;
    +}
    +
    +static struct acpi_bus_type scsi_acpi_bus = {
    + .bus = &scsi_bus_type,
    + .find_bridge = scsi_acpi_find_channel,
    + .find_device = scsi_acpi_find_device,
    +};
    +
    +static __init int scsi_acpi_register(void)
    +{
    + return register_acpi_bus_type(&scsi_acpi_bus);
    +}
    +#endif /* CONFIG_ACPI */
    +
    MODULE_DESCRIPTION("SCSI core");
    MODULE_LICENSE("GPL");

    @@ -1333,6 +1376,11 @@
    error = scsi_sysfs_register();
    if (error)
    goto cleanup_sysctl;
    +#ifdef CONFIG_ACPI
    + error = scsi_acpi_register();
    + if (error)
    + goto cleanup_sysfs;
    +#endif

    for (i = 0; i < NR_CPUS; i++)
    INIT_LIST_HEAD(&per_cpu(scsi_done_q, i));
    @@ -1343,6 +1391,8 @@
    printk(KERN_NOTICE "SCSI subsystem initialized\n");
    return 0;

    +cleanup_sysfs:
    + scsi_sysfs_unregister();
    cleanup_sysctl:
    scsi_exit_sysctl();
    cleanup_hosts:
    --- include/linux/libata.h 2005-12-05 14:32:50 +0000
    +++ /tmp/include/linux/libata.h 2005-12-08 02:35:59 +0000
    @@ -453,7 +468,9 @@
    extern int ata_scsi_device_suspend(struct scsi_device *);
    extern int ata_device_resume(struct ata_port *, struct ata_device *);
    extern int ata_device_suspend(struct ata_port *, struct ata_device *);
    -
    +#ifdef CONFIG_ACPI
    +extern void ata_acpi_notify(acpi_handle device, u32 type, void *data);
    +#endif
    /*
    * Default driver ops implementations
    */
    diff -u include/linux/libata.h /tmp/include/linux/libata.h
    --- include/linux/libata.h 2005-12-05 14:32:50 +0000
    +++ /tmp/include/linux/libata.h 2005-12-08 02:10:03 +0000
    @@ -448,12 +463,16 @@
    extern int ata_scsi_error(struct Scsi_Host *host);
    extern int ata_scsi_release(struct Scsi_Host *host);
    extern unsigned int ata_host_intr(struct ata_port *ap, struct ata_queued_cmd *qc);
    +extern void ata_hotplug_unplug(struct ata_port *ap);
    +extern void ata_hotplug_plug(struct ata_port *ap);
    extern int ata_ratelimit(void);
    extern int ata_scsi_device_resume(struct scsi_device *);
    extern int ata_scsi_device_suspend(struct scsi_device *);
    extern int ata_device_resume(struct ata_port *, struct ata_device *);
    extern int ata_device_suspend(struct ata_port *, struct ata_device *);
    -
    +#ifdef CONFIG_ACPI
    +extern void ata_acpi_notify(acpi_handle device, u32 type, void *data);
    +#endif
    /*
    * Default driver ops implementations
    */
    diff -u include/scsi/scsi_host.h /tmp/include/scsi/scsi_host.h
    --- include/scsi/scsi_host.h 2005-11-14 14:57:13 +0000
    +++ /tmp/include/scsi/scsi_host.h 2005-12-08 02:12:14 +0000
    @@ -5,6 +5,7 @@
    #include <linux/list.h>
    #include <linux/types.h>
    #include <linux/workqueue.h>
    +#include <linux/acpi.h>

    struct block_device;
    struct completion;
    @@ -300,6 +301,13 @@
    */
    int (*resume)(struct scsi_device *);
    int (*suspend)(struct scsi_device *);
    +
    +#ifdef CONFIG_ACPI
    + /*
    + * ACPI notification handler
    + */
    + void (*acpi_notify)(acpi_handle device, u32 type, void *data);
    +#endif

    /*
    * Name of proc directory
    --
    Matthew Garrett | mjg59@srcf.ucam.org
    -
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to majordomo@vger.kernel.org
    More majordomo info at http://vger.kernel.org/majordomo-info.html
    Please read the FAQ at http://www.tux.org/lkml/
    \
     
     \ /
      Last update: 2005-12-08 04:05    [W:8.223 / U:0.032 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site