Re: [PATCH v3 2/4] xen: make VMAP only support in MMU system

[Julien Grall <julien@xxxxxxx>]

Hi Jan,

On 19/08/2024 13:24, Jan Beulich wrote:
> On 19.08.2024 14:12, Julien Grall wrote:
> > On 19/08/2024 12:39, Jan Beulich wrote:
> > > Guys,
> > >
> > > On 19.08.2024 11:45, Ayan Kumar Halder wrote:
> > > > On 16/08/2024 17:40, Julien Grall wrote:
> > > > > On 14/08/2024 13:33, Ayan Kumar Halder wrote:
> > > mind me asking why I continue to be on the To: list of this communication
> > > between the two of you?
> > You were involved in the review and AFAICT will still touch the common
> > code. I am happy to remove you from the conversation if you are not
> > interested :).
> That wasn't my request though. It's fine to keep me Cc-ed.
TBH, in this context, the difference between CC and TO is very blur.

For me, CC is for people that may be interested but not involved in the 
conversation. In this case, Ayan was suggesting whether we should use 
ARCH_VMAP or not.
Given you reviewed the rest, I feel you have your say on this decision. 
Hence why the TO makes sense. But I also understand why people may view 
it differently...
Personally, I don't make any filter on "cc" vs "to" because this is a 
matter of opinion whether you think someone should be actively involved 
(or not) in the discussion. So it is best if I look at everything even 
if this is just having a glancing look at on the content.
If you have some clear and unambiguous rules on when you want to be in 
the CC or TO, then I would be happy to follow them.
Cheers,

--
Julien Grall