Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

NAS-130424 / None / scsi: mpi3mr: NVMe encapsulation support for tri-mode HBA #201

Merged
merged 2 commits into from
Nov 27, 2024

Conversation

ixhamza
Copy link
Contributor

@ixhamza ixhamza commented Nov 26, 2024

@ixhamza ixhamza requested a review from amotin November 26, 2024 12:27
@bugclerk bugclerk changed the title scsi: mpi3mr: NVMe encapsulation support for tri-mode HBA NAS-130424 / None / scsi: mpi3mr: NVMe encapsulation support for tri-mode HBA Nov 26, 2024
The driver serializes IOCTLs through a mutex lock,
but the access to IOCTL data buffer is not guarded
by the lock.It causes multiple user threads writing
to the driver's ioctl buffer simultaneously.

Access to the IOCTL data buffer is guarded by the IOCTL mutex lock.

Signed-off-by: Sumit Saxena <[email protected]>
Signed-off-by: Ranjan Kumar <[email protected]>
@ixhamza ixhamza requested a review from amotin November 27, 2024 16:25
@ixhamza
Copy link
Contributor Author

ixhamza commented Nov 27, 2024

@usaleem-ix - I have cherry-picked mpi3mr: synchronize the access to ioctl data buffer after @amotin identified a bug during internal review though the commit is still pending upstream approval. Just a heads up, in case you encounter conflicts during a later merge.

@ixhamza ixhamza merged commit 69dd353 into truenas/linux-6.12 Nov 27, 2024
6 checks passed
@bugclerk
Copy link

Not updating JIRA ticket https://ixsystems.atlassian.net/browse/NAS-130424 target versions as no JIRA version corresponds to this PR

@bugclerk
Copy link

This PR has been merged and conversations have been locked.
If you would like to discuss more about this issue please use our forums or raise a Jira ticket.

@truenas truenas locked as resolved and limited conversation to collaborators Nov 27, 2024
@ixhamza ixhamza deleted the NAS-130424-6.12-2 branch November 27, 2024 16:36
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants