Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
From:   Dexuan Cui <decui@microsoft.com>
To:     Stefano Garzarella <sgarzare@redhat.com>,
        "netdev@vger.kernel.org" <netdev@vger.kernel.org>
CC:     "linux-hyperv@vger.kernel.org" <linux-hyperv@vger.kernel.org>,
        KY Srinivasan <kys@microsoft.com>,
        Stefan Hajnoczi <stefanha@redhat.com>,
        Sasha Levin <sashal@kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "kvm@vger.kernel.org" <kvm@vger.kernel.org>,
        "David S. Miller" <davem@davemloft.net>,
        "virtualization@lists.linux-foundation.org" 
        <virtualization@lists.linux-foundation.org>,
        Stephen Hemminger <sthemmin@microsoft.com>,
        Jason Wang <jasowang@redhat.com>,
        "Michael S. Tsirkin" <mst@redhat.com>,
        Haiyang Zhang <haiyangz@microsoft.com>,
        Jorgen Hansen <jhansen@vmware.com>
Subject: RE: [RFC PATCH 07/13] vsock: handle buffer_size sockopts in the core
Thread-Topic: [RFC PATCH 07/13] vsock: handle buffer_size sockopts in the core
Thread-Index: AQHVdSassErYYYgXqES9Q6BNsB/gZ6dJYpqg
Date:   Thu, 3 Oct 2019 20:11:18 +0000
Message-ID: <KU1P153MB0166C0DEDAC1FAC009B0D257BF9F0@KU1P153MB0166.APCP153.PROD.OUTLOOK.COM>
References: <20190927112703.17745-1-sgarzare@redhat.com>
 <20190927112703.17745-8-sgarzare@redhat.com>
In-Reply-To: <20190927112703.17745-8-sgarzare@redhat.com>
Accept-Language: en-US
Content-Language: en-US
msip_labels: MSIP_Label_f42aa342-8706-4288-bd11-ebb85995028c_Enabled=True;
 MSIP_Label_f42aa342-8706-4288-bd11-ebb85995028c_SiteId=72f988bf-86f1-41af-91ab-2d7cd011db47;
 MSIP_Label_f42aa342-8706-4288-bd11-ebb85995028c_Owner=decui@microsoft.com;
 MSIP_Label_f42aa342-8706-4288-bd11-ebb85995028c_SetDate=2019-10-03T20:11:16.7229005Z;
 MSIP_Label_f42aa342-8706-4288-bd11-ebb85995028c_Name=General;
 MSIP_Label_f42aa342-8706-4288-bd11-ebb85995028c_Application=Microsoft Azure
 Information Protection;
 MSIP_Label_f42aa342-8706-4288-bd11-ebb85995028c_ActionId=dd135884-07be-4d20-80e3-90c49b8e3e74;
 MSIP_Label_f42aa342-8706-4288-bd11-ebb85995028c_Extended_MSFT_Method=Automatic
authentication-results: spf=none (sender IP is )
 smtp.mailfrom=decui@microsoft.com; 
x-originating-ip: [2001:4898:80e8:9:4930:a527:7f59:8664]
x-ms-publictraffictype: Email
x-ms-office365-filtering-correlation-id: 5efeb2ca-4a39-433b-3052-08d7483dda9f
x-ms-office365-filtering-ht: Tenant
x-ms-traffictypediagnostic: KU1P153MB0166:|KU1P153MB0166:
x-ms-exchange-transport-forked: True
x-microsoft-antispam-prvs: <KU1P153MB0166D696EF03F5C6CD65ABCDBF9F0@KU1P153MB0166.APCP153.PROD.OUTLOOK.COM>
x-ms-oob-tlc-oobclassifiers: OLM:4502;
x-forefront-prvs: 01792087B6
x-forefront-antispam-report: SFV:NSPM;SFS:(10019020)(4636009)(346002)(376002)(396003)(366004)(39860400002)(136003)(199004)(189003)(446003)(229853002)(33656002)(11346002)(52536014)(99286004)(476003)(54906003)(110136005)(4326008)(102836004)(2906002)(10290500003)(86362001)(6506007)(10090500001)(14454004)(53546011)(478600001)(8676002)(81166006)(81156014)(186003)(25786009)(8936002)(76176011)(66446008)(6246003)(66556008)(64756008)(22452003)(7696005)(8990500004)(7416002)(66946007)(74316002)(7736002)(305945005)(66476007)(2501003)(76116006)(55016002)(6116002)(5660300002)(46003)(71200400001)(316002)(486006)(6436002)(256004)(14444005)(9686003)(71190400001);DIR:OUT;SFP:1102;SCL:1;SRVR:KU1P153MB0166;H:KU1P153MB0166.APCP153.PROD.OUTLOOK.COM;FPR:;SPF:None;LANG:en;PTR:InfoNoRecords;A:1;MX:1;
received-spf: None (protection.outlook.com: microsoft.com does not designate
 permitted sender hosts)
x-ms-exchange-senderadcheck: 1
x-microsoft-antispam: BCL:0;
x-microsoft-antispam-message-info: rMOHf10btEu4EgO/K6ZPxyZAXXQ0U8BnQJQPxTNGQ1U0VGezDFg736HibahG4MFwAtWx+0ClN6we579FXPy19SEY2Vqa1/SxQmS/sO2p94LR/z/kf5FujdvnSKkiMRvOXuZdNc6BPYz9zwbmGzms1WVJmUMg2FMdGmmwR1scvviEZFokk5RAMKEF+7VOm7wdLJIbXMuyPW5Y6jlP2XynyaqExv/L2gQR9I4/EjJZYIoVJgPX3/TtmvnSRe+AkbyNoyY72qkGKEIHTys2WWUnBtaBYzsh1lshw53zQFSKhqKjdH/LVAhxMbqdkAvx8Yw7kehPTCL3HduByAmaKr2pnlqcMmfnCZGC/YcsDPWhC0klMq00bE+IdWbHC+E5Ec8UrlFBptr8h1Nog89+Wa83LTSf+FrGvrX5uHcmcvstPpA=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-OriginatorOrg: microsoft.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 5efeb2ca-4a39-433b-3052-08d7483dda9f
X-MS-Exchange-CrossTenant-originalarrivaltime: 03 Oct 2019 20:11:19.0097
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 72f988bf-86f1-41af-91ab-2d7cd011db47
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: 7yWk9auZt77FA8ZNRfuLwtx0+J2gTZhvbSBT1JKBPO4j52zIfDLHyqbw8t94Wsy4z7op6nJqrQsKYcxmo0qc+Q==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: KU1P153MB0166
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

> From: Stefano Garzarella <sgarzare@redhat.com>
> Sent: Friday, September 27, 2019 4:27 AM
> To: netdev@vger.kernel.org
>=20
> virtio_transport and vmci_transport handle the buffer_size
> sockopts in a very similar way.
>=20
> In order to support multiple transports, this patch moves this
> handling in the core to allow the user to change the options
> also if the socket is not yet assigned to any transport.
>=20
> This patch also adds the '.notify_buffer_size' callback in the
> 'struct virtio_transport' in order to inform the transport,
> when the buffer_size is changed by the user. It is also useful
> to limit the 'buffer_size' requested (e.g. virtio transports).
>=20
> Signed-off-by: Stefano Garzarella <sgarzare@redhat.com>
> ---
>  drivers/vhost/vsock.c                   |  7 +-
>  include/linux/virtio_vsock.h            | 15 +----
>  include/net/af_vsock.h                  | 14 ++--
>  net/vmw_vsock/af_vsock.c                | 43 ++++++++++---
>  net/vmw_vsock/hyperv_transport.c        | 36 -----------
>  net/vmw_vsock/virtio_transport.c        |  8 +--
>  net/vmw_vsock/virtio_transport_common.c | 78 ++++------------------
>  net/vmw_vsock/vmci_transport.c          | 86 +++----------------------
>  net/vmw_vsock/vmci_transport.h          |  3 -
>  9 files changed, 64 insertions(+), 226 deletions(-)
>=20

The hv_sock part (hyperv_transport.c) looks good to me.

Acked-by: Dexuan Cui <decui@microsoft.com>