Re: [PATCH v2] Argo: don't obtain excess page references

2024-03-07 Thread George Dunlap
On Wed, Mar 6, 2024 at 11:38 PM Christopher Clark wrote: > > On Sun, Feb 18, 2024 at 10:01 AM Julien Grall wrote: > > > > Hi Jan, > > > > On 14/02/2024 10:12, Jan Beulich wrote: > > > find_ring_mfn() already holds a page reference when trying to obtain a > > > writable type reference. We shouldn'

Re: [PATCH v2] Argo: don't obtain excess page references

2024-03-06 Thread Christopher Clark
On Sun, Feb 18, 2024 at 10:01 AM Julien Grall wrote: > > Hi Jan, > > On 14/02/2024 10:12, Jan Beulich wrote: > > find_ring_mfn() already holds a page reference when trying to obtain a > > writable type reference. We shouldn't make assumptions on the general > > reference count limit being effectiv

Re: Ping: [PATCH v2] Argo: don't obtain excess page references

2024-03-06 Thread Christopher Clark
On Wed, Feb 28, 2024 at 6:52 AM Jan Beulich wrote: > > On 18.02.2024 19:01, Julien Grall wrote: > > On 14/02/2024 10:12, Jan Beulich wrote: > >> find_ring_mfn() already holds a page reference when trying to obtain a > >> writable type reference. We shouldn't make assumptions on the general > >> re

Ping: [PATCH v2] Argo: don't obtain excess page references

2024-02-28 Thread Jan Beulich
On 18.02.2024 19:01, Julien Grall wrote: > On 14/02/2024 10:12, Jan Beulich wrote: >> find_ring_mfn() already holds a page reference when trying to obtain a >> writable type reference. We shouldn't make assumptions on the general >> reference count limit being effectively "infinity". Obtain merely

Re: [PATCH v2] Argo: don't obtain excess page references

2024-02-18 Thread Julien Grall
Hi Jan, On 14/02/2024 10:12, Jan Beulich wrote: find_ring_mfn() already holds a page reference when trying to obtain a writable type reference. We shouldn't make assumptions on the general reference count limit being effectively "infinity". Obtain merely a type ref, re-using the general ref by o

[PATCH v2] Argo: don't obtain excess page references

2024-02-14 Thread Jan Beulich
find_ring_mfn() already holds a page reference when trying to obtain a writable type reference. We shouldn't make assumptions on the general reference count limit being effectively "infinity". Obtain merely a type ref, re-using the general ref by only dropping the previously acquired one in the cas