Skip to content

loop: three small cleanups - #142

Open
blktests-ci-block-trial[bot] wants to merge 4 commits into
master_basefrom
series/717=>master
Open

loop: three small cleanups#142
blktests-ci-block-trial[bot] wants to merge 4 commits into
master_basefrom
series/717=>master

Conversation

@blktests-ci-block-trial

Copy link
Copy Markdown

Pull request for series with
subject: loop: three small cleanups
version: 1
url: http://redsun45:8000/project/linux-block/list/?series=717

@blktests-ci-block-trial

Copy link
Copy Markdown
Author

Upstream branch: 66498c7
series: http://redsun45:8000/project/linux-block/list/?series=717
version: 1

@blktests-ci-block-trial

Copy link
Copy Markdown
Author

Upstream branch: 73ae59e
series: http://redsun45:8000/project/linux-block/list/?series=717
version: 1

@blktests-ci-block-trial

Copy link
Copy Markdown
Author

Upstream branch: 45c13f3
series: http://redsun45:8000/project/linux-block/list/?series=717
version: 1

@blktests-ci-block-trial

Copy link
Copy Markdown
Author

Upstream branch: 45c13f3
series: http://redsun45:8000/project/linux-block/list/?series=717
version: 1

@blktests-ci-block-trial

Copy link
Copy Markdown
Author

Upstream branch: 502d457
series: http://redsun45:8000/project/linux-block/list/?series=717
version: 1

@blktests-ci-block-trial

Copy link
Copy Markdown
Author

Upstream branch: 73e3f07
series: http://redsun45:8000/project/linux-block/list/?series=717
version: 1

@blktests-ci-block-trial

Copy link
Copy Markdown
Author

Upstream branch: 73e3f07
series: http://redsun45:8000/project/linux-block/list/?series=717
version: 1

@blktests-ci-block-trial

Copy link
Copy Markdown
Author

Upstream branch: 18fbf51
series: http://redsun45:8000/project/linux-block/list/?series=717
version: 1

@blktests-ci-block-trial

Copy link
Copy Markdown
Author

Upstream branch: f7e3f4d
series: http://redsun45:8000/project/linux-block/list/?series=717
version: 1

@blktests-ci-block-trial

Copy link
Copy Markdown
Author

Upstream branch: bf10795
series: http://redsun45:8000/project/linux-block/list/?series=767
version: 2

@blktests-ci-block-trial

Copy link
Copy Markdown
Author

Upstream branch: abdf623
series: http://redsun45:8000/project/linux-block/list/?series=767
version: 2

@blktests-ci-block-trial

Copy link
Copy Markdown
Author

Upstream branch: 786262b
series: http://redsun45:8000/project/linux-block/list/?series=767
version: 2

@blktests-ci-block-trial

Copy link
Copy Markdown
Author

Upstream branch: 89a3129
series: http://redsun45:8000/project/linux-block/list/?series=767
version: 2

@blktests-ci-block-trial

Copy link
Copy Markdown
Author

Upstream branch: 89a3129
series: http://redsun45:8000/project/linux-block/list/?series=767
version: 2

@blktests-ci-block-trial

Copy link
Copy Markdown
Author

Upstream branch: 940de59
series: http://redsun45:8000/project/linux-block/list/?series=767
version: 2

@blktests-ci-block-trial

Copy link
Copy Markdown
Author

Upstream branch: 841e384
series: http://redsun45:8000/project/linux-block/list/?series=767
version: 2

@blktests-ci-block-trial

Copy link
Copy Markdown
Author

Upstream branch: a500db7
series: http://redsun45:8000/project/linux-block/list/?series=767
version: 2

Tao Cui added 3 commits September 4, 2026 08:44
The kerneldoc of loop_set_size() points readers at loop_validate_size(),
which does not exist in the tree.  Drop the reference.

Signed-off-by: Tao Cui <cuitao@kylinos.cn>
Reviewed-by: Bart Van Assche <bvanassche@acm.org>
Reviewed-by: Christoph Hellwig <hch@lst.de>
simple_strtol() is deprecated and swallows errors.  Use kstrtoint()
in the max_loop setup code; an invalid option keeps the default and
now says so.

Only mark max_loop_specified when the option was parsed successfully.
With simple_strtol() a garbage string yields max_loop = 0, and
loop_probe()'s "max_loop_specified && max_loop" check short-circuits
on 0, so legacy autoloading is not capped.  Keeping max_loop at its
default while still setting max_loop_specified would instead turn the
default into a hard upper bound for dynamic device creation.  This
mirrors what max_loop_param_set_int() already does for the module
parameter.

Signed-off-by: Tao Cui <cuitao@kylinos.cn>
lo_req_flush() never uses its request argument.  Drop it.

Signed-off-by: Tao Cui <cuitao@kylinos.cn>
Reviewed-by: Bart Van Assche <bvanassche@acm.org>
Reviewed-by: Christoph Hellwig <hch@lst.de>
@blktests-ci-block-trial
blktests-ci-block-trial Bot force-pushed the master_base branch 3 times, most recently from 4a1d19f to d9169e2 Compare September 13, 2026 00:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant