Re: [Xen-devel] [PATCH] xen/privcmd: remove unused variable pageidx
On 11/08/2017 08:30 AM, Juergen Gross wrote: > On 08/11/17 14:00, Colin King wrote: >> From: Colin Ian King>> >> Variable pageidx is assigned a value but it is never read, hence it >> is redundant and can be removed. Cleans up clang warning: >> >> drivers/xen/privcmd.c:199:2: warning: Value stored to 'pageidx' >> is never read >> >> Signed-off-by: Colin Ian King > Reviewed-by: Juergen Gross Applied to for-linus-4.15. -boris ___ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel
Re: [Xen-devel] [PATCH] xen/privcmd: remove unused variable pageidx
On 08/11/17 14:00, Colin King wrote: > From: Colin Ian King> > Variable pageidx is assigned a value but it is never read, hence it > is redundant and can be removed. Cleans up clang warning: > > drivers/xen/privcmd.c:199:2: warning: Value stored to 'pageidx' > is never read > > Signed-off-by: Colin Ian King Reviewed-by: Juergen Gross Juergen ___ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel
[Xen-devel] [PATCH] xen/privcmd: remove unused variable pageidx
From: Colin Ian KingVariable pageidx is assigned a value but it is never read, hence it is redundant and can be removed. Cleans up clang warning: drivers/xen/privcmd.c:199:2: warning: Value stored to 'pageidx' is never read Signed-off-by: Colin Ian King --- drivers/xen/privcmd.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/drivers/xen/privcmd.c b/drivers/xen/privcmd.c index feca75b07fdd..1c909183c42a 100644 --- a/drivers/xen/privcmd.c +++ b/drivers/xen/privcmd.c @@ -191,13 +191,10 @@ static int traverse_pages_block(unsigned nelem, size_t size, void *state) { void *pagedata; - unsigned pageidx; int ret = 0; BUG_ON(size > PAGE_SIZE); - pageidx = PAGE_SIZE; - while (nelem) { int nr = (PAGE_SIZE/size); struct page *page; -- 2.14.1 ___ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel