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

cas_cache: fix "blk_update_request: operation not supported" error #1213

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

hammerg
Copy link
Contributor

@hammerg hammerg commented May 25, 2022

A write request with the REQ_RAHEAD flag enabled cause the nvme driver
to send a write command with access frequency value that is reserved
(at least on specification version 1.4c). NVME devices might fail this
write command with an unsupported error. So we now clear the request's
flags based on its direction.

Signed-off-by: Gal Hammer gal.hammer@huawei.com
Signed-off-by: Shai Fultheim shai.fultheim@huawei.com

A write request with the REQ_RAHEAD flag enabled cause the nvme driver
to send a write command with access frequency value that is reserved
(at least on specification version 1.4c). NVME devices might fail this
write command with an unsupported error. So we now clear the request's
flag based on its direction.

Signed-off-by: Gal Hammer <gal.hammer@huawei.com>
Signed-off-by: Shai Fultheim <shai.fultheim@huawei.com>
@CAS-Linux-Jenkins
Copy link

Can one of the admins verify this patch?

@jfckm
Copy link

jfckm commented Jun 3, 2022

@CAS-Linux-Jenkins ok to test

@hammerg
Copy link
Contributor Author

hammerg commented Jun 29, 2022

Anyone? Thanks, Gal.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants