lkml.org 
[lkml]   [2016]   [Sep]   [20]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH v2 5/6] remoteproc/MIPS: Add a remoteproc driver for MIPS
From
Date
Hi Thomas,


On 20/09/16 10:47, Thomas Gleixner wrote:
> On Tue, 20 Sep 2016, Matt Redfearn wrote:
>> +/* Intercept CPU hotplug events for syfs purposes */
>> +static int mips_rproc_callback(struct notifier_block *nfb, unsigned long action,
>> + void *hcpu)
>> +{
> Please convert to cpu hotplug state machine.

OK, I've done that for this and the MIPS GIC patch, using the dynamic
CPUHP_AP_ONLINE_DYN state - I hope that is ok.

>
>> + unsigned int cpu = (unsigned long)hcpu;
>> +
>> + switch (action) {
>> + case CPU_UP_PREPARE:
>> + case CPU_DOWN_FAILED:
>> + mips_rproc_device_unregister(cpu);
>> + break;
>> + case CPU_DOWN_PREPARE:
>> + mips_rproc_device_register(cpu);
>> + break;
>> + }
> There is no reason why you need to setup the rproc device on
> DOWN_PREPARE. It's sufficient to do that when the CPU is dead, so you can
> use a symetric callback prep/dead.

Sure, the new state machine makes this much nicer registering the
on/offline callbacks on one state.

>
>> + /* Dynamically create mips-rproc class devices based on hotplug data */
>> + get_online_cpus();
>> + for_each_possible_cpu(cpu)
>> + if (!cpu_online(cpu))
>> + mips_rproc_device_register(cpu);
>> + register_hotcpu_notifier(&mips_rproc_notifier);
>> + put_online_cpus();
> Perhaps we should add support for "reverse" functionality to the state
> machine core. I'll have a look later how hard that'd be.

Yeah - I've had to work around the framework a little here since we
require the opposite sense to the callbacks - activate the driver when
the cpu is offlined and vice versa. In practice the only issue this gave
me was that by default the framework invokes the teardown callback when
the state is removed, so I used __cpuhp_remove_state() to avoid creating
a remote processor device as the driver is being removed.

Thanks,
Matt

>
> Thanks,
>
> tglx

\
 
 \ /
  Last update: 2016-09-20 17:33    [W:0.103 / U:1.628 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site