Differential D3296

The mp_naps have been removed as it does not seem to be needed
ClosedPublic
Actions

Authored by wma_semihalf.com on Aug 4 2015, 7:28 AM.

Details

Reviewers

zbb

Group Reviewers

arm64

Commits

rS286920: Remove redundant mp_naps from ARM64 secondary CPU start-up code

Summary

The global variable has been only used for CPU startup ordering
which does not seem to be needed anyway.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

wma_semihalf.com updated this revision to Diff 7643.Aug 4 2015, 7:28 AM

wma_semihalf.com retitled this revision from to The mp_naps have been removed as it does not seem to be needed.

wma_semihalf.com updated this object.

wma_semihalf.com edited the test plan for this revision. (Show Details)

wma_semihalf.com added a reviewer: arm64.

wma_semihalf.com set the repository for this revision to rS FreeBSD src repository - subversion.

Herald added subscribers: emaste, andrew, imp. · View Herald TranscriptAug 4 2015, 7:28 AM

Seems fine to me, but I'd like to give @andrew an opportunity to comment.

Andrew, do you have any comments? If not, i'd like to submit this change and close the review.

zbb accepted this revision.Aug 19 2015, 10:39 AM

zbb added a reviewer: zbb.

This revision is now accepted and ready to land.Aug 19 2015, 10:39 AM

Closed by commit rS286920: Remove redundant mp_naps from ARM64 secondary CPU start-up code (authored by zbb). · Explain WhyAug 19 2015, 11:45 AM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

head/

sys/

arm64/

mp_machdep.c

12 lines

Diff 8055

View Options

head/sys/arm64/arm64/mp_machdep.c

The mp_naps have been removed as it does not seem to be neededClosedPublicActions