| Message ID | 57F39FE60200007800114D4C@prv-mh.provo.novell.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
On Tue, 4 Oct 2016, Jan Beulich wrote: > Commit e170622f95 ("xen/arm: p2m: Re-implement p2m_set_mem_access using > p2m_{set,get}_entry") eliminated the only user of level_sizes[], > causing gcc6 to warn about the unused variable (as it's a const one > older gcc versions apparently don't care to emit a warning). > > Signed-off-by: Jan Beulich <jbeulich@suse.com> Acked-by: Stefano Stabellini <sstabellini@kernel.org> > --- a/xen/arch/arm/p2m.c > +++ b/xen/arch/arm/p2m.c > @@ -30,8 +30,6 @@ static unsigned int __read_mostly p2m_ro > unsigned int __read_mostly p2m_ipa_bits; > > /* Helpers to lookup the properties of each level */ > -static const paddr_t level_sizes[] = > - { ZEROETH_SIZE, FIRST_SIZE, SECOND_SIZE, THIRD_SIZE }; > static const paddr_t level_masks[] = > { ZEROETH_MASK, FIRST_MASK, SECOND_MASK, THIRD_MASK }; > static const uint8_t level_orders[] = > > >
--- a/xen/arch/arm/p2m.c +++ b/xen/arch/arm/p2m.c @@ -30,8 +30,6 @@ static unsigned int __read_mostly p2m_ro unsigned int __read_mostly p2m_ipa_bits; /* Helpers to lookup the properties of each level */ -static const paddr_t level_sizes[] = - { ZEROETH_SIZE, FIRST_SIZE, SECOND_SIZE, THIRD_SIZE }; static const paddr_t level_masks[] = { ZEROETH_MASK, FIRST_MASK, SECOND_MASK, THIRD_MASK }; static const uint8_t level_orders[] =
Commit e170622f95 ("xen/arm: p2m: Re-implement p2m_set_mem_access using p2m_{set,get}_entry") eliminated the only user of level_sizes[], causing gcc6 to warn about the unused variable (as it's a const one older gcc versions apparently don't care to emit a warning). Signed-off-by: Jan Beulich <jbeulich@suse.com>