Hi André,
On 4/5/22 11:05, André Almeida wrote:
Hi Igor,
Thanks for your patch!
Às 17:45 de 04/04/22, Igor Torrente escreveu:
Currently, the memory to the composition frame is being allocated using the kzmalloc. This comes with the limitation of maximum size of one page size(which in the x86_64 is 4Kb and 4MB for default and hugepage respectively).
Somes test of igt (e.g. kms_plane@pixel-format) uses more than 4MB when testing some pixel formats like ARGB16161616 and the following error were showing up when running kms_plane@plane-panning-bottom-right*:
[drm:vkms_composer_worker [vkms]] *ERROR* Cannot allocate memory for output frame.
This problem is addessed by allocating the memory using kvzalloc that
addessed -> addressed
OTOH, I would write this in imperative mood, as in "Address this by allocating..." or "Fix this..."
circunvents this limitation.
circunvents -> circumvents
Thanks, I will fix them!
V5: Improve the commit message and drop the debugging issues in VKMS TO-DO(Melissa Wen).
Patch changelog are very useful for the mailing list, but not very useful for the git log. For that reason, I usually put this right after the --- in the patch, so the log will be dropped when the patch is applied.
Those comment applies for the rest of your series.
Well, drivers in the DRM subsystem maintain the change history. As you can see in the commit below.
4db3189ce0621be901f249f8cd8226c977dd601d d80976d9ffd9d7f89a26134a299b236910477f3b 84ec374bd580364a32818c9fc269c19d6e931cab 50fff206c5e3a04fcb239ad58d89cad166711b7f
Aside from that, the current VKMS maintainer asked me to add them to the commit body.
And for that two reasons, I will keep them.
Thanks! --- Igor Torrente
Reviewed-by: Melissa Wen mwen@igalia.com Signed-off-by: Igor Torrente igormtorrente@gmail.com