public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Andrew Morton <akpm@linux-foundation.org>
To: Venki Pallipadi <venkatesh.pallipadi@intel.com>
Cc: Dave Jones <davej@redhat.com>, Len Brown <len.brown@intel.com>,
	linux-kernel <linux-kernel@vger.kernel.org>,
	Andi Kleen <ak@suse.de>, "H. Peter Anvin" <hpa@zytor.com>
Subject: Re: [PATCH] Display Intel Dynamic Acceleration feature in /proc/cpuinfo
Date: Thu, 24 May 2007 15:02:23 -0700	[thread overview]
Message-ID: <20070524150223.41063046.akpm@linux-foundation.org> (raw)
In-Reply-To: <20070523224637.GB15887@linux-os.sc.intel.com>

On Wed, 23 May 2007 15:46:37 -0700
Venki Pallipadi <venkatesh.pallipadi@intel.com> wrote:

> Display Intel Dynamic Acceleration feature in /proc/cpuinfo. This feature
> will be enabled automatically by current acpi-cpufreq driver and cpufreq.
> 
> Refer to Intel Software Developer's Manual for more details about the feature.
> 
> Signed-off-by: Venkatesh Pallipadi <venkatesh.pallipadi@intel.com>
> 
> Index: linux-2.6.22-rc-mm/arch/i386/kernel/cpu/proc.c
> ===================================================================
> --- linux-2.6.22-rc-mm.orig/arch/i386/kernel/cpu/proc.c
> +++ linux-2.6.22-rc-mm/arch/i386/kernel/cpu/proc.c
> @@ -41,7 +41,7 @@ static int show_cpuinfo(struct seq_file 
>  		"cxmmx", "k6_mtrr", "cyrix_arr", "centaur_mcr",
>  		NULL, NULL, NULL, NULL,
>  		"constant_tsc", "up", NULL, NULL, NULL, NULL, NULL, NULL,
> -		NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL,
> +		"ida", NULL, NULL, NULL, NULL, NULL, NULL, NULL,
>  		NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL,
>  
>  		/* Intel-defined (#2) */
> Index: linux-2.6.22-rc-mm/arch/x86_64/kernel/setup.c
> ===================================================================
> --- linux-2.6.22-rc-mm.orig/arch/x86_64/kernel/setup.c
> +++ linux-2.6.22-rc-mm/arch/x86_64/kernel/setup.c
> @@ -949,7 +949,7 @@ static int show_cpuinfo(struct seq_file 
>  		/* Other (Linux-defined) */
>  		"cxmmx", NULL, "cyrix_arr", "centaur_mcr", NULL,
>  		"constant_tsc", NULL, NULL,
> -		"up", NULL, NULL, NULL, NULL, NULL, NULL, NULL,
> +		"up", NULL, NULL, NULL, "ida", NULL, NULL, NULL,
>  		NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL,
>  		NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL,

Ho hum.  This clashes with hpa's git-newsetup tree, which goes for a great
tromp through the cpuinfo implementation.

Hunk #1 FAILED at 41.
1 out of 1 hunk FAILED -- saving rejects to file arch/i386/kernel/cpu/proc.c.rej
patching file arch/x86_64/kernel/setup.c
Hunk #1 FAILED at 951.
1 out of 1 hunk FAILED -- saving rejects to file arch/x86_64/kernel/setup.c.rej
patching file include/asm-i386/cpufeature.h
Hunk #1 FAILED at 81.
1 out of 1 hunk FAILED -- saving rejects to file include/asm-i386/cpufeature.h.rej
patching file include/asm-x86_64/cpufeature.h
Hunk #1 FAILED at 70.
1 out of 1 hunk FAILED -- saving rejects to file include/asm-x86_64/cpufeature.h.rej

I'm not sure I can be bothered repairing all that at present.  I think I'll
go and hide until it becomes somebody else's problem.

<looks at hpa's tree>

@@ -23,13 +23,14 @@ static int show_cpuinfo(struct seq_file 
                "fpu", "vme", "de", "pse", "tsc", "msr", "pae", "mce",
                "cx8", "apic", NULL, "sep", "mtrr", "pge", "mca", "cmov",
                "pat", "pse36", "pn", "clflush", NULL, "dts", "acpi", "mmx",
-               "fxsr", "sse", "sse2", "ss", "ht", "tm", "ia64", "pbe",
+               "fxsr", "sse", "sse2", "ss", "ht", "tm", "ia64", NULL,

<wonders where pbe went to>



  parent reply	other threads:[~2007-05-24 22:03 UTC|newest]

Thread overview: 27+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-05-23 22:46 [PATCH] Display Intel Dynamic Acceleration feature in /proc/cpuinfo Venki Pallipadi
2007-05-24 20:55 ` Andrew Morton
2007-05-24 21:01   ` Dave Jones
2007-05-24 21:08     ` Andi Kleen
2007-05-24 21:13       ` Venki Pallipadi
2007-05-24 21:25         ` Andi Kleen
2007-05-24 21:28           ` Venki Pallipadi
2007-05-24 21:37             ` Dave Jones
2007-05-24 21:56             ` Andi Kleen
2007-05-24 23:03             ` H. Peter Anvin
2007-05-24 23:13               ` Pallipadi, Venkatesh
2007-05-24 23:23                 ` H. Peter Anvin
2007-05-25  0:00                   ` Pallipadi, Venkatesh
2007-05-25  0:04                     ` H. Peter Anvin
2007-05-30  0:52                       ` Venki Pallipadi
2007-05-24 21:10     ` Venki Pallipadi
2007-05-24 22:02 ` Andrew Morton [this message]
2007-05-24 22:08   ` Venki Pallipadi
2007-05-24 22:16     ` H. Peter Anvin
2007-05-24 22:14   ` H. Peter Anvin
2007-05-24 22:41     ` Dave Jones
2007-05-24 22:51       ` H. Peter Anvin
2007-05-24 23:06         ` Dave Jones
2007-05-24 23:11           ` H. Peter Anvin
2007-05-24 23:50       ` Alan Cox
2007-05-24 23:53         ` H. Peter Anvin
2007-05-24 22:27   ` H. Peter Anvin

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20070524150223.41063046.akpm@linux-foundation.org \
    --to=akpm@linux-foundation.org \
    --cc=ak@suse.de \
    --cc=davej@redhat.com \
    --cc=hpa@zytor.com \
    --cc=len.brown@intel.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=venkatesh.pallipadi@intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox