Change in coreboot[master]: libpayload: cbgfx: Clear screen by sequential access - coreboot-gerrit

4 Oct 2021


      Hung-Te Lin has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/57104 )
Change subject: libpayload: cbgfx: Clear screen by sequential access
......................................................................
Patch Set 5:
(1 comment)
File payloads/libpayload/drivers/video/graphics.c:
https://review.coreboot.org/c/coreboot/+/57104/comment/ec46bd64_0a96da81 
PS4, Line 637: 		memset(FB, color & 0xff, fbinfo->y_resolution * bpl);
...
I don't why this becomes even slower using memcpy() on brya. See CL:3200909.
Maybe the VRAM's internal mapping is different from linear addressing?
Is it possible to have a background that is memset-friendly?
-- 
To view, visit https://review.coreboot.org/c/coreboot/+/57104
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Iecaff5b6abc24ba4b3859cbc44c0d61b2a90b2d9
Gerrit-Change-Number: 57104
Gerrit-PatchSet: 5
Gerrit-Owner: Yu-Ping Wu yupingso@google.com
Gerrit-Reviewer: Felix Held felix-coreboot@felixheld.de
Gerrit-Reviewer: Hung-Te Lin hungte@chromium.org
Gerrit-Reviewer: Julius Werner jwerner@chromium.org
Gerrit-Reviewer: build bot (Jenkins) no-reply@coreboot.org
Gerrit-CC: 9elements QA hardwaretestrobot@gmail.com
Gerrit-CC: Paul Menzel paulepanter@mailbox.org
Gerrit-Comment-Date: Mon, 04 Oct 2021 08:11:09 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Julius Werner jwerner@chromium.org
Comment-In-Reply-To: Yu-Ping Wu yupingso@google.com
Gerrit-MessageType: comment