Re: [PATCH v2 2/2] xen/include/public: deprecate GNTTABOP_transfer

[Juergen Gross <jgross@xxxxxxxx>]

On 15.02.22 22:13, Julien Grall wrote:
> Hi Juergen,
>
> On 03/02/2022 13:14, Juergen Gross wrote:
> > Add a comment to include/public/grant_table.h that GNTTABOP_transfer
> > is deprecated, in order to discourage new use cases.
>  From the commit message, it is unclear to me why we are discouraging 
> new use cases and indirectly encouraging current users to move away from 
> the feature.
> Patch #1 seems to imply this is because the feature is not present in 
> Linux upstream. But I don't think this is a sufficient reason to 
> deprecate a feature.
> A more compelling reason would be that the feature is broken and too 
> complex to fix it.
> So can you provide more details?
It is a feature available for PV domains only, and it is very complex
and hasn't been tested for ages.

> As a side note, should we also update SUPPORT.md?
Good question.


Juergen

Attachment: OpenPGP_0xB0DE9DD628BF132F.asc
Description: OpenPGP public key

Attachment: OpenPGP_signature
Description: OpenPGP digital signature