Messages in this thread | | | Date | Tue, 23 May 2023 21:19:24 -0700 | From | Dan Williams <> | Subject | Re: [PATCH v3 0/4] cxl: Handle background commands |
| |
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.
| |