Align SDHCI DMA bounce buffer to PAGE_SIZE
AbandonedPublic
Actions

Authored by mw on Nov 1 2020, 5:12 PM.

Tags

Referenced Files

	F82002673: D27041.diff
	Wed, Apr 24, 9:28 AM

	Unknown Object (File)
	Mon, Apr 22, 8:44 PM

	Unknown Object (File)
	Dec 24 2023, 7:27 PM

	Unknown Object (File)
	Dec 20 2023, 5:37 AM

	Unknown Object (File)
	Dec 13 2023, 6:21 AM

	Unknown Object (File)
	Dec 12 2023, 2:20 PM

	Unknown Object (File)
	Sep 23 2023, 7:43 AM

	Unknown Object (File)
	Aug 3 2023, 1:48 AM

View All 15 Files

Subscribers

ar_semihalf.com

imp

Details

Reviewers

manu
mmel
andrew

Summary

r366106 extended the number of bounce cases for arm64 busdma.
As a result the DMA memory allocation in sdhci_dma_alloc()
started failing with EFIBG error. It turned out, that a huge
alignment value (4K-512K) passed to bus_dma_tag_create()
can be problematic. Limit the bounce buffer alignment to
the PAGE_SIZE.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

mw created this revision.Nov 1 2020, 5:12 PM

Herald added a subscriber: imp. · View Herald TranscriptNov 1 2020, 5:12 PM

mw requested review of this revision.Nov 1 2020, 5:12 PM

I think it's wrong. In this case, multi-page alignment is required, the old comment is correct. Please see https://reviews.freebsd.org/D26735. I plan to commit this in the next day or two.

Actual fix - https://reviews.freebsd.org/D26735

Revision Contents
Changeset List

Path

Size

sys/

dev/

sdhci/

sdhci.c

4 lines

Diff 79041

View Options

Align SDHCI DMA bounce buffer to PAGE_SIZEAbandonedPublicActions