• Namhyung Kim's avatar
    perf build: Update build rule for generated files · 7822a891
    Namhyung Kim authored
    The bison and flex generate C files from the source (.y and .l)
    files.  When O= option is used, they are saved in a separate directory
    but the default build rule assumes the .C files are in the source
    directory.  So it might read invalid file if there are generated files
    from an old version.  The same is true for the pmu-events files.
    
    For example, the following command would cause a build failure:
    
      $ git checkout v6.3
      $ make -C tools/perf  # build in the same directory
    
      $ git checkout v6.5-rc2
      $ mkdir build  # create a build directory
      $ make -C tools/perf O=build  # build in a different directory but it
                                    # refers files in the source directory
    
    Let's update the build rule to specify those cases explicitly to depend
    on the files in the output directory.
    
    Note that it's not a complete fix and it needs the next patch for the
    include path too.
    
    Fixes: 80eeb67f ("perf jevents: Program to convert JSON file")
    Signed-off-by: default avatarNamhyung Kim <namhyung@kernel.org>
    Cc: Adrian Hunter <adrian.hunter@intel.com>
    Cc: Andi Kleen <ak@linux.intel.com>
    Cc: Anup Sharma <anupnewsmail@gmail.com>
    Cc: Ian Rogers <irogers@google.com>
    Cc: Ingo Molnar <mingo@kernel.org>
    Cc: Jiri Olsa <jolsa@kernel.org>
    Cc: Peter Zijlstra <peterz@infradead.org>
    Cc: stable@vger.kernel.org
    Link: https://lore.kernel.org/r/20230728022447.1323563-1-namhyung@kernel.orgSigned-off-by: default avatarArnaldo Carvalho de Melo <acme@redhat.com>
    7822a891
Makefile.build 4.24 KB