Subject: Re: [PATCH 2/4] perf: jevents: Program to convert JSON file to C style file

From: Sukadev Bhattiprolu <[email-masked]>
Date: Fri, 22 May 2015 11:09:17 -0700

Andi Kleen [[email-masked]] wrote:
| > pmu-events.c depends only on JSON files relevant to the arch perf is
| > being built on and there could be several JSON files per arch. So it
| > would complicate the Makefiles.
| 
| Could just use a wildcard dependency on */$(ARCH)/*.json 

Sure, but shouldn't we allow JSON files to be in subdirs

	pmu-events/arch/x86/HSX/Haswell_core.json

and this could go to arbitrary levels?

| 
| Also it would be good to move the generated file into the object
| directory. I tried it but it needs some more changes to the Makefiles.
| 
| -Andi