Messages in this thread | | | Date | Tue, 26 May 1998 19:00:25 +0100 | From | "Stephen C. Tweedie" <> | Subject | Re: patch for 2.1.102 swap code |
| |
Hi,
On Mon, 25 May 1998 06:42:53 -0700, "David S. Miller" <davem@dm.cobaltmicro.com> said:
> Alas, I thought about this some more. And one piece of code needs to > be fixed for this invariant about the semaphore being held in the > fault processing code paths to be true everywhere... ptrace()...
Yep --- I was just about to reply to your last mail with this point when I got your follow-up. I've also had one report that the writable cached page reports started when debugging an electric-fenced binary under gdb. Has anyody seen these vm messages who has definitely NOT been running gdb?
There's also the point that the whole swapout code munges page tables without ever taking the mm semaphore, but that case ought to be protected by the combination of (a) having the kernel spinlock and (b) never stalling between starting a vma walk and modifying the pte. (The swapout code is pretty paranoid about this.) However, I'm not absolutely 100% sure that we don't have any unfortunate races left by this exception. (For example, do we ever protect a vma by the mm semaphore without also doing a lock_kernel()?)
--Stephen
- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.rutgers.edu
| |