* [PATCH] increase MAX_NICS value
@ 2010-07-09 15:39 Alessandro Bono
2010-07-09 17:30 ` Brian Jackson
2010-07-10 5:59 ` Markus Armbruster
0 siblings, 2 replies; 4+ messages in thread
From: Alessandro Bono @ 2010-07-09 15:39 UTC (permalink / raw)
To: kvm
Hi all
max number of allowed nics per vm is hardcoded on net.h to 8. This value
it's too low in situation with big network, increase to 24
this should be safe as mentioned on this thread
http://kerneltrap.org/mailarchive/linux-kvm/2009/1/29/4848304/thread
Signed-off-by: Alessandro Bono <alessandro.bono@gmail.com>
--- net.h.old 2010-07-09 17:30:39.542170103 +0200
+++ net.h 2010-07-09 17:30:48.842166029 +0200
@@ -121,7 +121,7 @@
/* NIC info */
-#define MAX_NICS 8
+#define MAX_NICS 24
struct NICInfo {
uint8_t macaddr[6];
--
Cordiali Saluti
Alessandro Bono
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] increase MAX_NICS value
2010-07-09 15:39 [PATCH] increase MAX_NICS value Alessandro Bono
@ 2010-07-09 17:30 ` Brian Jackson
2010-07-12 12:54 ` Alessandro Bono
2010-07-10 5:59 ` Markus Armbruster
1 sibling, 1 reply; 4+ messages in thread
From: Brian Jackson @ 2010-07-09 17:30 UTC (permalink / raw)
To: Alessandro Bono; +Cc: kvm
On Friday, July 09, 2010 10:39:17 am Alessandro Bono wrote:
> Hi all
>
> max number of allowed nics per vm is hardcoded on net.h to 8. This value
> it's too low in situation with big network, increase to 24
> this should be safe as mentioned on this thread
> http://kerneltrap.org/mailarchive/linux-kvm/2009/1/29/4848304/thread
A better long term solution might be to convert it to QLIST or something
instead of hardcoding the array size.
>
> Signed-off-by: Alessandro Bono <alessandro.bono@gmail.com>
>
> --- net.h.old 2010-07-09 17:30:39.542170103 +0200
> +++ net.h 2010-07-09 17:30:48.842166029 +0200
> @@ -121,7 +121,7 @@
>
> /* NIC info */
>
> -#define MAX_NICS 8
> +#define MAX_NICS 24
>
> struct NICInfo {
> uint8_t macaddr[6];
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] increase MAX_NICS value
2010-07-09 15:39 [PATCH] increase MAX_NICS value Alessandro Bono
2010-07-09 17:30 ` Brian Jackson
@ 2010-07-10 5:59 ` Markus Armbruster
1 sibling, 0 replies; 4+ messages in thread
From: Markus Armbruster @ 2010-07-10 5:59 UTC (permalink / raw)
To: Alessandro Bono; +Cc: kvm
Alessandro Bono <alessandro.bono@gmail.com> writes:
> Hi all
>
> max number of allowed nics per vm is hardcoded on net.h to 8. This value
> it's too low in situation with big network, increase to 24
> this should be safe as mentioned on this thread
> http://kerneltrap.org/mailarchive/linux-kvm/2009/1/29/4848304/thread
>
> Signed-off-by: Alessandro Bono <alessandro.bono@gmail.com>
1. Doesn't this belong upstream? qemu-devel@nongnu.org
2. MAX_NICS applies only to -net nic. Use -device instead. See
docs/qdev-device-use.txt
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] increase MAX_NICS value
2010-07-09 17:30 ` Brian Jackson
@ 2010-07-12 12:54 ` Alessandro Bono
0 siblings, 0 replies; 4+ messages in thread
From: Alessandro Bono @ 2010-07-12 12:54 UTC (permalink / raw)
To: Brian Jackson; +Cc: kvm
On Fri, 2010-07-09 at 12:30 -0500, Brian Jackson wrote:
> On Friday, July 09, 2010 10:39:17 am Alessandro Bono wrote:
> > Hi all
> >
> > max number of allowed nics per vm is hardcoded on net.h to 8. This value
> > it's too low in situation with big network, increase to 24
> > this should be safe as mentioned on this thread
> > http://kerneltrap.org/mailarchive/linux-kvm/2009/1/29/4848304/thread
>
>
> A better long term solution might be to convert it to QLIST or something
> instead of hardcoding the array size.
Sure my patch it's simply a better default value and solve the problem
for big network. Obviously a dynamic structure it's best suited for this
information
>
>
> >
> > Signed-off-by: Alessandro Bono <alessandro.bono@gmail.com>
> >
> > --- net.h.old 2010-07-09 17:30:39.542170103 +0200
> > +++ net.h 2010-07-09 17:30:48.842166029 +0200
> > @@ -121,7 +121,7 @@
> >
> > /* NIC info */
> >
> > -#define MAX_NICS 8
> > +#define MAX_NICS 24
> >
> > struct NICInfo {
> > uint8_t macaddr[6];
--
Cordiali Saluti
Alessandro Bono
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2010-07-12 12:54 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-07-09 15:39 [PATCH] increase MAX_NICS value Alessandro Bono
2010-07-09 17:30 ` Brian Jackson
2010-07-12 12:54 ` Alessandro Bono
2010-07-10 5:59 ` Markus Armbruster
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.