Use dmb() macro instead of having separate implementations of do_sync
ClosedPublic
Actions

Authored by stevek on Aug 10 2015, 7:44 PM.

Details

Reviewers

Commits

rS286726: Instead of having separate do_sync functions for ARM_ARCH 6 vs.

Summary

Instead of having separate do_sync functions for ARM_ARCH 6 vs. ARM_ARCH >= 7, use the dmb() macro defined in machine/atomic.h

Test Plan

Build for ARMv6 (PANDABOARD) and ARMv7 (RPI-2) and boot on boards.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

stevek updated this revision to Diff 7821.Aug 10 2015, 7:44 PM

stevek retitled this revision from to Use dmb() macro instead of having separate implementations of do_sync.

stevek updated this object.

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

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

Herald added a reviewer: Contributor Reviewers (ports). · View Herald TranscriptAug 10 2015, 7:44 PM

Herald added subscribers: andrew, imp. · View Herald Transcript

Excellent. One fewer silly ifdef places.

eadler removed a reviewer: Contributor Reviewers (ports).Aug 11 2015, 12:52 AM

This revision is now accepted and ready to land.Aug 11 2015, 12:52 AM

Closed by commit rS286726: Instead of having separate do_sync functions for ARM_ARCH 6 vs. (authored by marcel). · Explain WhyAug 13 2015, 2:53 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

head/

sys/

arm/

stdatomic.c

10 lines

Diff 7926

View Options

head/sys/arm/arm/stdatomic.c

Use dmb() macro instead of having separate implementations of do_syncClosedPublicActions