linux-kernel - Re: [PATCH] Typo in x86 apic.c with DIVISOR setup

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Fri, 10 Oct 2008 08:38:07 +0400
From:	"Cyrill Gorcunov" <gorcunov@...il.com>
To:	"Venki Pallipadi" <venkatesh.pallipadi@...el.com>
Cc:	"Ingo Molnar" <mingo@...e.hu>,
	"Thomas Gleixner" <tglx@...utronix.de>,
	"H. Peter Anvin" <hpa@...or.com>,
	linux-kernel <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] Typo in x86 apic.c with DIVISOR setup

On Fri, Oct 10, 2008 at 4:22 AM, Venki Pallipadi
<venkatesh.pallipadi@...el.com> wrote:
>
> Fix a typo in x86 apic.c CONFG_X86_64 -> CONFIG_X86_64.
> The intention looks like was to have divisor of 1 on x86_64 kernel.
>
> Also, for DIVISOR value of 1 to work, APIC_TDR_DIV should also change
> depending on divide by 1 or divide by 16 is used.
>
> Signed-off-by: Venkatesh Pallipadi <venkatesh.pallipadi@...el.com>
>
> ---
>  arch/x86/kernel/apic.c |    6 ++++--
>  1 file changed, 4 insertions(+), 2 deletions(-)
>
> Index: tip/arch/x86/kernel/apic.c
> ===================================================================
> --- tip.orig/arch/x86/kernel/apic.c     2008-10-09 17:12:39.000000000 -0700
> +++ tip/arch/x86/kernel/apic.c  2008-10-09 17:13:29.000000000 -0700
> @@ -332,10 +332,12 @@ int lapic_get_maxlvt(void)
>  */
>
>  /* Clock divisor */
> -#ifdef CONFG_X86_64
> +#ifdef CONFIG_X86_64
>  #define APIC_DIVISOR 1
> +#define APIC_TDR_DIV APIC_TDR_DIV_1
>  #else
>  #define APIC_DIVISOR 16
> +#define APIC_TDR_DIV APIC_TDR_DIV_16
>  #endif
>
>  /*
> @@ -369,7 +371,7 @@ static void __setup_APIC_LVTT(unsigned i
>        tmp_value = apic_read(APIC_TDCR);
>        apic_write(APIC_TDCR,
>                (tmp_value & ~(APIC_TDR_DIV_1 | APIC_TDR_DIV_TMBASE)) |
> -               APIC_TDR_DIV_16);
> +               APIC_TDR_DIV);
>
>        if (!oneshot)
>                apic_write(APIC_TMICT, clocks / APIC_DIVISOR);
>

Hi Venki,

you know I don't have code under my hands now to take a look on
(i'm in office now) -- but if I recall correctly it's not a typo :) By
writting (1) instead of (16) you make counter decrease faster by
16 times (if to compare with what we have now). So we could get
counter underflow before we've finish calibration. That is was the
original intension of this 16 divisor being used. Did you test this
patch?
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/