>> + >> + if ( !dt_property_read_string(phandle_node, "cpupool-sched", > > This should be "node", not phandle_node
Hi Stefano, Yes sorry about that, I didn’t notice it because in my last test I was creating pools with default scheduler, I will pay more attention next time, same applies to the scheduler swap. I’ll fix them in next serie > > >> + &scheduler_name) ) >> + sched_id = check_and_get_sched_id(scheduler_name); > > It would be nice if we had XENLOG_INFO printks to print at boot for each > CPU: > - the PoolID > - the scheduler name > Not required, but I had to add them to figure out exactly what was going > on. I’ve added a printk in btcpupools_get_cpupool_id, do you want me to remove from there and print here? Cheers, Luca
