[PATCH next] cxl: Allow PSL timebase to not sync
Vaibhav Jain
vaibhav at linux.vnet.ibm.com
Tue Mar 15 15:09:29 AEDT 2016
Hi Mikey,
Michael Neuling <mikey at neuling.org> writes:
> I'm not happy with doing this unless we add something which advertises
> that it's synced or not to userspace.
>
> If we do that, I'm happy to just fail without the need of the parameter
> but advertise it to userspace.
>
> The parameter is a bit of a PITA too, as it's a driver level config not
> card level. You really want to turn it on/off based on the card, not
> the whole system.
Would a bool attribute in sysfs for the card (e.g timebase_sync) make
sense? We can by default ignore (and log) timebase sync errors during card
initialization and provide a simple wrapper in libcxl to query and set
this attribute afterwards.
Also it might be better if the AFU advertises the need for timebase
synchronization via a configuration record so that we enable it for the
psl iif one of the slices asks for it. I am not sure if this possible at
the moment but I believe it something we can put forth the psl/hdk team
before timebase usage becomes more widespread.
~ Vaibhav
More information about the Linuxppc-dev
mailing list