RE: [PATCH v3 3/7] fs,x86/resctrl: Add architecture hooks for every mount/unmount

[Luck, Tony]

> > -static inline bool intel_aet_get_events(void) { return false; }
>
> The declaration of intel_aet_get_events() was removed,
> and the function definition still remains in intel_aet.c.
> Not sure if this will trigger a warning like
> "no user for intel_aet_get_events". Since intel_aet_get_events() is
> renamed to static get_events() in a subsequent patch,
> perhaps we can mark intel_aet_get_events() as static in
> this patch too?

I need to explain more in the commit message. Or find a better way to do this.

I wanted to split the file system changes from the architecture changes to make
review easier.  But doing the file system first means that architecture is now
called on every mount, and it isn't ready for that. So, I cheated by breaking
the enumeration of AET in this patch and fixing it in next patch. The kernel
build is ok, so anyone bisecting some other issue won't see a problem (other
than AET not enumerated).

-Tony