On Thu, 07 Aug 2025, Qianfeng Rong <rongqianf...@vivo.com> wrote: > GFP_NOWAIT already includes __GFP_NOWARN, so let's remove the redundant > __GFP_NOWARN. > > Signed-off-by: Qianfeng Rong <rongqianf...@vivo.com>
Reviewed-by: Jani Nikula <jani.nik...@intel.com> > --- > drivers/gpu/drm/drm_modeset_lock.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/gpu/drm/drm_modeset_lock.c > b/drivers/gpu/drm/drm_modeset_lock.c > index beb91a13a312..58eac20a8138 100644 > --- a/drivers/gpu/drm/drm_modeset_lock.c > +++ b/drivers/gpu/drm/drm_modeset_lock.c > @@ -88,7 +88,7 @@ static noinline depot_stack_handle_t > __drm_stack_depot_save(void) > > n = stack_trace_save(entries, ARRAY_SIZE(entries), 1); > > - return stack_depot_save(entries, n, GFP_NOWAIT | __GFP_NOWARN); > + return stack_depot_save(entries, n, GFP_NOWAIT); > } > > static void __drm_stack_depot_print(depot_stack_handle_t stack_depot) > @@ -98,7 +98,7 @@ static void __drm_stack_depot_print(depot_stack_handle_t > stack_depot) > unsigned int nr_entries; > char *buf; > > - buf = kmalloc(PAGE_SIZE, GFP_NOWAIT | __GFP_NOWARN); > + buf = kmalloc(PAGE_SIZE, GFP_NOWAIT); > if (!buf) > return; -- Jani Nikula, Intel
