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

configure framework: detect make_req_fn type #1531

Merged
merged 1 commit into from
Oct 2, 2024

Conversation

mmichal10
Copy link
Contributor

No description provided.

@mmichal10 mmichal10 force-pushed the fix-make-req-type branch 4 times, most recently from d4fd518 to 6fc34bc Compare September 25, 2024 13:38
if compile_module $cur_name "struct block_device_operations x; x.submit_bio;" "linux/blkdev.h";
then
# If it's impossible to assign the casted return value of submit_bio()
# to a variable we're assuming the function is of type void.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

s/to a variable/to int/

# and removes make_request_fn
if compile_module $cur_name "struct block_device_operations x; x.submit_bio;" "linux/blkdev.h";
then
# If it's impossible to assign the casted return value of submit_bio()
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

s/assign the casted return value/cast the return value/

Copy link
Member

@robertbaldyga robertbaldyga Sep 25, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is not the assignment that fails. It's the casting operation itself.

Signed-off-by: Michal Mielewczyk <michal.mielewczyk@huawei.com>
Copy link
Contributor

@rafalste rafalste left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good except for inconsistent indentation (mixed 4 and 8 spaces). I've checked other configure.d files and there are even worse mixing of tabs and spaces, so you have at least only spaces in yours. ;)

@robertbaldyga robertbaldyga merged commit 7ce5b7e into Open-CAS:master Oct 2, 2024
2 checks passed
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.

3 participants