>>> On 09.02.15 at 19:53, <tamas.leng...@zentific.com> wrote:
> ---
> v4: Style fixes
But why did you not do all of them then?
> --- a/xen/arch/x86/mm/mem_paging.c
> +++ b/xen/arch/x86/mm/mem_paging.c
> @@ -25,31 +25,29 @@
> #include <xen/mem_event.h>
>
>
> -int mem_paging_memop(struct domain *d, xen_mem_event_op_t *mec)
> +int mem_paging_memop(struct domain *d, xen_mem_paging_op_t *mpc)
> {
> if ( unlikely(!d->mem_event->paging.ring_page) )
> return -ENODEV;
>
> - switch( mec->op )
> + switch( mpc->op )
> {
> case XENMEM_paging_op_nominate:
> {
> - unsigned long gfn = mec->gfn;
> - return p2m_mem_paging_nominate(d, gfn);
> + return p2m_mem_paging_nominate(d, mpc->gfn);
> }
> break;
I'm relatively certain I said this before: The braces now become
pointless and hence should be dropped.
Again for the parts this is applicable to,
Acked-by: Jan Beulich <jbeul...@suse.com>
provided the above gets adjusted throughout.
Jan
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel
