From mboxrd@z Thu Jan 1 00:00:00 1970 From: Oleg Nesterov Subject: [PATCH 5/5] fs/proc/task_mmu.c: update m->version in the main loop in m_start() Date: Thu, 7 Aug 2014 21:18:02 +0200 Message-ID: <20140807191802.GA1757@redhat.com> References: <20140807191718.GA1315@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Alexander Viro , Cyrill Gorcunov , David Howells , "David S. Miller" , "Eric W. Biederman" , "Kirill A. Shutemov" , "Kirill A. Shutemov" , Peter Zijlstra , Sasha Levin , linux-fsdevel@vger.kernel.org, linux-kernel@vger.kernel.org To: Andrew Morton Return-path: Content-Disposition: inline In-Reply-To: <20140807191718.GA1315@redhat.com> Sender: linux-kernel-owner@vger.kernel.org List-Id: linux-fsdevel.vger.kernel.org Change the main loop in m_start() to update m->version. Mostly for consistency, but this can help to avoid the same loop if the very 1st ->show() fails due to seq_overflow(). Signed-off-by: Oleg Nesterov --- fs/proc/task_mmu.c | 5 ++++- 1 files changed, 4 insertions(+), 1 deletions(-) diff --git a/fs/proc/task_mmu.c b/fs/proc/task_mmu.c index 1b897dc..4ec6b72 100644 --- a/fs/proc/task_mmu.c +++ b/fs/proc/task_mmu.c @@ -183,11 +183,14 @@ static void *m_start(struct seq_file *m, loff_t *ppos) m->version = 0; if (pos < mm->map_count) { - for (vma = mm->mmap; pos; pos--) + for (vma = mm->mmap; pos; pos--) { + m->version = vma->vm_start; vma = vma->vm_next; + } return vma; } + /* we do not bother to update m->version in this case */ if (pos == mm->map_count && priv->tail_vma) return priv->tail_vma; -- 1.5.5.1