[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2] ios: track sub IOS devices
From: |
Jose E. Marchesi |
Subject: |
Re: [PATCH v2] ios: track sub IOS devices |
Date: |
Sat, 29 Apr 2023 15:12:03 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) |
> Hi Jose.
>
> On Sun, Apr 23, 2023 at 03:11:15PM +0200, Jose E. Marchesi wrote:
>>
>> > In this commit, we keep track of number of sub-devices for each
>> > IO space. If the user close the super-IOS, it'll become a zombie
>> > IOS: it's dead but the corresponding `struct ios' is not un-allocated
>> > yet. When user closes the sub-IOS, and if it's the last sub-IOS,
>> > it'll de-allocate the corresponding `struct ios'.
>>
>> I like this idea. But I would suggest to move the sub-spaces business
>> completely to the IOS, at this point, instead of keeping a "sub" IOD.
>
> IMHO moving the logic from `ios-dev-sub.c' to `ios.c', only complicates the
> `ios.c' with no obvious benefits. I think keep the sub IOD is a good thing.
Agreed, the interplay between ios<->iod in this case can help to reduce
replication of logic. The sub IOD will be special, but no harm in that.
OK for master.
Thanks.