diff options
| author | Yang Shi <[email protected]> | 2015-11-20 23:57:10 +0000 |
|---|---|---|
| committer | Linus Torvalds <[email protected]> | 2015-11-21 00:17:32 +0000 |
| commit | 50e55bf626ad3ebbca45c0c0d03eb1710a139638 (patch) | |
| tree | c5ec9ea7b3b7c16e9ddf6af986e095c2b5bb5e8e /drivers/fpga/fpga-mgr.c | |
| parent | various: fix pci_set_dma_mask return value checking (diff) | |
| download | kernel-50e55bf626ad3ebbca45c0c0d03eb1710a139638.tar.gz kernel-50e55bf626ad3ebbca45c0c0d03eb1710a139638.zip | |
mm/page-writeback.c: initialize m_dirty to avoid compile warning
When building kernel with gcc 5.2, the below warning is raised:
mm/page-writeback.c: In function 'balance_dirty_pages.isra.10':
mm/page-writeback.c:1545:17: warning: 'm_dirty' may be used uninitialized in this function [-Wmaybe-uninitialized]
unsigned long m_dirty, m_thresh, m_bg_thresh;
The m_dirty{thresh, bg_thresh} are initialized in the block of "if
(mdtc)", so if mdts is null, they won't be initialized before being used.
Initialize m_dirty to zero, also initialize m_thresh and m_bg_thresh to
keep consistency.
They are used later by if condition: !mdtc || m_dirty <=
dirty_freerun_ceiling(m_thresh, m_bg_thresh)
If mdtc is null, dirty_freerun_ceiling will not be called at all, so the
initialization will not change any behavior other than just ceasing the
compile warning.
(akpm: the patch actually reduces .text size by ~20 bytes on gcc-4.x.y)
[[email protected]: add comment]
Signed-off-by: Yang Shi <[email protected]>
Cc: Tejun Heo <[email protected]>
Signed-off-by: Andrew Morton <[email protected]>
Signed-off-by: Linus Torvalds <[email protected]>
Diffstat (limited to 'drivers/fpga/fpga-mgr.c')
0 files changed, 0 insertions, 0 deletions
