Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1731619

Re: [PATCH 2/4] xen: limit grant v2 interface to the v1 functionality

Show all headers | View raw

On 09/13/2017 09:38 AM, Juergen Gross wrote:
> On 13/09/17 15:22, Boris Ostrovsky wrote:
>> On 09/12/2017 02:18 PM, Juergen Gross wrote:
>>> On 12/09/17 18:21, Boris Ostrovsky wrote:
>>>> On 09/12/2017 12:09 PM, Juergen Gross wrote:
>>>>> On 12/09/17 18:05, Boris Ostrovsky wrote:
>>>>>> On 09/12/2017 11:50 AM, Juergen Gross wrote:
>>>>>>> On 12/09/17 17:44, Boris Ostrovsky wrote:
>>>>>>>> On 09/08/2017 10:48 AM, Juergen Gross wrote:
>>>>>>>>> As there is currently no user for sub-page grants or transient grants
>>>>>>>>> remove that functionality. This at once makes it possible to switch
>>>>>>>>> from grant v2 to grant v1 without restrictions, as there is no loss of
>>>>>>>>> functionality other than the limited frame number width related to
>>>>>>>>> the switch.
>>>>>>>> But isn't that ABI violation? v2 is expected to support this (XSAs
>>>>>>>> notwithstanding)
>>>>>>> No, I don't think so.
>>>>>>>
>>>>>>> The hypervisor still supports it, but the domU (or dom0) isn't required
>>>>>>> to make use of all the features IMHO. Or are you aware of any backend
>>>>>>> querying the grant version of a frontend and acting in another way if v2
>>>>>>> is detected?
>>>>>> I am not aware of any but that doesn't mean that they don't (or won't)
>>>>>> exist.
>>>>> But isn't the frontend the one which is defining what is granted in
>>>>> which way? How should there be an ABI breakage when the frontend just
>>>>> isn't using sub-page or transitive grants?
>>>> People may provide both front and backend drivers and frontends, knowing
>>>> that v2 is available, could decide to use those features.
>>> No, without the functions to use them it will be impossible.
>> I don't follow this. Which functions? The ones this patch is removing?
> Yes, just after having been added one patch earlier.
>
> Right now the Linux kernel doesn't support grant V2 at all. So there
> surely is no driver making use of V2 features right now.
>
> Ican merge patches 1 and 2 in case you want. I just thought a pure
> revert of the former V2 remove patch would be easier to review,
> taking into account that the former V2 support was working in
> production environments (and even back then there was no user of
> sub-page or transient grants).

No, I don't have problems with *how* you are doing this (revert fully
first and then remove).

I am just not sure that removing these functions is the way to go
because we are ending up with partial implementation of v2. The fact
that noone is/has been using these features is IMO not particularly
relevant.

If these two were optional features then it would have been reasonable
to drop them.

-boris

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

Re: [PATCH 2/4] xen: limit grant v2 interface to the v1 functionality Boris Ostrovsky <boris.ostrovsky@oracle.com> - 2017-09-12 17:50 +0200
  Re: [PATCH 2/4] xen: limit grant v2 interface to the v1 functionality Juergen Gross <jgross@suse.com> - 2017-09-12 18:00 +0200
    Re: [PATCH 2/4] xen: limit grant v2 interface to the v1 functionality Juergen Gross <jgross@suse.com> - 2017-09-12 18:10 +0200
      Re: [PATCH 2/4] xen: limit grant v2 interface to the v1 functionality Boris Ostrovsky <boris.ostrovsky@oracle.com> - 2017-09-12 18:30 +0200
        Re: [PATCH 2/4] xen: limit grant v2 interface to the v1 functionality Juergen Gross <jgross@suse.com> - 2017-09-12 20:20 +0200
          Re: [PATCH 2/4] xen: limit grant v2 interface to the v1 functionality Boris Ostrovsky <boris.ostrovsky@oracle.com> - 2017-09-13 15:30 +0200
            Re: [PATCH 2/4] xen: limit grant v2 interface to the v1 functionality Juergen Gross <jgross@suse.com> - 2017-09-13 15:40 +0200
              Re: [PATCH 2/4] xen: limit grant v2 interface to the v1 functionality Boris Ostrovsky <boris.ostrovsky@oracle.com> - 2017-09-13 16:00 +0200
                Re: [PATCH 2/4] xen: limit grant v2 interface to the v1 functionality Juergen Gross <jgross@suse.com> - 2017-09-13 16:50 +0200
                Re: [PATCH 2/4] xen: limit grant v2 interface to the v1 functionality Boris Ostrovsky <boris.ostrovsky@oracle.com> - 2017-09-13 18:30 +0200
                Re: [PATCH 2/4] xen: limit grant v2 interface to the v1 functionality Juergen Gross <jgross@suse.com> - 2017-09-14 10:20 +0200
    Re: [PATCH 2/4] xen: limit grant v2 interface to the v1 functionality Boris Ostrovsky <boris.ostrovsky@oracle.com> - 2017-09-12 18:10 +0200

csiph-web