Re: [Xen-devel] [PATCH 8/8] xen: Swich parameter in get_page_from_gfn to use typesafe gfn

[Julien Grall <julien.grall@xxxxxxx>]

Hi,

On 11/12/18 4:58 PM, Andrii Anisov wrote:
> > What's wrong with including clean-up patch in a series adding a feature?
> I do not mean it is wrong.
> Just assuming that introducing a new feature and cleaning up a code
> might be different processes with a different review period.
We don't have different process nor different review period between 
clean-up and new feature.
I tend to do clean-up when writing new features... See my cacheflush 
series as well. If the clean-up is small then I will append/prepend to 
the feature series.
This patch modify a function that was called by this patch and therefore 
depends on the rest of the series. This was written with this series so 
it makes sense to me to do it together as it dictates the order I would 
like the patches applied and simplify tracking (I have many series in 
flight).
Cheers,

--
Julien Grall

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel