On 09.08.2023 15:34, Luca Fancellu wrote: >> On 9 Aug 2023, at 12:02, Nicola Vetrini <nicola.vetr...@bugseng.com> wrote: >> >> The function 'ioreq_server_max_frames' can be defined static, >> as its only uses are within the same file. This in turn avoids >> violating Rule 8.4 because no declaration is present. >> >> No functional change. >> >> Signed-off-by: Nicola Vetrini <nicola.vetr...@bugseng.com> > > Makes sense, > > Reviewed-by: Luca Fancellu <luca.fance...@arm.com> > > Maybe it’s also better adding this: > Fixes: 9244528955de ("xen/memory: Fix acquire_resource size semantics”) > > If the maintainers agree
I for one agree; in fact I did ask for Fixes: tags throughout this series, but in the context of another patch. As said there I also think titles and descriptions want writing differently here. Jan