Differential D52997

vm_object: Reset kvo_path on each iteration in vm_object_list_handler()
ClosedPublic
Actions

Authored by markj on Thu, Oct 9, 4:00 PM.

Details

Reviewers

kib

Commits

rG13d866b314f8: vm_object: Reset kvo_path on each iteration in vm_object_list_handler()

Summary

Otherwise we print a bogus path for anonymous objects.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

markj created this revision.Thu, Oct 9, 4:00 PM

Herald added a subscriber: imp. · View Herald TranscriptThu, Oct 9, 4:00 PM

markj requested review of this revision.Thu, Oct 9, 4:00 PM

Harbormaster completed remote builds in B67666: Diff 163842.Thu, Oct 9, 4:00 PM

Clear a couple of other fields.

Harbormaster completed remote builds in B67667: Diff 163843.Thu, Oct 9, 4:01 PM

Might be bzero() whole *kvo on each iteration?

This revision is now accepted and ready to land.Thu, Oct 9, 4:56 PM

Just clear the whole struct

This revision now requires review to proceed.Thu, Oct 9, 8:36 PM

Harbormaster completed remote builds in B67671: Diff 163871.Thu, Oct 9, 8:36 PM

kib accepted this revision.Thu, Oct 9, 8:41 PM

This revision is now accepted and ready to land.Thu, Oct 9, 8:41 PM

Closed by commit rG13d866b314f8: vm_object: Reset kvo_path on each iteration in vm_object_list_handler() (authored by markj). · Explain WhyFri, Oct 10, 1:29 PM

This revision was automatically updated to reflect the committed changes.

markj added a commit: rG13d866b314f8: vm_object: Reset kvo_path on each iteration in vm_object_list_handler().

Revision Contents
Changeset List

Path

Size

sys/

vm/

vm_object.c

9 lines

Diff 163932

View Options

vm_object: Reset kvo_path on each iteration in vm_object_list_handler()ClosedPublicActions