Re: [PATCH] perf scripts python: exported-sql-viewer.py: Fix use of TRUE with SQLite

From: Arnaldo Carvalho de Melo
Date: Wed Nov 13 2019 - 07:58:50 EST


Em Wed, Nov 13, 2019 at 02:28:24PM +0200, Adrian Hunter escreveu:
> On 13/11/19 2:15 PM, Arnaldo Carvalho de Melo wrote:
> > Em Wed, Nov 13, 2019 at 02:02:06PM +0200, Adrian Hunter escreveu:
> >> Prior to version 3.23 SQLite does not support TRUE or FALSE, so always use
> >> 1 and 0 for SQLite.
> >>
> >> Fixes: 26c11206f433 ("perf scripts python: exported-sql-viewer.py: Use new 'has_calls' column")
> >> Signed-off-by: Adrian Hunter <adrian.hunter@xxxxxxxxx>
> >> Cc: stable@xxxxxxxxxxxxxxx
> >
> > Thanks, applied and added the first tag with that fixed cset:
> >
> > Cc: stable@xxxxxxxxxxxxxxx # v5.3+
>
> Thanks, but I just noticed it doesn't apply to 5.3 or 5.4-rc7, sorry :-(.
> I guess the Fixes and stable tags should be dropped and I will send the
> stable patch separately.

Humm, I think this is still appropriate, i.e. the fixes/stables should
not be taken literally as being the patch to apply to those older
versions, just that this patch addresses a problem that was introduced
at that tag and that affects the stable kernels since tag v5.3.

Which in turn makes people try to apply it, if it fails, then the stable
guys ask for guidance.

Of course you can be proactive and do what you say you'll do, i.e. test
it and provide patches for the stable kernels, taking into account
whatever adjustments needed due to this code having been touched after
that problem.

So I'm keeping the fixes and cc stable lines,

Thanks,

- Arnaldo