Re: [PATCH] RESEND - SN: Add initial ACPI support

From: Jes Sorensen <>
Date: 2006-01-16 19:21:01
>>>>> "John" == John Keller <> writes:

John> Please disregard the earlier (01/14) post of this patch.  I
John> erroneously sent out an early version, which was not fully
John> merged with the latest ia64 git tree.

Hi John,

Looks much better. On top of Prarit's comments I just noticed this

+	if (!hubdev->hdi_flush_nasid_list.widget_p)
+		return;
+	hubdev->hdi_flush_nasid_list.widget_p =
+		kzalloc((HUB_WIDGET_ID_MAX + 1) *
+		    sizeof(struct sn_flush_device_kernel *),
+		    GFP_KERNEL);

This doesn't look right, if widget_p is NULL no alloc ortherwise
overwrite it? Did the order get reversed?

To unsubscribe from this list: send the line "unsubscribe linux-ia64" in
the body of a message to
More majordomo info at
Received on Mon Jan 16 19:21:39 2006

This archive was generated by hypermail 2.1.8 : 2006-01-16 19:21:48 EST