On Wed, 7 Feb 2024 at 13:00, Matthew Wilcox <willy@xxxxxxxxxxxxx> wrote: > To be perfectly clear, you're suggesting two things. > > Option (a) change "all" code like this: > - ret = -EINVAL; > + ret = -ERR(EINVAL); > > where ERR would do some magic with __func__ and __LINE__. > > Option (b) > > -#define EINVAL 22 > +#define E_INVAL 22 > +#define EINVAL ERR(E_INVAL) > > and then change all code that does something like: > > if (err == -EINVAL) > to > if (err == -E_INVAL) > > Or have I misunderstood? Something like that, yes. Thanks, Miklos