linux-kernel - RE: [PATCH 1/1] Drivers: hv: vmbus: Include the limit on the number of pfns we can handle

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <b484d406499b496bad392a56da7766d5@BY2PR03MB299.namprd03.prod.outlook.com>
Date:	Fri, 7 Mar 2014 06:58:30 +0000
From:	KY Srinivasan <kys@...rosoft.com>
To:	Greg KH <gregkh@...uxfoundation.org>
CC:	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	"devel@...uxdriverproject.org" <devel@...uxdriverproject.org>,
	"olaf@...fle.de" <olaf@...fle.de>,
	"apw@...onical.com" <apw@...onical.com>,
	"jasowang@...hat.com" <jasowang@...hat.com>
Subject: RE: [PATCH 1/1] Drivers: hv: vmbus: Include the limit on the number
 of pfns we can handle



> -----Original Message-----
> From: Greg KH [mailto:gregkh@...uxfoundation.org]
> Sent: Friday, March 7, 2014 12:19 PM
> To: KY Srinivasan
> Cc: linux-kernel@...r.kernel.org; devel@...uxdriverproject.org;
> olaf@...fle.de; apw@...onical.com; jasowang@...hat.com
> Subject: Re: [PATCH 1/1] Drivers: hv: vmbus: Include the limit on the number
> of pfns we can handle
> 
> On Thu, Mar 06, 2014 at 11:15:08PM -0800, K. Y. Srinivasan wrote:
> > Increase the maximum number of pfns we can handle is a single vmbus
> packet.
> 
> Is that a correct sentance?
I will fix the typo in the sentence.
> 
> 
> >
> > Signed-off-by: K. Y. Srinivasan <kys@...rosoft.com>
> > ---
> >  include/linux/hyperv.h |    2 +-
> >  1 files changed, 1 insertions(+), 1 deletions(-)
> >
> > diff --git a/include/linux/hyperv.h b/include/linux/hyperv.h index
> > 330ec44..ab7359f 100644
> > --- a/include/linux/hyperv.h
> > +++ b/include/linux/hyperv.h
> > @@ -37,7 +37,7 @@
> >  #include <linux/mod_devicetable.h>
> >
> >
> > -#define MAX_PAGE_BUFFER_COUNT				19
> > +#define MAX_PAGE_BUFFER_COUNT				32
> 
> What is the problem if this number isn't changed?
On the networking side we may send frames that require more than 19 PFN entries.
Number 32 is imposed by some limitations by the vmbus data structures. For some reason
MAX_PAGE_BUFFER_COUNT was set to a lower value and I want to bump this up to what vmbus
can support.
> 
> Again, I'm going to have to ask you, what kernel is this for?
> 
> Please remember this when sending patches that look like they might be
> fixing a bug, you don't want me to guess, as my "guess" usually is "delete the
> patch".

Sorry Greg; this is not a bug fix and can go into the *next kernel.
I will resend this patch after fixing the changelog comment.

Regards,

K. Y 
> 
> greg k-h
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/