From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ian Campbell Subject: Re: [PATCH v8 2/8] xen/arm: rename *_xen_dcache_* operations to *_dcache_* Date: Mon, 20 Oct 2014 15:55:07 +0100 Message-ID: <1413816907.13796.19.camel@citrix.com> References: <1413798499-28314-2-git-send-email-stefano.stabellini@eu.citrix.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1413798499-28314-2-git-send-email-stefano.stabellini@eu.citrix.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Stefano Stabellini Cc: julien.grall@citrix.com, xen-devel@lists.xensource.com, JBeulich@suse.com List-Id: xen-devel@lists.xenproject.org On Mon, 2014-10-20 at 10:48 +0100, Stefano Stabellini wrote: > Signed-off-by: Stefano Stabellini > Reviewed-by: Julien Grall I always prefer the commit log of a change like this to contain at least the word "because", so I don't have to guess in 6 months time why this was considered a good idea. This seems to stand independently from the rest and appears to be purely mechanical so no release risk, but I've not seen (or looked for) any comment from Konrad on this series. > /* Macros for flushing a single small item. The predicate is always > * compile-time constant so this will compile down to 3 instructions in > * the common case. */ > -#define clean_xen_dcache(x) do { \ > +#define clean_dcache(x) do { \ > typeof(x) *_p = &(x); \ > if ( sizeof(x) > MIN_CACHELINE_BYTES || sizeof(x) > alignof(x) ) \ > - clean_xen_dcache_va_range(_p, sizeof(x)); \ > + clean_dcache_va_range(_p, sizeof(x)); \ A bit too mechanical here it seems... Fix up the whitespace and you can add: Acked-by: Ian Campbell > else \ > asm volatile ( \ > "dsb sy;" /* Finish all earlier writes */ \ > - __clean_xen_dcache_one(0) \ > + __clean_dcache_one(0) \ > "dsb sy;" /* Finish flush before continuing */ \ > : : "r" (_p), "m" (*_p)); \ > } while (0) > > -#define clean_and_invalidate_xen_dcache(x) do { \ > +#define clean_and_invalidate_dcache(x) do { \ > typeof(x) *_p = &(x); \ > if ( sizeof(x) > MIN_CACHELINE_BYTES || sizeof(x) > alignof(x) ) \ > - clean_and_invalidate_xen_dcache_va_range(_p, sizeof(x)); \ > + clean_and_invalidate_dcache_va_range(_p, sizeof(x)); \ > else \ > asm volatile ( \ > "dsb sy;" /* Finish all earlier writes */ \ > - __clean_and_invalidate_xen_dcache_one(0) \ > + __clean_and_invalidate_dcache_one(0) \ > "dsb sy;" /* Finish flush before continuing */ \ > : : "r" (_p), "m" (*_p)); \ > } while (0)