Re: aiboost panics my kernel

[Paul Goyette <paul%whooppee.com@localhost>]

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 |
----------------------------------------------------------------------