Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
IronPort-PHdr: A9a23:XU6RDBdZMjqui1IRtlTRVpiTlGM/rYqcDmcuAtIPkb9Ccq2/uZ/lO
 R+X6fZsiQrPWoPWo7JBhvHNuq/tEWoH/d6asX8EfZANMn1NicgfkwE6RsLQD0r9Ia3wYiA6F
 dgEX1hgrDm3NEFPE5P4YFvf6nS58T8VHED5Mgx4buT4E4LflYK5zee3rpbSeA5PwjG6ZOAaE
 Q==
IronPort-HdrOrdr: A9a23:cROJEa9rbSPp0OVALWJuk+ENdb1zdoMgy1knxilNoENuE/Bwxv
 rBoB1E73DJYW4qKQ4dcdDpAtjmfZquz+8K3WB3B8biYOCGghrnEGgG1+vfKlLbalbDH4JmpM
 Jdmu1FeaHN5DtB/IbHCWuDYqwdKbC8mcjC74qzvhQdLz2CKZsQkjuRYTzrdHGeMTM2fabRY6
 Dsn/avyQDQHUg/X4CePD0oTuLDr9rEmNbNehgdHSMq7wGIkHeB9KP6OwLw5GZcbxp/hZMZtU
 TVmQ3w4auu99uhzAXH6mPV55NK3PP819p4AtCWgMR9EESvtu/oXvUlZ1SxhkFznAid0idtrD
 AKmWZ4Ay1H0QKUQohym2q05+Cv6kd015ao8y7ovZKqm72IeNt9MbsauWqcGSGpt3bJe7pHof
 92NiuixulqJAKFkyLn69fSURZ20kKyvHo5iOYWy2dSSI0EddZq3MEiFW5uYdw99RjBmcoa+S
 hVfbfhDf1tAB+nhrDizyFSKfmXLzsO9zu9Mzw/U/2uonFrdSpCvj4lLeQk7wA9HbwGOut529
 g=
From:   "Denys Zagorui -X (dzagorui - GLOBALLOGIC INC at Cisco)" 
        <dzagorui@cisco.com>
To:     Jiri Olsa <jolsa@redhat.com>
CC:     "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "peterz@infradead.org" <peterz@infradead.org>,
        "mingo@redhat.com" <mingo@redhat.com>,
        "acme@kernel.org" <acme@kernel.org>,
        "mark.rutland@arm.com" <mark.rutland@arm.com>,
        "alexander.shishkin@linux.intel.com" 
        <alexander.shishkin@linux.intel.com>,
        "namhyung@kernel.org" <namhyung@kernel.org>
Subject: Re: [PATCH v4 1/3] perf report: compile tips.txt in perf binary
Thread-Topic: [PATCH v4 1/3] perf report: compile tips.txt in perf binary
Thread-Index: AQHXQn9lG6UDAOt1jUGn1lCHhYHsvarXf1Og
Date:   Fri, 7 May 2021 05:46:29 +0000
Message-ID: <DM6PR11MB4025626D34B0521A01EBAB03D9579@DM6PR11MB4025.namprd11.prod.outlook.com>
References: <20210430133350.20504-1-dzagorui@cisco.com>,<YJP1HfjAd6hEirmg@krava>
In-Reply-To: <YJP1HfjAd6hEirmg@krava>
Accept-Language: en-GB, en-US
Content-Language: en-GB
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-Network-Message-Id: 78b8a487-7004-4a75-e46c-08d9111b7611
X-MS-Exchange-CrossTenant-originalarrivaltime: 07 May 2021 05:46:29.0720
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 5ae1af62-9505-4097-a69a-c1553ef7840e
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: w6G5n2I9MwNDJKrAjKmRgIkPvIKowz2S6NFCpMO0uJRxdTW8bE8PjUePhNAXF+iyG0yoMhoSBpQ9nFNe4L6Hnw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM6PR11MB4026
X-OriginatorOrg: cisco.com
X-Outbound-SMTP-Client: 173.36.7.19, xbe-aln-004.cisco.com
X-Outbound-Node: alln-core-11.cisco.com
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

> I dont mind to keep static count of tips, but would be great
> to have some check when there are more tips in the tips.txt
>
> or perhaps pick tip with random index within the tips data size?
> you would just do strtok until you reach the string that has the
> random index inside.. you wouldn't need str pointers then

There is no static counter of tips. MAX_TIPS is just maximum number
of tips. Number of tips in tips.txt file:

tools/perf$ wc -l Documentation/tips.txt
43 Documentation/tips.txt

By invoking strtok(..."\n") in a while loop we count how many tips do we ha=
ve
and also save a pointer to corresponding tip in str[]
After we randomly pick up one of those tips (how it works previously).

________________________________________
From: Jiri Olsa <jolsa@redhat.com>
Sent: 06 May 2021 16:54
To: Denys Zagorui -X (dzagorui - GLOBALLOGIC INC at Cisco)
Cc: linux-kernel@vger.kernel.org; peterz@infradead.org; mingo@redhat.com; a=
cme@kernel.org; mark.rutland@arm.com; alexander.shishkin@linux.intel.com; n=
amhyung@kernel.org
Subject: Re: [PATCH v4 1/3] perf report: compile tips.txt in perf binary

On Fri, Apr 30, 2021 at 06:33:48AM -0700, Denys Zagorui wrote:
> It seems there is some need to have an ability to invoke perf from
> build directory without installation
> (84cfac7f05e1: perf tools: Set and pass DOCDIR to builtin-report.c)
> DOCDIR definition contains an absolute path to kernel source directory.
> It is build machine related info and it makes perf binary unreproducible.
>
> This can be avoided by compiling tips.txt in perf directly.
>
> Signed-off-by: Denys Zagorui <dzagorui@cisco.com>
> ---
>  tools/perf/Build               |  2 +-
>  tools/perf/Documentation/Build |  9 ++++++++
>  tools/perf/builtin-report.c    | 39 ++++++++++++++++++++++++++--------
>  tools/perf/util/util.c         | 28 ------------------------
>  tools/perf/util/util.h         |  2 --
>  5 files changed, 40 insertions(+), 40 deletions(-)
>  create mode 100644 tools/perf/Documentation/Build
>
> diff --git a/tools/perf/Build b/tools/perf/Build
> index db61dbe2b543..3a2e768d7576 100644
> --- a/tools/perf/Build
> +++ b/tools/perf/Build
> @@ -45,12 +45,12 @@ CFLAGS_perf.o              +=3D -DPERF_HTML_PATH=3D"B=
UILD_STR($(htmldir_SQ))" \
>                             -DPREFIX=3D"BUILD_STR($(prefix_SQ))"
>  CFLAGS_builtin-trace.o          +=3D -DSTRACE_GROUPS_DIR=3D"BUILD_STR($(=
STRACE_GROUPS_DIR_SQ))"
>  CFLAGS_builtin-report.o         +=3D -DTIPDIR=3D"BUILD_STR($(tipdir_SQ))=
"
> -CFLAGS_builtin-report.o         +=3D -DDOCDIR=3D"BUILD_STR($(srcdir_SQ)/=
Documentation)"
>
>  perf-y +=3D util/
>  perf-y +=3D arch/
>  perf-y +=3D ui/
>  perf-y +=3D scripts/
>  perf-$(CONFIG_TRACE) +=3D trace/beauty/
> +perf-y +=3D Documentation/
>
>  gtk-y +=3D ui/gtk/
> diff --git a/tools/perf/Documentation/Build b/tools/perf/Documentation/Bu=
ild
> new file mode 100644
> index 000000000000..83e16764caa4
> --- /dev/null
> +++ b/tools/perf/Documentation/Build
> @@ -0,0 +1,9 @@
> +perf-y +=3D tips.o
> +
> +quiet_cmd_ld_tips =3D LD       $@
> +      cmd_ld_tips =3D $(LD) -r -b binary -o $@ $<
> +
> +$(OUTPUT)Documentation/tips.o: Documentation/tips.txt FORCE
> +     $(call rule_mkdir)
> +     $(call if_changed,ld_tips)

nice, I had no idea ld could do it like this ;-)

> +
> diff --git a/tools/perf/builtin-report.c b/tools/perf/builtin-report.c
> index 2a845d6cac09..88375ed76d53 100644
> --- a/tools/perf/builtin-report.c
> +++ b/tools/perf/builtin-report.c
> @@ -47,7 +47,6 @@
>  #include "util/time-utils.h"
>  #include "util/auxtrace.h"
>  #include "util/units.h"
> -#include "util/util.h" // perf_tip()
>  #include "ui/ui.h"
>  #include "ui/progress.h"
>  #include "util/block-info.h"
> @@ -107,6 +106,9 @@ struct report {
>       int                     nr_block_reports;
>  };
>
> +extern char _binary_Documentation_tips_txt_start[];
> +extern char _binary_Documentation_tips_txt_end[];
> +
>  static int report__config(const char *var, const char *value, void *cb)
>  {
>       struct report *rep =3D cb;
> @@ -604,19 +606,38 @@ static int report__gtk_browse_hists(struct report *=
rep, const char *help)
>       return hist_browser(rep->session->evlist, help, NULL, rep->min_perc=
ent);
>  }
>
> +#define MAX_TIPS        60
> +
> +static const char *perf_tip(void)
> +{
> +     char *str[MAX_TIPS];
> +     int i =3D 0;
> +
> +     _binary_Documentation_tips_txt_start[_binary_Documentation_tips_txt=
_end -
> +             _binary_Documentation_tips_txt_start - 1] =3D 0;
> +
> +     str[i] =3D strtok(_binary_Documentation_tips_txt_start, "\n");
> +     if (!str[i])
> +             return "Tips cannot be found!";
> +
> +     i++;
> +
> +     while (i < MAX_TIPS) {
> +             str[i] =3D strtok(NULL, "\n");
> +             if (!str[i])
> +                     break;
> +             i++;
> +     }

I dont mind to keep static count of tips, but would be great
to have some check when there are more tips in the tips.txt

or perhaps pick tip with random index within the tips data size?
you would just do strtok until you reach the string that has the
random index inside.. you wouldn't need str pointers then

jirka