Differential D56244

powerpc: refactor dmamap setup; free dmamap in error paths
Needs ReviewPublic
Actions

Authored by adrian on Fri, Apr 3, 4:26 PM.

Tags

Referenced Files

	F152459035: D56244.id174880.diff
	Wed, Apr 15, 2:27 AM

	Unknown Object (File)
	Sat, Apr 11, 8:07 PM

	Unknown Object (File)
	Sat, Apr 11, 2:58 PM

	Unknown Object (File)
	Fri, Apr 10, 6:28 PM

	Unknown Object (File)
	Thu, Apr 9, 2:13 PM

	Unknown Object (File)
	Wed, Apr 8, 2:26 PM

	Unknown Object (File)
	Tue, Apr 7, 3:34 PM

	Unknown Object (File)
	Sun, Apr 5, 11:14 PM

View All 9 Files

Subscribers

Details

Reviewers

None

Group Reviewers

Summary

refactor the dmamap allocation / setup path for future code changes to align with arm64 busdma code;
free the dmamap allocation if the dma segment list can't be allocated;
free the dmamap allocation during the busdma dmamem allocation path if the actual memory allocation itself fails.

Locally tested:

POWER9 ppc64le native boot, Raptor Engineering Blackbird
POWER8 ppc6le, IBM POWER S822LC

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 71946
Build 68829: arc lint + arc unit

Event Timeline

adrian created this revision.Fri, Apr 3, 4:26 PM

Herald added subscribers: jhibbits, imp. · View Herald TranscriptFri, Apr 3, 4:26 PM

adrian requested review of this revision.Fri, Apr 3, 4:26 PM

Harbormaster completed remote builds in B71946: Diff 174880.Fri, Apr 3, 4:26 PM

adrian added a parent revision: D56243: powerpc: busdma declaration shuffling.Fri, Apr 3, 4:27 PM

adrian added a reviewer: PowerPC.

adrian added a project: PowerPC.

Herald added subscribers: alfredo, luporl, bdragon. · View Herald TranscriptFri, Apr 3, 4:29 PM

Revision Contents
Changeset List

Path

Size

sys/

powerpc/

powerpc/

busdma_bounce.c

48 lines

Diff 174880

sys/powerpc/powerpc/busdma_bounce.c

Loading...