Re: [PATCH v3 0/4] cxl: Handle background commands

From: Dan Williams
Date: Wed May 24 2023 - 00:19:38 EST

Davidlohr Bueso wrote:
> On Tue, 23 May 2023, Dan Williams wrote:
> >Like Linus I want to see the conflicts. Whenever possible base on latest
> >Linus tag like v6.4-rc3.
> So I'm clear, do you want me to resend this series based on -rc3?

No, not this time but for next time my preference is "when in doubt, pick
a mainline tag".

> Frankly I'm always confused as to what is the correct/preferred branch to
> do development on.

My plan is to do better about publishing topic branches for
work-in-progress items like this. So now we have for-6.5/cxl-background
for this common baseline for sanitization and firmware update. At least
one of those will need to be queued before this topic moves forward
since the new infrastructure needs a user.

The problem with picking 'next' or to a lesser extent 'fixes' as a
development baseline is that it ties unrelated topics together. If
something happens and we decided a topic neeeds to drop out of 'next' or
be rebased, that's much easier to do if random topics have not grown
implicit silent depenendencies over time.

Like in this case these patches do not apply to v6.4-rc3 because they
collide with the movement of cxl_await_media_ready() to cxl_pci.
However, nothing about that media-ready fix is needed for background
command support so the patches can just depend on a mainline tag.