[LU-8632] Address of function 'page_count' is used instead of a local variable

• Type: Bug
• Resolution: Fixed
• Priority: Minor
• Fix Version/s: Lustre 2.9.0
• Labels: kw
• Severity: 3
• Assignee: Dmitry Eremin (Inactive)
• Reporter: Dmitry Eremin (Inactive)

Description

In the following commit

commit 7f2aae8d80a73de7408668bbe569d5f4d8553efe
Author: Jinshan Xiong <[e-mail]>
Date:   Mon Sep 12 11:17:10 2016 -0700

    LU-8135 osc: limits the number of chunks in write RPC
    
    OSC has to make sure that it won't issue write RPCs with too many
    chunks otherwise it will casue ZFS to create transactions much
    bigger than DMU_MAX_ACCESS in size, which will end up with write
    failure.
    
    Signed-off-by: Jinshan Xiong <[e-mail]>
    Change-Id: Ib68b09afca35c253ef0a6b569f64f555e08bd11b
    Reviewed-on: http://review.whamcloud.com/22369
    Tested-by: Jenkins
    Reviewed-by: Andreas Dilger <[e-mail]>
    Reviewed-by: Patrick Farrell <[e-mail]>
    Tested-by: Maloo <[e-mail]>
    Reviewed-by: Oleg Drokin <[e-mail]>

the local variable 'page_count' in osc_send_read_rpc() was removed:

-       unsigned int page_count = 0;

but ASSERT with this variable was not:

2171:	LASSERT(page_count > 0);

Issue Links

• is related to: LU-8135 sanity test_101g fails with 'not all RPCs are 16 MiB BRW rpcs'