Hello Dave, On 09/09/2014 03:06 AM, Dave Anderson wrote:
I also don't like the idea of making so many changes to the behavior of so many architecture-neutral commands -- and then restricting it to x86_64 only.
I can only take x86_64 into consideration, for I am not clear about other architecture's cpu hot remove. I put the architecture check in the new API, check_offline_cpu, we can involve other architecture that supports cpu hot remove.
Many of the changes reflect the contents of per-cpu data structures of offlined cpus, but even though the cpu is currently offline, the data structures still exist. Why prevent the user from viewing their contents?
I think just showing online cpu's data is reasonable. What about adding a internal crash variables(used by command set) to hide/show offline cpu's data? For the patchset involves too many fixes, I will firstly send one/two fix to see whether the change is good. -- Regards Qiao Nuohan -- Crash-utility mailing list [email protected] https://www.redhat.com/mailman/listinfo/crash-utility
