Current-Users archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

Re: aiboost panics my kernel



On Wed, 16 Apr 2008, Juan Romero Pardines wrote:

for (i = 1; i < o->Package.Count; i++) {
   elem = &o->Package.Elements[i];
   if (elem->Type != ACPI_TYPE_ANY) {
       ...
   }

to

if (elem->Type != ACPI_TYPE_ANY ||
   elem->Type != ACPI_TYPE_LOCAL_REFERENCE)) {
   ...
}

Hmmm, that doesn't look right!  Won't that always evaluate to TRUE?  :)

Also, do you intend to remove the for (...) loop, or simply change the
if (...) statement?  (Confusing, given the indentation...)

----------------------------------------------------------------------
|   Paul Goyette   | PGP DSS Key fingerprint: |  E-mail addresses:   |
| Customer Service | FA29 0E3B 35AF E8AE 6651 |  paul%whooppee.com@localhost   |
| Network Engineer | 0786 F758 55DE 53BA 7731 | pgoyette%juniper.net@localhost |
----------------------------------------------------------------------


Home | Main Index | Thread Index | Old Index