Re: [PATCH 0/2] Silence allocation warnings for sync arrays

[Matthew Brost <matthew.brost@intel.com>]

On Mon, Nov 24, 2025 at 12:19:14PM +0100, Thomas Hellström wrote:
> On Thu, 2025-11-20 at 23:42 +0000, Shuicheng Lin wrote:
> > Suppress allocation warning as driver already return -ENOMEM safely.
> 
> Even if the driver returns -ENOMEM, in practice a malicious user could
> allocate most or all of system memory. 
> 
> So IMO we need to restrict num_syncs to some reasonable number, and if
> the allocation is persistent we should add __GFP_ACCOUNT.

Neither allocation is persistent - the kmallocs here only live for
execution time of the IOCTL.

Matt

> 
> /Thomas
> 
> 
> > 
> > Shuicheng Lin (2):
> >   drm/xe: Silence allocation warnings for sync arrays
> >   drm/xe/oa: Silence allocation warnings for sync arrays
> > 
> >  drivers/gpu/drm/xe/xe_exec.c | 3 ++-
> >  drivers/gpu/drm/xe/xe_oa.c   | 3 ++-
> >  drivers/gpu/drm/xe/xe_vm.c   | 3 ++-
> >  3 files changed, 6 insertions(+), 3 deletions(-)
> > 
>