On Mon, Oct 23, 2023 at 02:00:00PM +0000, Mark O'Donovan wrote:I'd suggest to wait, as we'll need to fixup nvmet, too.
The first patch is a small unrelated fix which makes it clear that the
response data section of the Success1 message is not optional.
The second patch removes use of the host sequence number (S2) as an
indicator of bi-directional authentication.
The third patch causes us to always set the host sequence number (S2)
to a non-zero value instead of the 0 value reserved for the secure
channel feature.
Should these go in now for 6.6? We're pretty close to the end. If for
6.7, there is a merge conflict that I think would be easiest resolved if
I wait until the block tree resyncs after the 6.7 merge window opens.