lkml.org 
[lkml]   [2022]   [Sep]   [5]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
SubjectRe: [PATCH 0/3] iova: Some misc changes
From
On 2022-09-02 13:18, John Garry wrote:
> On 17/08/2022 11:24, John Garry wrote:
>> On 17/08/2022 11:09, John Garry wrote:
>>> This series includes:
>>> - remove checks in the code which are not required
>>> - the re-org, which I had originally posted separately
>>
>> BTW, Can we drop the !IOMMU_IOVA stubs in iova.h? I could not find a
>> kernel config which actually exercises that code (so testing changes
>> there is difficult).
>
> Any thoughts on this? Since I got no review of patch #3 I assume that it
> is not keenly welcome either.

Yeah, I applied patch #3 to have a look at the result, but couldn't
really convince myself either way - there are certainly a few functions
in weirdly incongruous places at the moment, but afterwards we end up
with certain other things in rather contrived order for the sake of
avoiding declarations, so overall it just didn't feel objectively better
to me. Plus the fact that rewriting nearly 2/3 of the file stands to
make backporting tweaks or fixes unnecessarily painful is hard to
overlook. Hence I guess I'm leaning towards "worth trying to see how it
looked, but let's not".

As for the stubs, it seems that they're currently unused due to linkage
issues with IOMMU_IOVA=m - if we want better compile-test coverage, I
wonder if we couldn't replace the IS_ENABLED() with IS_REACHABLE() and
restore some of the previously-conditional selects?

Robin.

\
 
 \ /
  Last update: 2022-09-05 17:53    [W:0.063 / U:0.396 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site