Linux ATA/IDE development
 help / color / mirror / Atom feed
From: Shaohua Li <shaohua.li@intel.com>
To: linux-ide <linux-ide@vger.kernel.org>,
	lkml <linux-kernel@vger.kernel.org>
Cc: pavel <pavel@ucw.cz>, Len Brown <len.brown@intel.com>,
	Matthew Garrett <mjg59@srcf.ucam.org>, akpm <akpm@osdl.org>
Subject: [RFC]add ACPI hooks for IDE suspend/resume
Date: Tue, 06 Dec 2005 14:10:04 +0800	[thread overview]
Message-ID: <1133849404.3026.10.camel@sli10-mobl.sh.intel.com> (raw)

Hi,
Adding ACPI IDE hook in IDE suspend/resume. The ACPI spec
explicitly says we must call some ACPI methods to restore IDE drives.
The sequences defined by ACPI spec are:
suspend:
1. Get the DMA and PIO info from IDE channel's _GTM method.

resume:
1. Calling IDE channel's _STM to set the transfer timing setting.
2. For each drive on the IDE channel, running drive's _GTF to get the
ATA commands required to reinitialize each drive.
3. Sending the ATA commands gotton from step 2 to drives.

TODO: invoking ATA commands.

Though we didn't invoke ATA commands, this patch fixes the bug at 
http://bugzilla.kernel.org/show_bug.cgi?id=5604. And Matthew said this
actually fixes a lot of systems in his test.
I'm not familiar with IDE, so comments/suggestions are welcome.

Thanks,
Shaohua

---

 linux-2.6.15-rc5-root/drivers/ide/ide.c |  282 ++++++++++++++++++++++++++++++++
 1 files changed, 282 insertions(+)

diff -puN drivers/ide/ide.c~acpi-ide drivers/ide/ide.c
--- linux-2.6.15-rc5/drivers/ide/ide.c~acpi-ide	2005-12-07 03:01:36.000000000 +0800
+++ linux-2.6.15-rc5-root/drivers/ide/ide.c	2005-12-07 03:01:36.000000000 +0800
@@ -155,6 +155,10 @@
 #include <linux/device.h>
 #include <linux/bitops.h>
 
+#ifdef CONFIG_ACPI
+#include <linux/acpi.h>
+#endif
+
 #include <asm/byteorder.h>
 #include <asm/irq.h>
 #include <asm/uaccess.h>
@@ -1214,6 +1218,279 @@ int system_bus_clock (void)
 
 EXPORT_SYMBOL(system_bus_clock);
 
+#ifdef CONFIG_ACPI
+static int ide_acpi_find_device(struct device *dev, acpi_handle *handle)
+{
+	int i, tmp;
+	acpi_integer addr;
+
+	if (sscanf(dev->bus_id, "%u.%u", &tmp, &i) != 2)
+		return -ENODEV;
+
+	addr = (acpi_integer)i;
+	*handle = acpi_get_child(DEVICE_ACPI_HANDLE(dev->parent), addr);
+	if (!*handle)
+		return -ENODEV;
+	return 0;
+}
+
+/* This assumes the ide controller is a PCI device */
+static int ide_acpi_find_channel(struct device *dev, acpi_handle *handle)
+{
+	int num;
+	int channel;
+	acpi_integer addr;
+
+	num = sscanf(dev->bus_id, "ide%x", &channel);
+
+	if (num != 1 || !dev->parent)
+		return -ENODEV;
+	addr = (acpi_integer)channel;
+	*handle = acpi_get_child(DEVICE_ACPI_HANDLE(dev->parent), addr);
+	if (!*handle)
+		return -ENODEV;
+	return 0;
+}
+
+static struct acpi_bus_type ide_acpi_bus = {
+	.bus = &ide_bus_type,
+	.find_device = ide_acpi_find_device,
+	.find_bridge = ide_acpi_find_channel,
+};
+
+static int __init ide_acpi_init(void)
+{
+	return register_acpi_bus_type(&ide_acpi_bus);
+}
+
+/* The _GTM return package length is 5 dwords */
+#define GTM_LEN (sizeof(u32) * 5)
+struct acpi_ide_state {
+	acpi_handle handle; /* channel device's handle */
+	u32 gtm[GTM_LEN/sizeof(u32)]; /* info from _GTM */
+	struct hd_driveid id_buff[2]; /* one chanel has two drives */
+	int suspend_drives;
+	int resume_drives;
+};
+
+static void acpi_ide_data_handler(acpi_handle handle,
+	u32 function, void *context)
+{
+	/* nothing to do */
+}
+
+/* acpi data for a chanel */
+static struct acpi_ide_state *ide_alloc_acpi_state(acpi_handle handle)
+{
+	struct acpi_ide_state * state;
+	acpi_status status;
+
+	state = kzalloc(sizeof(struct acpi_ide_state), GFP_KERNEL);
+	if (!state)
+		return NULL;
+	status = acpi_attach_data(handle, acpi_ide_data_handler, state);
+	if (ACPI_FAILURE(status))
+		return NULL;
+	return state;
+}
+
+static struct acpi_ide_state *ide_get_acpi_state(acpi_handle handle)
+{
+	struct acpi_ide_state * state;
+	acpi_status status;
+
+	status = acpi_get_data(handle, acpi_ide_data_handler, (void **)&state);
+	if (ACPI_FAILURE(status))
+		return NULL;
+	return state;
+}
+
+static void ide_free_acpi_state(acpi_handle handle)
+{
+	struct acpi_ide_state *state;
+
+	state = ide_get_acpi_state(handle);
+	acpi_detach_data(handle, acpi_ide_data_handler);
+	kfree(state);
+}
+
+static int acpi_ide_suspend(struct device *dev)
+{
+	acpi_handle handle, parent_handle;
+	struct acpi_ide_state *state;
+	acpi_status status;
+	struct acpi_buffer buffer = {ACPI_ALLOCATE_BUFFER, NULL};
+	union acpi_object *package;
+	ide_drive_t *drive = dev->driver_data;
+	int drive_id = 0;
+
+	handle = DEVICE_ACPI_HANDLE(dev);
+	if (!handle) {
+		printk(KERN_DEBUG "IDE device's ACPI handler is NULL\n");
+		return -ENODEV;
+	}
+	if (ACPI_FAILURE(acpi_get_parent(handle, &parent_handle))) {
+		printk(KERN_ERR "ACPI get parent handler error\n");
+		return -ENODEV;
+	}
+	state = ide_get_acpi_state(parent_handle);
+	if (!state) {
+		state = ide_alloc_acpi_state(parent_handle);
+		if (!state)
+			return -ENODEV;
+	}
+
+	/* invoke _GTM only once */
+	state->suspend_drives++;
+	if (state->suspend_drives > 1) {
+		drive_id = 1;
+		goto id;
+	}
+
+	status = acpi_evaluate_object(parent_handle, "_GTM", NULL, &buffer);
+	if (ACPI_FAILURE(status)) {
+		printk(KERN_ERR "Error evaluating _GTM\n");
+		return -ENODEV;
+	}
+	package = (union acpi_object *) buffer.pointer;
+	if (package->buffer.length != GTM_LEN) {
+		printk(KERN_ERR "Buffer length returned by _GTM is wrong\n");
+		acpi_os_free(buffer.pointer);
+		return -ENODEV;
+	}
+	memcpy(state->gtm, package->buffer.pointer, GTM_LEN);
+	state->handle = parent_handle;
+	acpi_os_free(buffer.pointer);
+id:
+	taskfile_lib_get_identify(drive, (u8*)&state->id_buff[drive_id]);
+	return 0;
+}
+
+static int acpi_ide_invoke_stm(struct acpi_ide_state *state)
+{
+	struct acpi_object_list input;
+	union acpi_object params[3];
+	acpi_status status;
+
+	input.count = 3;
+	input.pointer = params;
+	params[0].type = ACPI_TYPE_BUFFER;
+	params[0].buffer.length = sizeof(state->gtm);
+	params[0].buffer.pointer = (char*)state->gtm;
+
+	params[1].type = ACPI_TYPE_BUFFER;
+	params[1].buffer.length = sizeof(state->id_buff[0]);
+	params[1].buffer.pointer = (char *)&state->id_buff[0];
+
+	params[2].type = ACPI_TYPE_BUFFER;
+	params[2].buffer.length = sizeof(state->id_buff[1]);
+	params[2].buffer.pointer = (char *)&state->id_buff[1];
+
+	status = acpi_evaluate_object(state->handle, "_STM", &input, NULL);
+	if (ACPI_FAILURE(status)) {
+		printk(KERN_ERR "Evaluating _STM error\n");
+		return -ENODEV;
+	}
+	return 0;
+}
+
+static int acpi_ide_invoke_gtf(acpi_handle handle, ide_drive_t *drive)
+{
+	struct acpi_buffer output = {ACPI_ALLOCATE_BUFFER, NULL};
+#if 0
+	ide_task_t args;
+	int index = 0;
+	unsigned char *data;
+#endif
+	union acpi_object *package;
+	acpi_status status;
+
+	status = acpi_evaluate_object(handle, "_GTF", NULL, &output);
+	if (ACPI_FAILURE(status)) {
+		printk(KERN_ERR "evaluate _GTF error\n");
+		return -ENODEV;
+	}
+
+	package = (union acpi_object *) output.pointer;
+	if (package->type != ACPI_TYPE_BUFFER
+			|| (package->buffer.length % 7) != 0) {
+		acpi_os_free(output.pointer);
+		printk(KERN_ERR "_GTF returned value is wrong\n");
+		return -ENODEV;
+	}
+#if 0
+	printk(KERN_DEBUG "Start invoking _GTF commands\n");
+
+	data = package->buffer.pointer;
+	/* sumbit ATA commands */
+	while (index < package->buffer.length) {
+		memset(&args, 0, sizeof(ide_task_t));
+		args.tfRegister[IDE_ERROR_OFFSET] = data[index];
+		args.tfRegister[IDE_NSECTOR_OFFSET] = data[index + 1];
+		args.tfRegister[IDE_SECTOR_OFFSET] = data[index + 2];
+		args.tfRegister[IDE_LCYL_OFFSET] = data[index + 3];
+		args.tfRegister[IDE_HCYL_OFFSET] = data[index + 4];
+		args.tfRegister[IDE_SELECT_OFFSET] = data[index + 5];
+		args.tfRegister[IDE_STATUS_OFFSET] = data[index + 6];
+		args.command_type = IDE_DRIVE_TASK_NO_DATA;
+		args.handler = &task_no_data_intr;
+		/* submit command */
+		index += 7;
+	}
+#endif
+	acpi_os_free(output.pointer);
+	return 0;
+}
+
+static int acpi_ide_resume(struct device *dev)
+{
+	acpi_handle handle, parent_handle;
+	struct acpi_ide_state *state;
+	ide_drive_t *drive = dev->driver_data;
+
+	handle = DEVICE_ACPI_HANDLE(dev);
+	if (!handle) {
+		printk(KERN_DEBUG "IDE device's ACPI handler is NULL\n");
+		return -ENODEV;
+	}
+	if (ACPI_FAILURE(acpi_get_parent(handle, &parent_handle))) {
+		printk(KERN_ERR "ACPI get parent handler error\n");
+		return -ENODEV;
+	}
+	state = ide_get_acpi_state(parent_handle);
+	if (state == NULL)
+		return -ENODEV;
+
+	/* invoke _STM only once */
+	state->resume_drives++;
+	if (state->resume_drives == 1) {
+		printk(KERN_DEBUG "Start invoking _STM\n");
+		if (acpi_ide_invoke_stm(state))
+			return -ENODEV;
+	}
+
+	if (state->resume_drives == state->suspend_drives)
+		ide_free_acpi_state(parent_handle);
+	return acpi_ide_invoke_gtf(handle, drive);
+}
+
+#else
+static int __init ide_acpi_init(void)
+{
+	return 0;
+}
+
+static int acpi_ide_suspend(struct device *dev)
+{
+	return 0;
+}
+
+static int acpi_ide_resume(struct device *dev)
+{
+	return 0;
+}
+#endif
+
 static int generic_ide_suspend(struct device *dev, pm_message_t state)
 {
 	ide_drive_t *drive = dev->driver_data;
@@ -1221,6 +1498,8 @@ static int generic_ide_suspend(struct de
 	struct request_pm_state rqpm;
 	ide_task_t args;
 
+	acpi_ide_suspend(dev);
+
 	memset(&rq, 0, sizeof(rq));
 	memset(&rqpm, 0, sizeof(rqpm));
 	memset(&args, 0, sizeof(args));
@@ -1240,6 +1519,8 @@ static int generic_ide_resume(struct dev
 	struct request_pm_state rqpm;
 	ide_task_t args;
 
+	acpi_ide_resume(dev);
+
 	memset(&rq, 0, sizeof(rq));
 	memset(&rqpm, 0, sizeof(rqpm));
 	memset(&args, 0, sizeof(args));
@@ -1923,6 +2204,7 @@ static int __init ide_init(void)
 	system_bus_speed = ide_system_bus_speed();
 
 	bus_register(&ide_bus_type);
+	ide_acpi_init();
 
 	init_ide_data();
 
_



             reply	other threads:[~2005-12-06 22:10 UTC|newest]

Thread overview: 25+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-12-06  6:10 Shaohua Li [this message]
2005-12-06 22:20 ` [RFC]add ACPI hooks for IDE suspend/resume Matthew Garrett
2005-12-06 22:50   ` Alan Cox
2005-12-07  8:17   ` Bartlomiej Zolnierkiewicz
2005-12-07 13:14     ` Matthew Garrett
2005-12-07 14:19       ` Bartlomiej Zolnierkiewicz
2005-12-07 14:26         ` Bartlomiej Zolnierkiewicz
2005-12-07 14:34           ` Matthew Garrett
2005-12-07 14:33         ` Matthew Garrett
2005-12-07 14:45           ` Bartlomiej Zolnierkiewicz
2005-12-07 14:58             ` Matthew Garrett
2005-12-07 15:44               ` Bartlomiej Zolnierkiewicz
2005-12-07 15:53                 ` Matthew Garrett
2005-12-07 15:41             ` Alan Cox
2005-12-07 15:49               ` Bartlomiej Zolnierkiewicz
2005-12-07  1:22                 ` Shaohua Li
2005-12-07 19:15                   ` Bartlomiej Zolnierkiewicz
2005-12-07 22:35                     ` Alan Cox
2005-12-07 22:42                       ` Bartlomiej Zolnierkiewicz
2005-12-07 22:43                       ` Pavel Machek
2005-12-07 22:48                         ` Matthew Garrett
2005-12-07  0:11 ` Randy.Dunlap
2005-12-07  1:15   ` Shaohua Li
  -- strict thread matches above, loose matches on Subject: below --
2005-12-07 21:58 Li, Shaohua
2005-12-07 23:19 Li, Shaohua

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=1133849404.3026.10.camel@sli10-mobl.sh.intel.com \
    --to=shaohua.li@intel.com \
    --cc=akpm@osdl.org \
    --cc=len.brown@intel.com \
    --cc=linux-ide@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mjg59@srcf.ucam.org \
    --cc=pavel@ucw.cz \
    /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