Re: Re: PBLK length again

[Herman Sheremetyev <mlists-6MNMYbGzEYJWk0Htik3J/w@public.gmane.org>]

Hi Karol,

> Well, since Intel shipped / still ships chipsets that actually do have a 
> PBLK of length 7, that's somehow odd.
> Best regards,

If what you're saying is right then this patch should probably be included 
in the main source since right now value of 6 is the only one hardcoded in
to be accepted.  Is there some other place than this list I should submit it?

Thanks,

-Herman

--- linux-2.6.7/drivers/acpi/processor.c        2004-06-20 14:49:10.066168648 -0400
+++ linux-2.6.7.hack/drivers/acpi/processor.c   2004-06-19 21:26:33.000000000 -0400
@@ -2214,7 +2230,8 @@

        if (!object.processor.pblk_address)
                ACPI_DEBUG_PRINT((ACPI_DB_INFO, "No PBLK (NULL address)\n"));
-       else if (object.processor.pblk_length != 6)
+       else if ((object.processor.pblk_length != 6 ) &&
+               (object.processor.pblk_length != 7 ))
                ACPI_DEBUG_PRINT((ACPI_DB_ERROR, "Invalid PBLK length [%d]\n",
                        object.processor.pblk_length));
        else {
@@ -2233,7 +2250,7 @@
                 *
                 * (In particular, allocating the IO range for Cardbus)
                 */
-               request_region(pr->throttling.address, 6, "ACPI CPU throttle");
+               request_region(pr->throttling.address, object.processor.pblk_length, "ACPI CPU throttle");
                request_region(acpi_fadt.xpm_tmr_blk.address, 4, "ACPI timer");
        }




-------------------------------------------------------
This SF.Net email sponsored by Black Hat Briefings & Training.
Attend Black Hat Briefings & Training, Las Vegas July 24-29 - 
digital self defense, top technical experts, no vendor pitches, 
unmatched networking opportunities. Visit www.blackhat.com