Re: [XEN PATCH] xen/spinlock: mechanically rename parameter name 'debug'

[Nicola Vetrini <nicola.vetrini@xxxxxxxxxxx>]

On 24/07/23 10:14, Jan Beulich wrote:
> On 21.07.2023 17:29, Nicola Vetrini wrote:
> > --- a/xen/common/spinlock.c
> > +++ b/xen/common/spinlock.c
> > @@ -78,7 +78,7 @@ static int __init cf_check lockdebug_init(void)
> >   }
> >   presmp_initcall(lockdebug_init);
> >   
> > -void check_lock(union lock_debug *debug, bool try)
> > +void check_lock(union lock_debug *dbg, bool try)
> I consider it reasonably likely that something named "dbg" might also
> appear, when we already have something with as generic a named as
> "debug". How about naming these parameters "ld", "ldbg", "lkdbg", or
> yet something along these lines, but (at least slightly) less generic?
No problem. I prefer "lkdbg" among the alternatives.

--
Nicola Vetrini, BSc
Software Engineer, BUGSENG srl (https://bugseng.com)