[Top] [All Lists]

Re: [PATCH] MIPS: Make CP0 config registers readable via sysfs.

To: David Daney <>
Subject: Re: [PATCH] MIPS: Make CP0 config registers readable via sysfs.
From: Manuel Lauss <>
Date: Wed, 12 Dec 2012 20:14:52 +0100
Cc: Lars-Peter Clausen <>, "Hill, Steven" <>, Ralf Baechle <>, "" <>
Dkim-signature: v=1; a=rsa-sha256; c=relaxed/relaxed;; s=20120113; h=mime-version:in-reply-to:references:from:date:message-id:subject:to :cc:content-type; bh=TnqBGculqCCGGk4KBeU39Ppu/SnnRf1SLescYWAOj7I=; b=Uqrci1kOkTaBApSil5vdx8+689N4pA2b5s/F5bl652E/2X7s1i5Dna879HrsqU9QWo RCiZhpcZnytdZHstutfgS4k+a5QX3ZmcG9JbspzlnZg73Tq+BLb4H8exBqr+lvWEtr08 x55NjXvmyyapcpZeQnEnW2lz5C+SYkDAT3tYFdSgoCScPVjeW1YmIIKjrQU0jMAdTLKQ U6qc48aLabwYMmIbL2utEsLT4IC608sWieHQcmsUAxwv/7AIzpQtDpwOJ0uHH/KzC3mB WCr41bQD+k8Pjbc8dGyW8q1H4w35inDJE+xdHsxOyYB2nYQVpGuXnjykqxmYOuS7iwnR JWdg==
In-reply-to: <>
List-archive: <>
List-help: <>
List-id: linux-mips <>
List-owner: <>
List-post: <>
List-software: Ecartis version 1.0.0
List-subscribe: <>
List-unsubscribe: <>
References: <> <> <> <> <>
On Wed, Dec 12, 2012 at 8:08 PM, David Daney <> wrote:
> On 12/12/2012 10:49 AM, Lars-Peter Clausen wrote:
>> On 12/12/2012 05:44 PM, Hill, Steven wrote:
>>> Lars,
>>> This patch was requested by our DSP/Codec group to help with selecting
>>> the best user-space codecs at runtime. Simply reading /proc/cpuinfo was
>>> insufficient. I posted this patch more for feedback and interest with
>>> minimal expectations that it would make it upstream. This patch will always
>>> be in our supported branches, but I will defer to everyone else on its worth
>>> for upstream.
>>> -Steve
>> Well if it is something that is useful it makes sense to upstream it,
>> especially if you are developing applications. Many people before you have
>> learned the hard way that stashing stuff away in their private branches
>> was not
>> the best idea. If you are smart you are going to avoid that.
>> It may not be the best solution though to just dump all the cp register to
>> userspace. As Florian suggested there might be a smarter way to solve
>> this.
> If you want to have glibc's automatically select optimal libraries for
> a given platform, then the elf_platform is the way to do it.
> However I don't think this is necessarily the case here.  elf_platform
> cannot easily handle a bunch of orthogonal capabilities.  So I am in favor
> of the principle of this patch.
> There should be a per CPU file (probably in /sys/devices/system/cpu/cpuXX)
> with the CP0_Config values.
> One question I have is:  Should it be a single file with one row for each
> implemented Config register, or one file per register?

I'm all for the one-file-per-register approach.  I find it easier to
just read the
right register instead of having to find out how many rows to read to get
the value I'm looking for.


<Prev in Thread] Current Thread [Next in Thread>