On Wed 17-03-21 11:40:03, Feng Tang wrote: > From: Ben Widawsky <ben.widawsky@xxxxxxxxx> > > Now that preferred_nodes is just a mask, and policies are mutually > exclusive, there is no reason to have a separate mask. > > This patch is optional. It definitely helps clean up code in future > patches, but there is no functional difference to leaving it with the > previous name. I do believe it helps demonstrate the exclusivity of the > fields. Yeah, let's just do it after the whole thing is merged. The separation helps a bit to review the code at this stage because it is so much easier to grep for preferred_nodes than nodes. -- Michal Hocko SUSE Labs
