Sorry about late reply. On Wed, Feb 01, 2023 at 09:06:35AM +0100, Christoph Hellwig wrote: > On Fri, Jan 27, 2023 at 01:40:00PM -1000, Tejun Heo wrote: > > So, we're shifting the dead test from schedule to the actual throttle path, > > which makes sense but I think this should at least be mentioned in the > > description if not put in its own patch. > > That's what I tried to say with: > > "Move the check for the dead disk to the latest place now that is is > unbundled from the reference grab." > > what else would you want me to write? It'd be better if the paragraph actually said where that change is being made along with why and whether it causes any behavior difference. This seems safe to me but I can't tell why it's being moved. Maybe it's better because that's "latest" but that's a lot hanging on that single word. If this is better because we're testing it later instead of to accomodate other changes being made, this should be a separate patch, right? It's a subtle behavior change which is buried with a bunch of other mechanical changes. Thanks. -- tejun
