Skip to content

Commit 377840e

Browse files
author
Trond Myklebust
committed
NFS: Remove the redundant function nfs_pgio_has_mirroring()
We need to trust that desc->pg_mirror_idx is set correctly, whether or not mirroring is enabled. Signed-off-by: Trond Myklebust <[email protected]>
1 parent 862f35c commit 377840e

File tree

2 files changed

+2
-11
lines changed

2 files changed

+2
-11
lines changed

fs/nfs/internal.h

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -274,12 +274,6 @@ void nfs_free_request(struct nfs_page *req);
274274
struct nfs_pgio_mirror *
275275
nfs_pgio_current_mirror(struct nfs_pageio_descriptor *desc);
276276

277-
static inline bool nfs_pgio_has_mirroring(struct nfs_pageio_descriptor *desc)
278-
{
279-
WARN_ON_ONCE(desc->pg_mirror_count < 1);
280-
return desc->pg_mirror_count > 1;
281-
}
282-
283277
static inline bool nfs_match_open_context(const struct nfs_open_context *ctx1,
284278
const struct nfs_open_context *ctx2)
285279
{

fs/nfs/pagelist.c

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -33,9 +33,7 @@ static const struct rpc_call_ops nfs_pgio_common_ops;
3333
struct nfs_pgio_mirror *
3434
nfs_pgio_current_mirror(struct nfs_pageio_descriptor *desc)
3535
{
36-
return nfs_pgio_has_mirroring(desc) ?
37-
&desc->pg_mirrors[desc->pg_mirror_idx] :
38-
&desc->pg_mirrors[0];
36+
return &desc->pg_mirrors[desc->pg_mirror_idx];
3937
}
4038
EXPORT_SYMBOL_GPL(nfs_pgio_current_mirror);
4139

@@ -1231,8 +1229,7 @@ static void nfs_pageio_complete_mirror(struct nfs_pageio_descriptor *desc,
12311229
struct nfs_pgio_mirror *mirror = &desc->pg_mirrors[mirror_idx];
12321230
u32 restore_idx = desc->pg_mirror_idx;
12331231

1234-
if (nfs_pgio_has_mirroring(desc))
1235-
desc->pg_mirror_idx = mirror_idx;
1232+
desc->pg_mirror_idx = mirror_idx;
12361233
for (;;) {
12371234
nfs_pageio_doio(desc);
12381235
if (desc->pg_error < 0 || !mirror->pg_recoalesce)

0 commit comments

Comments
 (0)