From mboxrd@z Thu Jan 1 00:00:00 1970 From: Bartlomiej Zolnierkiewicz Subject: Re: [PATCH] video: fbdev: via: remove VLA usage Date: Mon, 12 Mar 2018 17:04:25 +0100 Message-id: <7211546.mLmn3tHjVa@amdc3058> In-reply-to: MIME-version: 1.0 Content-transfer-encoding: 7Bit Content-type: text/plain; charset="us-ascii" References: <20180308173949.GA19305@embeddedgus> To: "Gustavo A. R. Silva" Cc: Emil Velikov , Eric Engestrom , Florian Tobias Schandinat , linux-fbdev , Kees Cook , kernel-hardening@lists.openwall.com, "Linux-Kernel@Vger. Kernel. Org" , ML dri-devel , "Gustavo A. R. Silva" List-ID: On Friday, March 09, 2018 11:02:24 AM Emil Velikov wrote: > On 9 March 2018 at 10:59, Eric Engestrom wrote: > > On Thursday, 2018-03-08 11:39:49 -0600, Gustavo A. R. Silva wrote: > >> In preparation to enabling -Wvla, remove VLA usage. > >> > >> Also, fixed as part of the directive to remove all VLAs from > >> the kernel: https://lkml.org/lkml/2018/3/7/621 > >> > >> Signed-off-by: Gustavo A. R. Silva > > > > Sorry for my reply on v1, I missed that Emil already replied to another > > post of the same patch (side note on that, making use of `-vX` and > > `--in-reply-to` helps track the evolution of patches) > > > > This looks good to me: > > Reviewed-by: Eric Engestrom > > > Same here > Reviewed-by: Emil Velikov Patch queued for 4.17, thanks. Best regards, -- Bartlomiej Zolnierkiewicz Samsung R&D Institute Poland Samsung Electronics From mboxrd@z Thu Jan 1 00:00:00 1970 From: Bartlomiej Zolnierkiewicz Date: Mon, 12 Mar 2018 16:04:25 +0000 Subject: Re: [PATCH] video: fbdev: via: remove VLA usage Message-Id: <7211546.mLmn3tHjVa@amdc3058> List-Id: References: <20180308173949.GA19305@embeddedgus> In-Reply-To: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: "Gustavo A. R. Silva" Cc: linux-fbdev , Kees Cook , kernel-hardening@lists.openwall.com, Florian Tobias Schandinat , Emil Velikov , "Linux-Kernel@Vger. Kernel. Org" , ML dri-devel , "Gustavo A. R. Silva" On Friday, March 09, 2018 11:02:24 AM Emil Velikov wrote: > On 9 March 2018 at 10:59, Eric Engestrom wrote: > > On Thursday, 2018-03-08 11:39:49 -0600, Gustavo A. R. Silva wrote: > >> In preparation to enabling -Wvla, remove VLA usage. > >> > >> Also, fixed as part of the directive to remove all VLAs from > >> the kernel: https://lkml.org/lkml/2018/3/7/621 > >> > >> Signed-off-by: Gustavo A. R. Silva > > > > Sorry for my reply on v1, I missed that Emil already replied to another > > post of the same patch (side note on that, making use of `-vX` and > > `--in-reply-to` helps track the evolution of patches) > > > > This looks good to me: > > Reviewed-by: Eric Engestrom > > > Same here > Reviewed-by: Emil Velikov Patch queued for 4.17, thanks. Best regards, -- Bartlomiej Zolnierkiewicz Samsung R&D Institute Poland Samsung Electronics From mboxrd@z Thu Jan 1 00:00:00 1970 From: Bartlomiej Zolnierkiewicz Subject: Re: [PATCH] video: fbdev: via: remove VLA usage Date: Mon, 12 Mar 2018 17:04:25 +0100 Message-ID: <7211546.mLmn3tHjVa@amdc3058> References: <20180308173949.GA19305@embeddedgus> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Received: from mailout1.samsung.com (mailout1.samsung.com [203.254.224.24]) by gabe.freedesktop.org (Postfix) with ESMTPS id 1BC0C6E063 for ; Mon, 12 Mar 2018 16:04:31 +0000 (UTC) In-reply-to: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: "Gustavo A. R. Silva" Cc: linux-fbdev , Kees Cook , kernel-hardening@lists.openwall.com, Florian Tobias Schandinat , Emil Velikov , "Linux-Kernel@Vger. Kernel. Org" , ML dri-devel , "Gustavo A. R. Silva" List-Id: dri-devel@lists.freedesktop.org T24gRnJpZGF5LCBNYXJjaCAwOSwgMjAxOCAxMTowMjoyNCBBTSBFbWlsIFZlbGlrb3Ygd3JvdGU6 Cj4gT24gOSBNYXJjaCAyMDE4IGF0IDEwOjU5LCBFcmljIEVuZ2VzdHJvbSA8ZXJpYy5lbmdlc3Ry b21AaW1ndGVjLmNvbT4gd3JvdGU6Cj4gPiBPbiBUaHVyc2RheSwgMjAxOC0wMy0wOCAxMTozOTo0 OSAtMDYwMCwgR3VzdGF2byBBLiBSLiBTaWx2YSB3cm90ZToKPiA+PiBJbiBwcmVwYXJhdGlvbiB0 byBlbmFibGluZyAtV3ZsYSwgcmVtb3ZlIFZMQSB1c2FnZS4KPiA+Pgo+ID4+IEFsc28sIGZpeGVk IGFzIHBhcnQgb2YgdGhlIGRpcmVjdGl2ZSB0byByZW1vdmUgYWxsIFZMQXMgZnJvbQo+ID4+IHRo ZSBrZXJuZWw6IGh0dHBzOi8vbGttbC5vcmcvbGttbC8yMDE4LzMvNy82MjEKPiA+Pgo+ID4+IFNp Z25lZC1vZmYtYnk6IEd1c3Rhdm8gQS4gUi4gU2lsdmEgPGd1c3Rhdm9AZW1iZWRkZWRvci5jb20+ Cj4gPgo+ID4gU29ycnkgZm9yIG15IHJlcGx5IG9uIHYxLCBJIG1pc3NlZCB0aGF0IEVtaWwgYWxy ZWFkeSByZXBsaWVkIHRvIGFub3RoZXIKPiA+IHBvc3Qgb2YgdGhlIHNhbWUgcGF0Y2ggKHNpZGUg bm90ZSBvbiB0aGF0LCBtYWtpbmcgdXNlIG9mIGAtdlhgIGFuZAo+ID4gYC0taW4tcmVwbHktdG9g IGhlbHBzIHRyYWNrIHRoZSBldm9sdXRpb24gb2YgcGF0Y2hlcykKPiA+Cj4gPiBUaGlzIGxvb2tz IGdvb2QgdG8gbWU6Cj4gPiBSZXZpZXdlZC1ieTogRXJpYyBFbmdlc3Ryb20gPGVyaWMuZW5nZXN0 cm9tQGltZ3RlYy5jb20+Cj4gPgo+IFNhbWUgaGVyZQo+IFJldmlld2VkLWJ5OiBFbWlsIFZlbGlr b3YgPGVtaWwudmVsaWtvdkBjb2xsYWJvcmEuY29tPgoKUGF0Y2ggcXVldWVkIGZvciA0LjE3LCB0 aGFua3MuCgpCZXN0IHJlZ2FyZHMsCi0tCkJhcnRsb21pZWogWm9sbmllcmtpZXdpY3oKU2Ftc3Vu ZyBSJkQgSW5zdGl0dXRlIFBvbGFuZApTYW1zdW5nIEVsZWN0cm9uaWNzCgpfX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fXwpkcmktZGV2ZWwgbWFpbGluZyBsaXN0 CmRyaS1kZXZlbEBsaXN0cy5mcmVlZGVza3RvcC5vcmcKaHR0cHM6Ly9saXN0cy5mcmVlZGVza3Rv cC5vcmcvbWFpbG1hbi9saXN0aW5mby9kcmktZGV2ZWwK