Re: [PATCH v7 00/19] Add Command Duration Limits support

[Niklas Cassel <Niklas.Cassel@wdc.com>]

On Tue, May 23, 2023 at 07:08:27PM +0900, Damien Le Moal wrote:
> On 5/23/23 18:56, Niklas Cassel wrote:
> > On Mon, May 22, 2023 at 05:41:19PM -0400, Martin K. Petersen wrote:
> >>
> >> Niklas,
> >>
> >>> This series adds support for Command Duration Limits.
> >>
> >> Applied to 6.5/scsi-staging, thanks!
> > 
> > Thank you Martin!
> > 
> > 
> > Damien, Martin,
> > considering that the libata changes depend on the scsi changes,
> > and considering that further libata EH cleanups are planned for
> > 6.5 now when the IPR driver is gone, I think that the best move
> > is to follow the advice of:
> > https://docs.kernel.org/maintainer/rebasing-and-merging.html#merging-from-sibling-or-upstream-trees
> 
> Hannes cleanup of EH will create a conflict with the scsi tree but can go in
> through the ata tree independently so I was not planning on doing a rebase,
> especially not on the scsi tree. I will notify Stephen about the conflict send
> him a resolution to apply and carry for linux-next. When the 6.5 merge window
> open, I will wait for the James to send the scsi PR and send my PR to Linus
> after that with the conflict resolution, as usual.

I wasn't suggesting that you do a rebase on the scsi tree, I was
suggesting for a topic branch to be merged through both trees.

But I was just thinking how to make it as easy as possible to
handle additional libata patches that should go on top of the
CDL series (without creating conflicts for Stephen and Linus).

> 
> So far, I do not see any big issue with that.

If you think that the conflicts will be trivial, feel free to
ignore my suggestion :)


Kind regards,
Niklas