Re: CVS commit: src/sys/arch/x86/include

[Paul Goyette <paul%whooppee.com@localhost>]

Any chance that this will fix kern/52919?

If not, can we do some additional re-arrangement of struct cpu_info to 
address the problem?



On Sun, 15 Jul 2018, Maxime Villard wrote:

> Module Name:	src
> Committed By:	maxv
> Date:		Sun Jul 15 08:47:43 UTC 2018
>
> Modified Files:
> 	src/sys/arch/x86/include: cpu.h
>
> Log Message:
> Hum. Move the __HAVE_DIRECT_MAP block a little below, otherwise dynamically
> loaded kernel modules use a wrong offset for some ci_* fields. Found when
> modloading tprof_amd on an AMD 10h, the read of ci_signature was at a
> wrong address, and the cpu family was not detected correctly.
>
>
> To generate a diff of this commit:
> cvs rdiff -u -r1.94 -r1.95 src/sys/arch/x86/include/cpu.h
>
> Please note that diffs are not public domain; they are subject to the
> copyright notices on the relevant files.
>
>
> !DSPAM:5b4b0a35163311665672011!

+------------------+--------------------------+----------------------------+
| Paul Goyette     | PGP Key fingerprint:     | E-mail addresses:          |
| (Retired)        | FA29 0E3B 35AF E8AE 6651 | paul at whooppee dot com   |
| Kernel Developer | 0786 F758 55DE 53BA 7731 | pgoyette at netbsd dot org |
+------------------+--------------------------+----------------------------+