login
Header Space

 
 

[PATCH 0/4, v7] PCI, ACPI: Physical PCI slot objects

Score:
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
To: Gary Hade <garyhade@...>, <kaneshige.kenji@...>, <warthog19@...>
Cc: Matthew Wilcox <matthew@...>, <gregkh@...>, <kristen.c.accardi@...>, <rick.jones2@...>, <linux-kernel@...>, <linux-pci@...>, <linux-acpi@...>
Date: Thursday, February 28, 2008 - 8:23 pm

Hi Gary, John, Kenji-san, et. al,

Well, first Gary was on holiday for a month, and then I was on
holiday for a month, but I'm back now, and have refreshed this
patch series against 2.6.25.

The major thing that happened was all the kobject changes
(learned my lesson about taking long holidays when holding onto a
largish chunk of code that hasn't been accepted yet ;), and so
the only real change is in patch 3/4.

The kobject changes were nice, btw. In the prior versions of this
series, I could never figure out why my kobjects weren't getting
released when their refcounts went to 1, and had some hacky code
in there to manually release them. (I'm sure I was doing
something wrong, but I couldn't figure out what.) I was able to
remove that hack in this series because the kobjects are working
the way they're supposed to.

I did turn on kobject debugging, and all seems well except for
one little thing. I based my module (pci_slot) on acpiphp, and
the kobject system complains:

kobject: 'acpiphp' (a00000020476aed0): does not have a release()
function, it is broken and must be fixed.

kobject: 'pci_slot' (a000000204791e50): does not have a release()
function, it is broken and must be fixed.

Not quite sure what to do about these yet, but since no one has
fixed acpiphp yet, I'm thinking that I can't be *too* wrong. :)

I'm *hoping* that my misunderstanding of kobjects last time
around is what caused the weirdness on Gary's machine, and that
we won't see any more problems.

I've tested this series on my hp rx6600 with both acpiphp and
pciehp drivers, and as before, any and all combinations of those
modules can be modprobe'd/rmmod'ed in any order, etc.

I'd love to see some more testing of this, and then (hopefully!)
upstream acceptance.

Thanks!

/ac

--
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[PATCH 0/4, v7] PCI, ACPI: Physical PCI slot objects, Alex Chiang, (Thu Feb 28, 8:23 pm)
[PATCH 4/4] ACPI PCI slot detection driver, Alex Chiang, (Thu Feb 28, 8:29 pm)
Re: [PATCH 4/4] ACPI PCI slot detection driver, Greg KH, (Sat Mar 1, 1:25 am)
Re: [PATCH 4/4] ACPI PCI slot detection driver, Matthew Wilcox, (Sat Mar 1, 10:43 am)
Re: [PATCH 4/4] ACPI PCI slot detection driver, Greg KH, (Tue Mar 4, 1:49 am)
Re: [PATCH 4/4] ACPI PCI slot detection driver, Alex Chiang, (Tue Mar 4, 6:58 pm)
Re: [PATCH 4/4] ACPI PCI slot detection driver, Greg KH, (Tue Mar 4, 7:15 pm)
Re: [PATCH 4/4] ACPI PCI slot detection driver, Alex Chiang, (Tue Mar 4, 7:46 pm)
Re: [PATCH 4/4] ACPI PCI slot detection driver, Jesse Barnes, (Tue Mar 4, 2:18 pm)
Re: [PATCH 4/4] ACPI PCI slot detection driver, Greg KH, (Tue Mar 4, 3:30 pm)
Re: [PATCH 4/4] ACPI PCI slot detection driver, Alex Chiang, (Tue Mar 4, 7:09 pm)
Re: [PATCH 4/4] ACPI PCI slot detection driver, Kenji Kaneshige, (Tue Mar 4, 9:11 pm)
Re: [PATCH 4/4] ACPI PCI slot detection driver, Alex Chiang, (Wed Mar 5, 4:20 pm)
Re: [PATCH 4/4] ACPI PCI slot detection driver, Kenji Kaneshige, (Tue Mar 11, 9:10 am)
Re: [PATCH 4/4] ACPI PCI slot detection driver, Kristen Carlson Accardi, (Tue Mar 11, 7:34 pm)
Re: [PATCH 4/4] ACPI PCI slot detection driver, Kenji Kaneshige, (Wed Mar 12, 8:59 am)
Re: [PATCH 4/4] ACPI PCI slot detection driver, Kristen Carlson Accardi, (Tue Mar 11, 2:04 pm)
Re: [PATCH 4/4] ACPI PCI slot detection driver, Kenji Kaneshige, (Wed Mar 12, 6:50 am)
Re: [PATCH 4/4] ACPI PCI slot detection driver, Alex Chiang, (Tue Mar 11, 3:14 pm)
Re: [PATCH 4/4] ACPI PCI slot detection driver, Kenji Kaneshige, (Wed Mar 12, 7:33 am)
Re: [PATCH 4/4] ACPI PCI slot detection driver, Alex Chiang, (Wed Mar 12, 11:24 pm)
Re: [PATCH 4/4] ACPI PCI slot detection driver, Gary Hade, (Thu Mar 13, 10:16 pm)
Re: [PATCH 4/4] ACPI PCI slot detection driver, Kenji Kaneshige, (Fri Mar 14, 1:34 am)
Re: [PATCH 4/4] ACPI PCI slot detection driver, Alex Chiang, (Tue Mar 18, 4:49 pm)
Re: [PATCH 4/4] ACPI PCI slot detection driver, Matthew Wilcox, (Tue Mar 11, 9:28 am)
Re: [PATCH 4/4] ACPI PCI slot detection driver, Kenji Kaneshige, (Wed Mar 12, 12:08 am)
Re: [PATCH 4/4] ACPI PCI slot detection driver, Jesse Barnes, (Tue Mar 11, 12:56 pm)
Re: [PATCH 4/4] ACPI PCI slot detection driver, Kenji Kaneshige, (Wed Mar 12, 1:51 am)
[PATCH 3/(3+1)] ACPI PCI slot detection driver, Kenji Kaneshige, (Tue Mar 11, 9:13 am)
Re: [PATCH 3/(3+1)] ACPI PCI slot detection driver, Kenji Kaneshige, (Tue Mar 11, 9:17 am)
Re: [PATCH 4/4] ACPI PCI slot detection driver, Kenji Kaneshige, (Wed Mar 5, 10:07 pm)
Re: [PATCH 4/4] ACPI PCI slot detection driver, Matthew Wilcox, (Wed Mar 5, 4:34 pm)
Re: [PATCH 4/4] ACPI PCI slot detection driver, Kristen Carlson Accardi, (Tue Mar 4, 4:12 pm)
Re: [PATCH 4/4] ACPI PCI slot detection driver, Jesse Barnes, (Tue Mar 4, 4:02 pm)
[PATCH 3/4] Introduce pci_slot, Alex Chiang, (Thu Feb 28, 8:28 pm)
[PATCH 3/4, v8] Introduce pci_slot, Alex Chiang, (Tue Mar 4, 7:30 pm)
Re: [PATCH 3/4] Introduce pci_slot, Greg KH, (Sat Mar 1, 1:24 am)
Re: [PATCH 3/4] Introduce pci_slot, Alex Chiang, (Mon Mar 3, 4:56 pm)
Re: [PATCH 3/4] Introduce pci_slot, Greg KH, (Tue Mar 4, 1:58 am)
[PATCH 2/4] Construct one fakephp slot per pci slot, Alex Chiang, (Thu Feb 28, 8:27 pm)
[PATCH 1/4] Remove path attribute from sgi_hotplug, Alex Chiang, (Thu Feb 28, 8:26 pm)
Re: [PATCH 1/4] Remove path attribute from sgi_hotplug, Jesse Barnes, (Mon Mar 3, 2:48 pm)
Re: [PATCH 1/4] Remove path attribute from sgi_hotplug, Prarit Bhargava, (Mon Mar 3, 2:54 pm)
speck-geostationary