Re: [PATCH] fix cache coherency issues

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Atsushi Nemoto wrote:
On Tue, 14 Feb 2006 17:56:57 +0900, Yoichi Yuasa <yoichi_yuasa@xxxxxxxxxxxxxx> said:
yuasa> This patch fixed the boot problem, but the kernel still has
yuasa> cache coherency problem.

yuasa> ~# ./cachetest yuasa> Test separation: 4096 bytes: FAIL - cache not coherent

Thank you for testing.

As for the cachetest program, I think the test program is wrong.

It try to mmap offset 0 of a shared file to odd address page with
MAP_FIXED.  It means "I want non-coherent mapping if dcache alias
exists".  Currently the kernel surely gives what the program want.

The kernel might have to return EINVAL in such case, but I'm not sure
which is the right behavior.  Please look at David S. Miller's
comments, for example, http://lkml.org/lkml/2003/9/1/48

I've been maintaining for years that EINVAL is the correct
thing to do when the kernel detects that a user is requesting
a mapping that the kernel knows will create aliasing problems,
but this ran counter to the "serves the user right" philosophy
that seemed to dominate the Linux kernel community.  I'm
pleased to see that the tide seems to be shifting.  As an old-time
BSD/SysV hacker from the 1980s, I view system calls as a
contract with the user: if the kernel says it's OK, it's
supposed to work, period.

But I better get down off my soapbox before I launch into
my rant about the OOM killer being proof of a failed paradigm... ;o)

		Regards,

		Kevin K.



[Index of Archives]     [Linux MIPS Home]     [LKML Archive]     [Linux ARM Kernel]     [Linux ARM]     [Linux]     [Git]     [Yosemite News]     [Linux SCSI]     [Linux Hams]

  Powered by Linux