On 11/22/22 21:58, Noralf Trønnes via B4 Submission Endpoint wrote: > From: Noralf Trønnes <noralf@xxxxxxxxxxx> > > If a framebuffer flush fails the driver will do one retry by requeing the > worker. Currently the worker is used even for synchronous flushing, but a > later patch will inline it, so this needs to change. Thinking about how to > solve this I came to the conclusion that this retry mechanism was a fix > for a problem that was only in the mind of the developer (me) and not > something that solved a real problem. > > So let's remove this for now and revisit later should it become necessary. > gud_add_damage() has now only one caller so it can be inlined. > Makes sense. > Signed-off-by: Noralf Trønnes <noralf@xxxxxxxxxxx> > --- > drivers/gpu/drm/gud/gud_pipe.c | 48 +++++++----------------------------------- > 1 file changed, 8 insertions(+), 40 deletions(-) Reviewed-by: Javier Martinez Canillas <javierm@xxxxxxxxxx> -- Best regards, Javier Martinez Canillas Core Platforms Red Hat