From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mark Salter Subject: Re: [PATCH 1/3] arm64: spin-table: handle unmapped cpu-release-addrs Date: Tue, 29 Jul 2014 11:30:24 -0400 Message-ID: <1406647824.753.12.camel@deneb.redhat.com> References: <1406630950-32432-1-git-send-email-ard.biesheuvel@linaro.org> <1406630950-32432-2-git-send-email-ard.biesheuvel@linaro.org> <1406646945.753.5.camel@deneb.redhat.com> <2385684.COvj9k7y2z@wuerfel> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <2385684.COvj9k7y2z@wuerfel> Sender: linux-efi-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Arnd Bergmann Cc: linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, Ard Biesheuvel , mark.rutland-5wv7dgnIgG8@public.gmane.org, linux-efi-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, matt.fleming-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org, leif.lindholm-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org List-Id: linux-efi@vger.kernel.org On Tue, 2014-07-29 at 17:20 +0200, Arnd Bergmann wrote: > On Tuesday 29 July 2014 11:15:45 Mark Salter wrote: > > > - > > > - __flush_dcache_area(release_addr, sizeof(release_addr[0])); > > > + writeq_relaxed(__pa(secondary_holding_pen), release_addr); > > > + __flush_dcache_area(release_addr, sizeof(*release_addr)); > > > > __flush_dcache_area((__force void *)release_addr, ... > > > > to avoid sparse warning. > > > > I think it would be cleaner to drop the __iomem annotation and use vmap() > rather than ioremap(). That requires having a 'struct page' though, which > I'm not sure you have. > You won't. If you did have a struct page, then __va() would work.