>> When dumping the statistics for zones in the allowed nodes in the >> function show_free_areas(), skip_free_areas_node() got called for >> multiple times to figure out the same information: the allowed nodes >> for dump. It's reasonable to get the allowed nodes at once. >> > >I am not sure if cpuset_current_mems_allowed could be changed >during show_free_areas(), also show_free_areas() is not called >in any hot path... > Yeah, but I think it's reasonable to dump consistent nodes here. If cpuset_current_mems_allowed gets changed on the fly, we won't get consistent dump. So the code change would avoid non-consistent case if possible. Thanks, Gavin >-- >To unsubscribe, send a message with 'unsubscribe linux-mm' in >the body to majordomo@xxxxxxxxx. For more info on Linux MM, >see: http://www.linux-mm.org/ . >Don't email: <a href=mailto:"dont@xxxxxxxxx"> email@xxxxxxxxx </a> > -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@xxxxxxxxx. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: <a href=mailto:"dont@xxxxxxxxx"> email@xxxxxxxxx </a>