lkml.org 
[lkml]   [2005]   [Sep]   [30]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
SubjectRe: [PATCH 0/7] ppc64: Assorted minor EEH cleanups
From
On Thu, Sep 29, 2005 at 07:48:00PM -0500, linas was heard to remark:
>
> They compile but (ahem) are not tested,

They are now tested. They work (I had a corupted initrd yesterday).
Please apply and foward as soon as possible.

During testing I found two unrelated bugs; wasn't able to squeeze out
patches for today; maybe monday.

Paul, these are:
1) You added an eeh_capable flag that is never initialized, and so
this blocks operation. I don't think this flag is needed, as it
duplicates a bitflag in eeh_mode. (Unless your plan is to use
bitfields; do you want to use C language bitfields?)

2) PCI hotplug is broken because the flag phb->is_dynamic is never
set to one. As a result, hotplug add calls __alloc_bootmem
instead of kmalloc(), and crashes. I was testing a potential
patch just now, but the clock ran out.

--linas

p.s. I hope to spit out the rest of the patces, including the kthread
handling, early next week. I've got things mostly ported, and am
testing. Let me know how to best coordinate on this.


-
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-10-01 00:31    [from the cache]
©2003-2011 Jasper Spaans. Advertise on this site