On Sat, Nov 19, 2016 at 10:16:43PM -0500, David Miller wrote: > From: John Fastabend <john.fastab...@gmail.com> > Date: Sat, 19 Nov 2016 19:14:15 -0800 > > > On 16-11-19 07:08 PM, David Miller wrote: > >> From: Stefan Hajnoczi <stefa...@redhat.com> > >> Date: Thu, 17 Nov 2016 15:49:23 +0000 > >> > >>> @@ -159,6 +199,10 @@ virtio_transport_send_pkt(struct virtio_vsock_pkt > >>> *pkt) > >>> return -ENODEV; > >>> } > >>> > >>> + if (le32_to_cpu(pkt->hdr.dst_cid) == vsock->guest_cid) { > >>> + return virtio_transport_send_pkt_loopback(vsock, pkt); > >>> + } > >>> + > >> > >> A single-statement basic block should get surrounding curly > >> braces. > >> > > > > Should _not_ get curly braces in case it wasn't obvious ;) > > Indeed, thanks for the correction :)
Thanks, I'll remember to run checkpatch.pl and send v2. Stefan
signature.asc
Description: PGP signature