Quoting Tvrtko Ursulin (2021-02-10 10:53:43) > +static struct clients *display_clients(struct clients *clients) > +{ > + struct client *ac, *c, *cp = NULL; > + struct clients *aggregated; > + int tmp, num = 0; > + > + if (!aggregate_pids) > + return sort_clients(clients, client_cmp);
Still two calls to return sort_clients(foo, client_cmp) in this function :) [a clients = aggregated; after processing would merge the two paths]. Reviewed-by: Chris Wilson <ch...@chris-wilson.co.uk> -Chris _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx