Messages in this thread | | | Date | Tue, 9 Jun 2020 16:04:18 +0200 | From | Michal Hocko <> | Subject | Re: [PATCH v2 08/12] mm/migrate: change the interface of the migration target alloc/free functions |
| |
On Wed 27-05-20 15:44:59, Joonsoo Kim wrote: > From: Joonsoo Kim <iamjoonsoo.kim@lge.com> > > To prepare unifying duplicated functions in following patches, this patch > changes the interface of the migration target alloc/free functions. > Functions now use struct alloc_control as an argument.
It also pulls private argument into alloc_control and keeps it that way. Wouldn't it be better to use explicit types and names in a union? Each allocation callback has to understant the meaning anyway. I would consider the resulting code cleaner that way. What do you think? -- Michal Hocko SUSE Labs
| |