[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <c05c2217-3fc3-63a0-b332-004603315f84@arm.com>
Date: Fri, 28 Jan 2022 16:54:02 +0000
From: Robin Murphy <robin.murphy@....com>
To: John Garry <john.garry@...wei.com>,
"joro@...tes.org" <joro@...tes.org>,
"will@...nel.org" <will@...nel.org>,
"mst@...hat.com" <mst@...hat.com>,
"jasowang@...hat.com" <jasowang@...hat.com>
Cc: "xieyongji@...edance.com" <xieyongji@...edance.com>,
"iommu@...ts.linux-foundation.org" <iommu@...ts.linux-foundation.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
"virtualization@...ts.linux-foundation.org"
<virtualization@...ts.linux-foundation.org>,
Linuxarm <linuxarm@...wei.com>
Subject: Re: [PATCH] iommu/iova: Separate out rcache init
On 2022-01-28 11:32, John Garry wrote:
> On 26/01/2022 17:00, Robin Murphy wrote:
>> As above, I vote for just forward-declaring the free routine in iova.c
>> and keeping it entirely private.
>
> BTW, speaking of forward declarations, it's possible to remove all the
> forward declarations in iova.c now that the FQ code is gone - but with a
> good bit of rearranging. However I am not sure how much people care
> about that or whether the code layout is sane...
Indeed, I was very tempted to raise the question there of whether there
was any more cleanup or refactoring that could be done to justify
collecting all the rcache code together at the top of iova.c. But in the
end I didn't, so my opinion still remains a secret...
Robin.
Powered by blists - more mailing lists