[OSUNIX-dev] Re: [hpcdev-discuss] OpenSolaris open64 port - 1st round patch review

[Rayson Ho]

On Tue, Mar 3, 2009 at 9:03 AM, "C. Bergström" <cbergstrom at netsyncro.com> wrote:
> This code has exactly what we want and may be more portable. (Along with if
> we change it minimally to fit our need we can just continue to merge
> changes.)
>
> http://svn.mplayerhq.hu/mplayer/trunk/cpuinfo.c?view=markup

Ah, looks good. Didn't know that someone has already written code to
parse the CPUID information in a form that is mostly reusable. Then I
guess we should use the code as the license is compatible with Open64.

If we were to start from scratch, I would use read
"/dev/cpu/self/cpuid" (see:
http://docs.sun.com/app/docs/doc/819-2254/cpuid-7d?a=view ) so that
the port to SPARC is a bit easier. However, getting Open64 to generate
SPARC code is at least 10 times more work than the current port, and I
believe whoever wants to get the SPARC port done wonldn't mind writing
an extra few more lines of new code :)


Also:
- I've downloaded the patch but have not started to review it yet... I
will do it soon!
- what exactly is the problem of point number 2 (below)??

Rayson



>
>
> Tracing this back will show how they extracted it..
>  idstr[12] = 0;
>  printf("vendor_id\t: %s\n", idstr);
>
> I have two slightly bigger problems though..
>
> 1) I need to refocus this week so I can start to help write up the gsoc
> projects (not just open64)
>
> 2) ....
>
> dbx - core
> Corefile specified executable:
> "/usr/local/src/bar/open64/lib/gcc-lib/x86_64-open64-freebsd/4.2/wgen42"
> For information about new features see `help changes'
> To remove this message, put `dbxenv suppress_startup_message 7.7' in your
> .dbxrc
> Reading wgen42
> core file header read successfully
> Reading ld.so.1
> Reading libstdc++.so.6.0.11
>
> dbx: internal error: signal SIGSEGV (no mapping at the fault address)
> dbx's coredump will appear in /tmp
> Abort (core dumped)
>
> Anyone who can test the patch or look for errors?
>
> Thanks
>
> ./C
>
>
>