RE: [PATCH 1/1] Drivers: hv: Introduce a policy for controlling channel affinity

From: KY Srinivasan
Date: Wed Aug 31 2016 - 13:56:55 EST




> -----Original Message-----
> From: Greg KH [mailto:gregkh@xxxxxxxxxxxxxxxxxxx]
> Sent: Wednesday, August 31, 2016 4:14 PM
> To: KY Srinivasan <kys@xxxxxxxxxxxxx>
> Cc: linux-kernel@xxxxxxxxxxxxxxx; devel@xxxxxxxxxxxxxxxxxxxxxx;
> olaf@xxxxxxxxx; apw@xxxxxxxxxxxxx; vkuznets@xxxxxxxxxx;
> jasowang@xxxxxxxxxx; leann.ogasawara@xxxxxxxxxxxxx; K. Y. Srinivasan
> <kys@xxxxxxxxxxxxxxxxxxxxxx>
> Subject: Re: [PATCH 1/1] Drivers: hv: Introduce a policy for controlling channel
> affinity
>
> On Wed, Jul 06, 2016 at 06:02:15PM -0700, kys@xxxxxxxxxxxxxxxxxxxxxx
> wrote:
> > From: K. Y. Srinivasan <kys@xxxxxxxxxxxxxxxxxxxxxx>
> >
> > Introduce a mechanism to control how channels will be affinitized. We will
> > support two policies:
> >
> > 1. HV_BALANCED: All performance critical channels will be dstributed
> > evenly amongst all the available NUMA nodes. Once the Node is assigned,
> > we will assign the CPU based on a simple round robin scheme.
> >
> > 2. HV_LOCALIZED: Only the primary channels are distributed across all
> > NUMA nodes. Sub-channels will be in the same NUMA node as the primary
> > channel. This is the current behaviour.
> >
> > The default policy will be the HV_BALANCED as it can minimize the remote
> > memory access on NUMA machines with applications that span NUMA nodes.
> >
> > Signed-off-by: K. Y. Srinivasan <kys@xxxxxxxxxxxxxxxxxxxxxx>
> > Signed-off-by: K. Y. Srinivasan <kys@xxxxxxxxxxxxx>
> > ---
> > drivers/hv/channel_mgmt.c | 68 +++++++++++++++++++++++++---------------
> ----
> > include/linux/hyperv.h | 23 +++++++++++++++
> > 2 files changed, 62 insertions(+), 29 deletions(-)
>
> Also doesn't apply cleanly :(
>

I will fix this and resend.

K. Y