Re: [PATCH v3 2/3] virtio-balloon: use actual number of stats for stats queue buffers

2017-03-28 Thread Arnd Bergmann
On Tue, Mar 28, 2017 at 6:46 PM, Ladi Prosek wrote: > The virtio balloon driver contained a not-so-obvious invariant that > update_balloon_stats has to update exactly VIRTIO_BALLOON_S_NR counters > in order to send valid stats to the host. This commit fixes it by having > update_balloon_stats retu

[PULL] vhost: fixes

2017-03-28 Thread Michael S. Tsirkin
The following changes since commit c02ed2e75ef4c74e41e421acb4ef1494671585e8: Linux 4.11-rc4 (2017-03-26 14:15:16 -0700) are available in the git repository at: git://git.kernel.org/pub/scm/linux/kernel/git/mst/vhost.git tags/for_linus for you to fetch changes up to f0bb2d50dfcc519f06f901aac

Re: [PATCH] virtio_balloon: prevent uninitialized variable use

2017-03-28 Thread Ladi Prosek
On Tue, Mar 28, 2017 at 6:18 PM, Michael S. Tsirkin wrote: > On Mon, Mar 27, 2017 at 12:02:33PM +0200, Ladi Prosek wrote: >> On Fri, Mar 24, 2017 at 9:59 PM, Michael S. Tsirkin wrote: >> > On Fri, Mar 24, 2017 at 09:40:07PM +0100, Arnd Bergmann wrote: >> >> On Fri, Mar 24, 2017 at 9:11 PM, Ladi P

[PATCH v3 3/3] virtio_balloon: prevent uninitialized variable use

2017-03-28 Thread Ladi Prosek
From: Arnd Bergmann The latest gcc-7.0.1 snapshot reports a new warning: virtio/virtio_balloon.c: In function 'update_balloon_stats': virtio/virtio_balloon.c:258:26: error: 'events[2]' is used uninitialized in this function [-Werror=uninitialized] virtio/virtio_balloon.c:260:26: error: 'events[

[PATCH v3 2/3] virtio-balloon: use actual number of stats for stats queue buffers

2017-03-28 Thread Ladi Prosek
The virtio balloon driver contained a not-so-obvious invariant that update_balloon_stats has to update exactly VIRTIO_BALLOON_S_NR counters in order to send valid stats to the host. This commit fixes it by having update_balloon_stats return the actual number of counters, and its callers use it when

[PATCH v3 1/3] virtio_balloon: don't push uninitialized buffers to stats virtqueue

2017-03-28 Thread Ladi Prosek
When init_vqs runs, virtio_balloon.stats is either uninitialized or contains stale values. The host updates its state with garbage data because it has no way of knowing that this is just a marker buffer used for signaling. This patch updates the stats before pushing the initial buffer. Signed-off

[PATCH v3 0/3] virtio_balloon: don't push uninitialized buffers to stats virtqueue

2017-03-28 Thread Ladi Prosek
This series fixes issues with variable initialization in the virtio balloon driver which manifest as the driver sending invalid memory stats to the host. v1->v2: * Call update_balloon_stats instead of filling the buffer with invalid tags. * Add BUG_ON to update_balloon_stats to formalize the in

Re: [PATCH] virtio_balloon: prevent uninitialized variable use

2017-03-28 Thread Michael S. Tsirkin
On Mon, Mar 27, 2017 at 12:02:33PM +0200, Ladi Prosek wrote: > On Fri, Mar 24, 2017 at 9:59 PM, Michael S. Tsirkin wrote: > > On Fri, Mar 24, 2017 at 09:40:07PM +0100, Arnd Bergmann wrote: > >> On Fri, Mar 24, 2017 at 9:11 PM, Ladi Prosek wrote: > >> > On Fri, Mar 24, 2017 at 7:38 PM, David Hilde

Re: [PATCH] virtio-blk: add DISCARD support to virtio-blk driver

2017-03-28 Thread Stefan Hajnoczi
On Tue, Mar 28, 2017 at 3:15 AM, Liu, Changpeng wrote: >> -Original Message- >> From: Stefan Hajnoczi [mailto:stefa...@gmail.com] >> Sent: Tuesday, March 28, 2017 4:20 AM >> To: Liu, Changpeng >> Cc: virtio-...@lists.oasis-open.org; >> virtualization@lists.linux-foundation.org; linux- >>