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


Groups > linux.kernel > #1434704 > unrolled thread

Re: [PATCH v14 net-next 1/1] hv_sock: introduce Hyper-V Sockets

Started byOlaf Hering <olaf@aepfle.de>
First post2016-06-30 18:20 +0200
Last post2016-07-01 08:50 +0200
Articles 3 — 2 participants

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.


Contents

  Re: [PATCH v14 net-next 1/1] hv_sock: introduce Hyper-V Sockets Olaf Hering <olaf@aepfle.de> - 2016-06-30 18:20 +0200
    RE: [PATCH v14 net-next 1/1] hv_sock: introduce Hyper-V Sockets Dexuan Cui <decui@microsoft.com> - 2016-06-30 18:30 +0200
    Re: [PATCH v14 net-next 1/1] hv_sock: introduce Hyper-V Sockets Olaf Hering <olaf@aepfle.de> - 2016-07-01 08:50 +0200

#1434704 — Re: [PATCH v14 net-next 1/1] hv_sock: introduce Hyper-V Sockets

FromOlaf Hering <olaf@aepfle.de>
Date2016-06-30 18:20 +0200
SubjectRe: [PATCH v14 net-next 1/1] hv_sock: introduce Hyper-V Sockets
Message-ID<rPMR3-3kx-1@gated-at.bofh.it>

[Multipart message — attachments visible in raw view] — view raw

On Thu, Jun 30, Dexuan Cui wrote:

> -#define AF_MAX		43	/* For now.. */
> +#define AF_MAX		44	/* For now.. */

Should this patch also change the places where AF_MAX is used,
like all the arrays in net/core/sock.c?

Olaf

[toc] | [next] | [standalone]


#1434713

FromDexuan Cui <decui@microsoft.com>
Date2016-06-30 18:30 +0200
Message-ID<rPN0J-3nZ-9@gated-at.bofh.it>
In reply to#1434704
> From: Olaf Hering [mailto:olaf@aepfle.de]
> Sent: Friday, July 1, 2016 0:12
> To: Dexuan Cui <decui@microsoft.com>
> Cc: davem@davemloft.net; gregkh@linuxfoundation.org;
> netdev@vger.kernel.org; linux-kernel@vger.kernel.org;
> devel@linuxdriverproject.org; apw@canonical.com; jasowang@redhat.com;
> Vitaly Kuznetsov <vkuznets@redhat.com>; Cathy Avery <cavery@redhat.com>;
> KY Srinivasan <kys@microsoft.com>; Haiyang Zhang
> <haiyangz@microsoft.com>; joe@perches.com; Rolf Neugebauer
> <rolf.neugebauer@docker.com>
> Subject: Re: [PATCH v14 net-next 1/1] hv_sock: introduce Hyper-V Sockets
> 
> On Thu, Jun 30, Dexuan Cui wrote:
> 
> > -#define AF_MAX		43	/* For now.. */
> > +#define AF_MAX		44	/* For now.. */
> 
> Should this patch also change the places where AF_MAX is used,
> like all the arrays in net/core/sock.c?
> 
> Olaf

Thanks for the reminder, Olaf!

I think we may as well make a separate patch for this. 
It is in my To-Do list.

Thanks,
-- Dexuan

[toc] | [prev] | [next] | [standalone]


#1435060

FromOlaf Hering <olaf@aepfle.de>
Date2016-07-01 08:50 +0200
Message-ID<rQ0r1-3gE-47@gated-at.bofh.it>
In reply to#1434704

[Multipart message — attachments visible in raw view] — view raw

On Thu, Jun 30, Olaf Hering wrote:

> On Thu, Jun 30, Dexuan Cui wrote:
> 
> > -#define AF_MAX		43	/* For now.. */
> > +#define AF_MAX		44	/* For now.. */
> 
> Should this patch also change the places where AF_MAX is used,
> like all the arrays in net/core/sock.c?

Also, there are appearently two competing changes for AF_MAX, one is
yours and the other one is AF_SMC. I'm not subscribed to netdev, so its
not clear which one gets applied first.

Olaf.

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web