svn commit: r333703 - head/sys/vm

Conrad Meyer cem at freebsd.org
Thu May 17 17:38:22 UTC 2018


On Wed, May 16, 2018 at 9:27 PM, Mark Johnston <markj at freebsd.org> wrote:
> Author: markj
> Date: Thu May 17 04:27:08 2018
> New Revision: 333703
> URL: https://svnweb.freebsd.org/changeset/base/333703
>
> Log:
>   Fix a race in vm_page_pagequeue_lockptr().
>
>   The value of m->queue must be cached after comparing it with PQ_NONE,
>   since it may be concurrently changing.
>
>   Reported by:  glebius

What were the symptoms of this issue?  The test plan in the linked
phabricator revision says:

"Gleb reported seeing panics as a result of the use of a bogus index
into the pagequeue array, and also reported that this patch fixed the
panics."

So an attempt to lock pagequeues[PQ_NONE=255].pq_mutex, which is
either something later in the vm_domain object, or bogus memory?  One
of the mtx asserts trips?

Thanks,
Conrad


More information about the svn-src-all mailing list