Re: [PATCH 0/2] shm: omit forced shm destroy if task IPC namespace was changed
From: Eric W. Biederman
Date: Fri Sep 24 2021 - 12:45:21 EST
Manfred Spraul <manfred@xxxxxxxxxxxxxxxx> writes:
> Hi Eric,
> I'd like to restart the discussion, the issue should be fixed.
I am going to come right out and say having read through everything
my suggests were confused and wrong.
Somehow I thought the cleanups I was suggesting world result in
shm_clist only being modified from the task that owns the list.
Which would result in no need to use a per-task list.
Having looked through my suggestions again I was completely wrong.
The only useful bit I have to contribute from that original suggestion
is let's please have smallish patches that change one thing at a time.
That code is sufficiently interesting that it is way too easy to get
lost in big patches.
I am not going to discuss my broken suggestions right now because every
time I look into them I go into a rabbit hole and I don't get anything
productive done on fixing these issues, just something close and
frustrating. Apologies if I derailed your patch.