Re: [PATCH v2] mm/balloon_compaction: Fix trivial spelling

2020-06-10 Thread Sergei Shtylyov
Hello! On 09.06.2020 17:34, Kieran Bingham wrote: The word 'descriptor' is misspelled throughout the tree. Fix it up accordingly: decriptors -> descriptors decriptor -> descriptor really. ;-) Signed-off-by: Kieran Bingham Reviewed-by: David Hildenbrand --- mm/balloon_compact

Re: [PATCH 2/6] i915/gvt/kvm: a NULL ->mm does not mean a thread is a kthread

2020-04-04 Thread Sergei Shtylyov
Hello! On 04.04.2020 12:40, Christoph Hellwig wrote: Use the proper API instead. Fixes: f440c8a572d7 ("drm/i915/gvt/kvmgt: read/write GPA via KVM API") Signed-off-by: Christoph Hellwig --- drivers/gpu/drm/i915/gvt/kvmgt.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/

Re: [PATCH v2 10/11] vsock_test: skip read() in test_stream*close tests on a VMCI host

2019-08-01 Thread Sergei Shtylyov
Hello! On 08/01/2019 06:25 PM, Stefano Garzarella wrote: > When VMCI transport is used, if the guest closes a connection, > all data is gone and EOF is returned, so we should skip the read > of data written by the peer before closing the connection. > > Reported-by: Jorgen Hansen > Signed-off-b

Re: [PATCH] vhost: correct the related warning message

2018-12-16 Thread Sergei Shtylyov
On 12/13/2018 06:21 PM, Michael S. Tsirkin wrote: >>> Fixes: 'commit d588cf8f618d ("target: Fix se_tpg_tfo->tf_subsys regression >>> + remove tf_subsystem")' >>> 'commit cbbd26b8b1a6 ("[iov_iter] new primitives - >>> copy_from_iter_full() and friends")' >> >> Fixes: d588cf8f618d ("target

Re: [PATCH] vhost: correct the related warning message

2018-12-15 Thread Sergei Shtylyov
On 13.12.2018 4:10, wangyan wrote: Fixes: 'commit d588cf8f618d ("target: Fix se_tpg_tfo->tf_subsys regression + remove tf_subsystem")' 'commit cbbd26b8b1a6 ("[iov_iter] new primitives - copy_from_iter_full() and friends")' Fixes: d588cf8f618d ("target: Fix se_tpg_tfo->tf_subsys regre

Re: [PATCH net] vhost-vsock: fix use after free

2018-09-27 Thread Sergei Shtylyov
Hello! On 9/27/2018 11:43 AM, Jason Wang wrote: Just a couple of typos... The access of vsock is not protected by vhost_vsock_lock. This may lead use after free since vhost_vsock_dev_release() may free the Lead to use. pointer at the same time. Fix this by holding the lock during the

Re: [RFC v4 5/5] virtio_ring: enable packed ring

2018-05-16 Thread Sergei Shtylyov
On 05/16/2018 01:21 PM, Tiwei Bie wrote: >>> Signed-off-by: Tiwei Bie >>> --- >>> drivers/virtio/virtio_ring.c | 2 ++ >>> 1 file changed, 2 insertions(+) >>> >>> diff --git a/drivers/virtio/virtio_ring.c b/drivers/virtio/virtio_ring.c >>> index de3839f3621a..b158692263b0 100644 >>> --- a/driv

Re: [RFC v4 5/5] virtio_ring: enable packed ring

2018-05-16 Thread Sergei Shtylyov
On 5/16/2018 11:37 AM, Tiwei Bie wrote: Signed-off-by: Tiwei Bie --- drivers/virtio/virtio_ring.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/virtio/virtio_ring.c b/drivers/virtio/virtio_ring.c index de3839f3621a..b158692263b0 100644 --- a/drivers/virtio/virtio_ring.c +++ b

Re: [PATCH] virtio_net: lower limit on buffer size

2017-06-03 Thread Sergei Shtylyov
On 06/02/2017 11:25 PM, J. Bruce Fields wrote: commit d85b758f72b0 "virtio_net: fix support for small rings" Commit d85b758f72b0 ("virtio_net: fix support for small rings") was supposed to increase the buffer size for small rings but had an unintentional side effect of decreasing it for l

Re: [PATCH] virtio_net: lower limit on buffer size

2017-06-02 Thread Sergei Shtylyov
Hello! On 6/2/2017 2:56 AM, Michael S. Tsirkin wrote: commit d85b758f72b0 "virtio_net: fix support for small rings" Commit d85b758f72b0 ("virtio_net: fix support for small rings") was supposed to increase the buffer size for small rings but had an unintentional side effect of decreasing

Re: [PATCH net-next 1/8] ptr_ring: introduce batch dequeuing

2017-03-21 Thread Sergei Shtylyov
Hello! On 3/21/2017 7:04 AM, Jason Wang wrote: Signed-off-by: Jason Wang --- include/linux/ptr_ring.h | 65 1 file changed, 65 insertions(+) diff --git a/include/linux/ptr_ring.h b/include/linux/ptr_ring.h index 6c70444..4771ded 100644 --- a/i

Re: [RFC v2 -next 2/2] virtio_net: Read the advised MTU

2016-03-19 Thread Sergei Shtylyov
Hello. On 3/16/2016 12:04 AM, Aaron Conole wrote: This patch checks the feature bit for the VIRTIO_NET_F_MTU feature. If it exists, read the advised MTU and use it. No proper error handling is provided for the case where a user changes the negotiated MTU. A future commit will add proper error

Re: [RFC -next 2/2] virtio_net: Read and use the advised MTU

2016-03-10 Thread Sergei Shtylyov
Hello. On 03/10/2016 05:28 PM, Aaron Conole wrote: This patch checks the feature bit for the VIRTIO_NET_F_MTU feature. If it exists, read the advised MTU and use it. No proper error handling is provided for the case where a user changes the negotiated MTU. A future commit will add proper error

Re: [PATCH 01/16] drm: fixes crct set_mode when crtc mode_fixup is null.

2016-02-16 Thread Sergei Shtylyov
Hello. On 02/16/2016 05:10 PM, Carlos Palminha wrote: This patch set nukes all the dummy crtc mode_fixup implementations. (made on top of Daniel topic/drm-misc branch) Signed-off-by: Carlos Palminha --- drivers/gpu/drm/drm_crtc_helper.c | 9 ++--- 1 file changed, 6 insertions(+), 3 del

Re: [PATCH v2 32/32] virtio_ring: use virt_store_mb

2016-01-01 Thread Sergei Shtylyov
Hello. On 12/31/2015 10:09 PM, Michael S. Tsirkin wrote: We need a full barrier after writing out event index, using virt_store_mb there seems better than open-coding. As usual, we need a wrapper to account for strong barriers. It's tempting to use this in vhost as well, for that, we'll need

Re: [PATCH 2/4] vhost: move features to core

2015-09-10 Thread Sergei Shtylyov
Hello. On 09/10/2015 10:23 AM, Michael S. Tsirkin wrote: virtio 1 and any layout are core features, move them there. This fixes vhost test. Signed-off-by: Michael S. Tsirkin [...] diff --git a/drivers/vhost/test.c b/drivers/vhost/test.c index d9c501e..f2882ac 100644 --- a/drivers/vhost/tes

Re: [PATCH net] virtio-net: drop NETIF_F_FRAGLIST

2015-08-04 Thread Sergei Shtylyov
Hello. On 8/4/2015 12:55 PM, Jason Wang wrote: virtio declares support for NETIF_F_FRAGLIST, but assumes that there are at most MAX_SKB_FRAGS + 2 fragments which isn't always true with a fraglist. A longer fraglist in the skb will make the call to skb_to_sgvec overflow the sg array, leading

Re: [PATCH] vhost/net: length miscalculation

2015-01-07 Thread Sergei Shtylyov
Hello. On 01/07/2015 11:55 AM, Michael S. Tsirkin wrote: commit 8b38694a2dc8b18374310df50174f1e4376d6824 vhost/net: virtio 1.0 byte swap had this chunk: - heads[headcount - 1].len += datalen; + heads[headcount - 1].len = cpu_to_vhost32(vq, len - datalen); This adds datalen w

Re: [PATCH 03/10] ovs: Enable handling of UFO6 packets.

2014-12-17 Thread Sergei Shtylyov
Hello. On 12/17/2014 09:20 PM, Vladislav Yasevich wrote: Since UFO6 packets can now be identified by SKB_GSO_UDP6, add proper checks to handel UFO6 flows. Legacy applications may still have UFO6 packets identified by SKB_GSO_UDP, so we need to continue to handle them correclty. Signed-off-by

Re: [PATCH v7 28/46] vhost: make features 64 bit

2014-11-30 Thread Sergei Shtylyov
Hello. On 11/30/2014 6:11 PM, Michael S. Tsirkin wrote: We need to use bit 32 for virtio 1.0 Signed-off-by: Michael S. Tsirkin Reviewed-by: Jason Wang --- drivers/vhost/vhost.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/vhost/vhost.h b/drivers/vhos

Re: [PATCH 09/16] virtio-net: drop config_mutex

2014-10-06 Thread Sergei Shtylyov
On 10/6/2014 4:22 PM, Michael S. Tsirkin wrote: config_mutex served two purposes: prevent multiple concurrent config change handlers, and synchronize access to config_enable flag. Since commit dbf2576e37da0fcc7aacbfbb9fd5d3de7888a3c1 workqueue: make all workqueues non-reentrant all workq

Re: [PATCH 09/16] virtio-net: drop config_mutex

2014-10-06 Thread Sergei Shtylyov
On 10/6/2014 3:56 PM, Michael S. Tsirkin wrote: config_mutex served two purposes: prevent multiple concurrent config change handlers, and synchronize access to config_enable flag. Since commit dbf2576e37da0fcc7aacbfbb9fd5d3de7888a3c1 workqueue: make all workqueues non-reentrant all workq

Re: [PATCH 09/16] virtio-net: drop config_mutex

2014-10-06 Thread Sergei Shtylyov
Hello. On 10/5/2014 8:07 PM, Michael S. Tsirkin wrote: config_mutex served two purposes: prevent multiple concurrent config change handlers, and synchronize access to config_enable flag. Since commit dbf2576e37da0fcc7aacbfbb9fd5d3de7888a3c1 workqueue: make all workqueues non-reentrant a

Re: [PATCH] VMXNET3: Check for map error in vmxnet3_set_mc

2014-09-02 Thread Sergei Shtylyov
Hello. On 09/02/2014 08:30 PM, Andy King wrote: We should check if the map of the table actually succeeds, and also free resources accordingly. This fixes the kernel panic reported by Tetsuo Handa. There's "Reported-by:" line for that. Version bumped to 1.2.1.0 Acked-by: Shelley Gong

Re: [PATCH] virtio-net: free bufs correctly on invalid packet length

2013-12-05 Thread Sergei Shtylyov
Hello. On 12/05/2013 11:16 PM, Michael Dalton wrote: When a packet with invalid length arrives, ensure that the packet is freed correctly if mergeable packet buffers and big packets (GUEST_TSO4) are both enabled. Looks like a similar patch have been posted by Andrey Vagin today. You also

Re: [PATCH] vhost/scsi: use vmalloc for order-10 allocation

2013-09-17 Thread Sergei Shtylyov
Hello. On 09/17/2013 11:21 AM, Michael S. Tsirkin wrote: As vhost scsi device struct is large, if the device is created on a busy system, kzalloc() might fail, so this patch does a fallback to vzalloc(). As vmalloc() adds overhead on data-path, add __GFP_REPEAT to kzalloc() flags to do this

Re: [PATCH V2 4/6] vhost_net: determine whether or not to use zerocopy at one time

2013-08-30 Thread Sergei Shtylyov
Hello. On 08/30/2013 08:29 AM, Jason Wang wrote: Currently, even if the packet length is smaller than VHOST_GOODCOPY_LEN, if upend_idx != done_idx we still set zcopy_used to true and rollback this choice later. This could be avoided by determine zerocopy once by checking all conditions at one t

Re: [PATCH 2/2] virtio_net: fix race in RX VQ processing

2013-07-08 Thread Sergei Shtylyov
Hello. On 08-07-2013 13:04, Michael S. Tsirkin wrote: virtio net called virtqueue_enable_cq on RX path after napi_complete, so with NAPI_STATE_SCHED clear - outside the implicit napi lock. This violates the requirement to synchronize virtqueue_enable_cq wrt virtqueue_add_buf. In particular, us

Re: [PATCH net] vhost-net: fix use-after-free in vhost_net_flush

2013-06-20 Thread Sergei Shtylyov
Hello. On 20-06-2013 15:48, Michael S. Tsirkin wrote: vhost_net_ubuf_put_and_wait has a confusing name: it will actually also free it's argument. Thus since commit 1280c27f8e29acf4af2da914e80ec27c3dbd5c01 Please also specify that commit's summary line in parens. vhost_net_flush tries to

Re: [PATCH v8, part3 12/14] mm: correctly update zone->mamaged_pages

2013-05-30 Thread Sergei Shtylyov
On 26-05-2013 17:38, Jiang Liu wrote: Typo in the subject: s/mamaged_pages/managed_pages/. Enhance adjust_managed_page_count() to adjust totalhigh_pages for highmem pages. And change code which directly adjusts totalram_pages to use adjust_managed_page_count() because it adjusts totalram_pag

Re: [PATCH 00/16] treewide: Convert dev_printk(KERN_ to dev_(

2012-10-30 Thread Sergei Shtylyov
Hello. On 28-10-2012 12:05, Joe Perches wrote: dev_ create smaller objects than dev_printk(KERN_. Convert non-debug calls to this form. Joe Perches (16): tile: Convert dev_printk(KERN_ to dev_( [...] tile: Convert dev_printk(KERN_ to dev_( [...] tile: Convert dev_printk(KERN_ to