Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1391699

RE: [PATCH 1/3] Drivers: hv: balloon: don't crash when memory is added in non-sorted order

From KY Srinivasan <kys@microsoft.com>
Newsgroups linux.kernel
Subject RE: [PATCH 1/3] Drivers: hv: balloon: don't crash when memory is added in non-sorted order
Date 2016-04-30 23:40 +0200
Message-ID <rtKMh-6nl-15@gated-at.bofh.it> (permalink)
References <rtKjg-69D-9@gated-at.bofh.it> <rtKjg-69D-11@gated-at.bofh.it> <rtKjg-69D-7@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw



> -----Original Message-----
> From: Greg KH [mailto:gregkh@linuxfoundation.org]
> Sent: Saturday, April 30, 2016 2:03 PM
> To: KY Srinivasan <kys@microsoft.com>
> Cc: linux-kernel@vger.kernel.org; devel@linuxdriverproject.org;
> olaf@aepfle.de; apw@canonical.com; vkuznets@redhat.com;
> jasowang@redhat.com
> Subject: Re: [PATCH 1/3] Drivers: hv: balloon: don't crash when memory is
> added in non-sorted order
> 
> On Fri, Apr 29, 2016 at 11:45:17AM -0700, K. Y. Srinivasan wrote:
> > From: Vitaly Kuznetsov <vkuznets@redhat.com>
> >
> > When we iterate through all HA regions in handle_pg_range() we have an
> > assumption that all these regions are sorted in the list and the
> > 'start_pfn >= has->end_pfn' check is enough to find the proper region.
> > Unfortunately it's not the case with WS2016 where host can hot-add
> regions
> > in a different order. We end up modifying the wrong HA region and
> crashing
> > later on pages online. Modify the check to make sure we found the region
> > we were searching for while iterating. Fix the same check in pfn_covered()
> > as well.
> >
> > Signed-off-by: Vitaly Kuznetsov <vkuznets@redhat.com>
> > Signed-off-by: K. Y. Srinivasan <kys@microsoft.com>
> > ---
> >  drivers/hv/hv_balloon.c |    4 ++--
> >  1 files changed, 2 insertions(+), 2 deletions(-)
> 
> I only received patch 1/3 here, what happened to the other 2?
> 
> Can you resend the whole series?

Will do.

Thanks,

K. Y
> 
> thanks,
> 
> greg k-h

Back to linux.kernel | Previous | NextPrevious in thread | Find similar | Unroll thread


Thread

Re: [PATCH 1/3] Drivers: hv: balloon: don't crash when memory is  added in non-sorted order Greg KH <gregkh@linuxfoundation.org> - 2016-04-30 23:10 +0200
  RE: [PATCH 1/3] Drivers: hv: balloon: don't crash when memory is  added in non-sorted order KY Srinivasan <kys@microsoft.com> - 2016-04-30 23:40 +0200

csiph-web