Date: Fri, 22 Jan 2010 21:23:09 +0000 (UTC) From: John Baldwin <jhb@FreeBSD.org> To: cvs-src-old@freebsd.org Subject: cvs commit: src/sys/amd64/amd64 pmap.c src/sys/i386/i386 pmap.c Message-ID: <201001222123.o0MLNikM019817@repoman.freebsd.org>
next in thread | raw e-mail | index | archive | help
jhb 2010-01-22 21:23:09 UTC FreeBSD src repository Modified files: (Branch: RELENG_7) sys/amd64/amd64 pmap.c sys/i386/i386 pmap.c Log: SVN rev 202837 on 2010-01-22 21:23:09Z by jhb MFC 189698,192035: Optimize the inner loop of pmap_copy(). Correct a rare use-after-free error in pmap_copy(). This error was introduced in amd64 revision 1.540 and i386 revision 1.547. However, it had no harmful effects until after a recent change, r189698, on amd64. (In other words, the error is harmless in RELENG_7.) The error is triggered by the failure to allocate a pv entry for the one and only mapping in a page table page. I am addressing the error by changing pmap_copy() to abort if either pv entry allocation or page table page allocation fails. This is appropriate because the creation of mappings by pmap_copy() is optional. They are a (possible) optimization, and not a requirement. Correct a nearby whitespace error in the i386 pmap_copy(). Revision Changes Path 1.590.2.31 +9 -4 src/sys/amd64/amd64/pmap.c 1.594.2.25 +4 -2 src/sys/i386/i386/pmap.c
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201001222123.o0MLNikM019817>