lkml.org 
[lkml]   [2016]   [Mar]   [31]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
Date
SubjectRe: [PATCH 6/6] VMware balloon: Update vmw_balloon.c to use the VMW_PORT macro
Hi,

Does any one know when this series will be applied?

Sinclair

On Mon, Feb 08, 2016 at 11:41:57AM -0800, Greg KH wrote:
> On Tue, Jan 19, 2016 at 01:46:05PM -0800, Sinclair Yeh wrote:
> > Updated VMWARE_BALLOON_CMD to use the common VMW_PORT macro.
> > Doing this rather than replacing all instances of VMWARE_BALLOON_CMD
> > to minimize code change.
> >
> > Signed-off-by: Sinclair Yeh <syeh@vmware.com>
> > Reviewed-by: Thomas Hellstrom <thellstrom@vmware.com>
> > Reviewed-by: Alok N Kataria <akataria@vmware.com>
> > Acked-by: Xavier Deguillard <xdeguillard@vmware.com>
> > Cc: pv-drivers@vmware.com
> > Cc: Xavier Deguillard <xdeguillard@vmware.com>
> > Cc: linux-kernel@vger.kernel.org
> > Cc: virtualization@lists.linux-foundation.org
> > Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
>
>
> Acked-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
>
> > -#define VMWARE_BALLOON_CMD(cmd, arg1, arg2, result) \
> > -({ \
> > - unsigned long __status, __dummy1, __dummy2, __dummy3; \
> > - __asm__ __volatile__ ("inl %%dx" : \
> > - "=a"(__status), \
> > - "=c"(__dummy1), \
> > - "=d"(__dummy2), \
> > - "=b"(result), \
> > - "=S" (__dummy3) : \
> > - "0"(VMW_BALLOON_HV_MAGIC), \
> > - "1"(VMW_BALLOON_CMD_##cmd), \
> > - "2"(VMW_BALLOON_HV_PORT), \
> > - "3"(arg1), \
> > - "4" (arg2) : \
> > - "memory"); \
> > - if (VMW_BALLOON_CMD_##cmd == VMW_BALLOON_CMD_START) \
> > - result = __dummy1; \
> > - result &= -1UL; \
> > - __status & -1UL; \
> > +#define VMWARE_BALLOON_CMD(cmd, arg1, arg2, result) \
> > +({ \
> > + unsigned long __status, __dummy1, __dummy2; \
> > + unsigned long __si, __di; \
> > + VMW_PORT(VMW_BALLOON_CMD_##cmd, arg1, arg2, 0, \
> > + VMW_BALLOON_HV_PORT, VMW_BALLOON_HV_MAGIC, \
> > + __status, result, __dummy1, __dummy2, __si, __di); \
> > + if (VMW_BALLOON_CMD_##cmd == VMW_BALLOON_CMD_START) \
> > + result = __dummy1; \
> > + result &= -1UL; \
> > + __status & -1UL; \
> > })
>
> Honestly, it doesn't look anymore "readable" to me, but it's your code
> to maintain, not mine... :)
>

\
 
 \ /
  Last update: 2016-03-31 17:01    [W:0.065 / U:0.392 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site