Align SDHCI DMA bounce buffer to PAGE_SIZE
AbandonedPublic
Actions

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

Tags

Referenced Files

	Unknown Object (File)
	Thu, Jan 9, 3:54 AM

	Unknown Object (File)
	Oct 5 2024, 4:18 AM

	Unknown Object (File)
	Oct 2 2024, 6:23 AM

	Unknown Object (File)
	Sep 30 2024, 8:16 PM

	Unknown Object (File)
	Sep 27 2024, 7:08 AM

	Unknown Object (File)
	Aug 20 2024, 4:23 AM

	Unknown Object (File)
	Aug 12 2024, 3:34 PM

	Unknown Object (File)
	May 19 2024, 8:25 PM

View All 24 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