[PATCH v4 08/35] iommu/mediatek: Use kmalloc for protect buffer
Yong Wu
yong.wu at mediatek.com
Tue Feb 15 21:54:59 PST 2022
On Thu, 2022-01-27 at 12:08 +0100, AngeloGioacchino Del Regno wrote:
> Il 25/01/22 09:56, Yong Wu ha scritto:
> > No need zero for the protect buffer that is only accessed by the
> > IOMMU HW
> > translation fault happened.
> >
> > Signed-off-by: Yong Wu <yong.wu at mediatek.com>
>
> I would rather keep this a devm_kzalloc instead... the cost is very
> minimal and
> this will be handy when new hardware will be introduced, as it may
> require a bigger
> buffer: in that case, "older" platforms will use only part of it and
> we may get
> garbage data at the end.
Currently this is to avoid zero 512 bytes for all the platforms.
Sorry, I don't understand why it is unnecessary when the new hardware
requires a bigger buffer. If the buffer becomes bigger, then clearing
it to 0 need more cost. then this patch is more helpful?
The content in this buffer is garbage, we won't care about or analyse
it.
>
> Regards,
> Angelo
More information about the Linux-mediatek
mailing list