[PATCH] drm/virtio: move drm_connector_update_edid_property() call

2019-04-04 Thread Gerd Hoffmann
drm_connector_update_edid_property can sleep, we must not call it while holding a spinlock. Move the callsize. Reported-by: Max Filippov Signed-off-by: Gerd Hoffmann --- drivers/gpu/drm/virtio/virtgpu_vq.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

Re: [PATCH RFC 1/4] vsock/virtio: reduce credit update messages

2019-04-04 Thread Stefan Hajnoczi
On Thu, Apr 04, 2019 at 12:58:35PM +0200, Stefano Garzarella wrote: > @@ -256,6 +257,7 @@ virtio_transport_stream_do_dequeue(struct vsock_sock *vsk, > struct virtio_vsock_sock *vvs = vsk->trans; > struct virtio_vsock_pkt *pkt; > size_t bytes, total = 0; > + s64 free_space;

Re: [PATCH RFC 0/4] vsock/virtio: optimizations to increase the throughput

2019-04-04 Thread Michael S. Tsirkin
On Thu, Apr 04, 2019 at 06:47:15PM +0200, Stefano Garzarella wrote: > On Thu, Apr 04, 2019 at 11:52:46AM -0400, Michael S. Tsirkin wrote: > > I simply love it that you have analysed the individual impact of > > each patch! Great job! > > Thanks! I followed Stefan's suggestions! > > > > > For

Re: [PATCH v2 6/6] drm/cirrus: rewrite and modernize driver.

2019-04-04 Thread Sam Ravnborg
Hi Gerd. Very nice diffstat - good work indeed! > I think it'll still be alot easier to review than a > longish baby-step patch series. Agreed. Some random nits below. With the relevant parts addressed you can add my: Reviewed-by: Sam Ravnborg > new file mode 100644 > index

Re: [PATCH RFC 0/4] vsock/virtio: optimizations to increase the throughput

2019-04-04 Thread Michael S. Tsirkin
On Thu, Apr 04, 2019 at 12:58:34PM +0200, Stefano Garzarella wrote: > This series tries to increase the throughput of virtio-vsock with slight > changes: > - patch 1/4: reduces the number of credit update messages sent to the > transmitter > - patch 2/4: allows the host to split

Re: [Qemu-devel] [PATCH v4 5/5] xfs: disable map_sync for async flush

2019-04-04 Thread Pankaj Gupta
> > > > > > So I would put the helper in include/linux/dax.h and have it like: > > > > > > bool daxdev_mapping_supported(struct vm_area_struct *vma, > > Should this be static inline if you're putting it in the header file? yes. Thanks. > > A comment ought to be added to describe what this

[PATCH v2 6/6] drm/cirrus: rewrite and modernize driver.

2019-04-04 Thread Gerd Hoffmann
Time to kill some bad sample code people are copying from ;) This is a complete rewrite of the cirrus driver. The cirrus_mode_set() function is pretty much the only function which is carried over largely unmodified. Everything else is upside down. It is a single monster patch. But given that

Re: [PATCH RFC 0/4] vsock/virtio: optimizations to increase the throughput

2019-04-04 Thread Stefan Hajnoczi
On Thu, Apr 04, 2019 at 12:58:34PM +0200, Stefano Garzarella wrote: > This series tries to increase the throughput of virtio-vsock with slight > changes: > - patch 1/4: reduces the number of credit update messages sent to the > transmitter > - patch 2/4: allows the host to split

Re: [PATCH] drm/cirrus: rewrite and modernize driver.

2019-04-04 Thread Gerd Hoffmann
Hi, > So I guess I have to add a dest_clip bool parameter when moving them. > /me looks for a good place. drm_fb_helpers.c I think. https://git.kraxel.org/cgit/linux/log/?h=drm-rewrite-cirrus updated. v2 series will follow later today or tomorrow. cheers, Gerd

Re: [PATCH] drm/cirrus: rewrite and modernize driver.

2019-04-04 Thread Noralf Trønnes
Den 04.04.2019 12.27, skrev Gerd Hoffmann: > Hi, > >>> tinydrm_xrgb_to_* >>> >>> imo these could be put into some drm_format_helpers.c to be shared. >> >> I agree, my long term goal is to get rid of tinydrm.ko. Just haven't got >> there yet. >> >> Gerd, if you end up using some of those

Re: [PATCH v4 5/5] xfs: disable map_sync for async flush

2019-04-04 Thread Pankaj Gupta
> > On Thu, Apr 04, 2019 at 02:12:30AM -0400, Pankaj Gupta wrote: > > > All this ad hoc IS_DAX conditional logic is getting pretty nasty. > > > > > > xfs_file_mmap( > > > > > > { > > > struct inode*inode = file_inode(filp); > > > > > > if (vma->vm_flags & VM_SYNC) { > > >

Re: [PATCH] drm/cirrus: rewrite and modernize driver.

2019-04-04 Thread Gerd Hoffmann
Hi, > > tinydrm_xrgb_to_* > > > > imo these could be put into some drm_format_helpers.c to be shared. > > I agree, my long term goal is to get rid of tinydrm.ko. Just haven't got > there yet. > > Gerd, if you end up using some of those functions, feel free to move > just those you need

Re: [Qemu-devel] [PATCH v4 5/5] xfs: disable map_sync for async flush

2019-04-04 Thread Pankaj Gupta
> On Thu 04-04-19 05:09:10, Pankaj Gupta wrote: > > > > > > On Thu, Apr 04, 2019 at 09:09:12AM +1100, Dave Chinner wrote: > > > > > On Wed, Apr 03, 2019 at 04:10:18PM +0530, Pankaj Gupta wrote: > > > > > > Virtio pmem provides asynchronous host page cache flush > > > > > > mechanism. we don't

Re: [PATCH] drm/cirrus: rewrite and modernize driver.

2019-04-04 Thread Noralf Trønnes
Den 04.04.2019 10.52, skrev Daniel Vetter: > On Thu, Apr 4, 2019 at 10:30 AM Gerd Hoffmann wrote: >> >> Hi, >> Speaking of wayland: Seems at least gnome-shell insists on using XR24. >>> >>> Yeah XR24 is pretty much mandatory. Noralf added a few helpers to >>> convert XR24 to other

Re: [PATCH v4 5/5] xfs: disable map_sync for async flush

2019-04-04 Thread Jan Kara
On Thu 04-04-19 05:09:10, Pankaj Gupta wrote: > > > > On Thu, Apr 04, 2019 at 09:09:12AM +1100, Dave Chinner wrote: > > > > On Wed, Apr 03, 2019 at 04:10:18PM +0530, Pankaj Gupta wrote: > > > > > Virtio pmem provides asynchronous host page cache flush > > > > > mechanism. we don't support

Re: [PATCH v4 5/5] xfs: disable map_sync for async flush

2019-04-04 Thread Pankaj Gupta
> > On Thu, Apr 04, 2019 at 09:09:12AM +1100, Dave Chinner wrote: > > > On Wed, Apr 03, 2019 at 04:10:18PM +0530, Pankaj Gupta wrote: > > > > Virtio pmem provides asynchronous host page cache flush > > > > mechanism. we don't support 'MAP_SYNC' with virtio pmem > > > > and xfs. > > > > > > > >

Re: [PATCH] drm/cirrus: rewrite and modernize driver.

2019-04-04 Thread Daniel Vetter
On Thu, Apr 4, 2019 at 10:30 AM Gerd Hoffmann wrote: > > Hi, > > > > Speaking of wayland: Seems at least gnome-shell insists on using XR24. > > > > Yeah XR24 is pretty much mandatory. Noralf added a few helpers to > > convert XR24 to other formats, for display not supporting anything > > else.

Re: [PATCH] drm/cirrus: rewrite and modernize driver.

2019-04-04 Thread Gerd Hoffmann
Hi, > > Speaking of wayland: Seems at least gnome-shell insists on using XR24. > > Yeah XR24 is pretty much mandatory. Noralf added a few helpers to > convert XR24 to other formats, for display not supporting anything > else. Because userspace. Have a pointer to these helpers? grepping

Re: [Qemu-devel] [PATCH v4 2/5] virtio-pmem: Add virtio pmem driver

2019-04-04 Thread Pankaj Gupta
> > > > > Subject: Re: [Qemu-devel] [PATCH v4 2/5] virtio-pmem: Add virtio pmem > > > driver > > > > > > On Wed, Apr 03, 2019 at 04:10:15PM +0530, Pankaj Gupta wrote: > > > > This patch adds virtio-pmem driver for KVM guest. > > > > > > > > Guest reads the persistent memory range information

Re: [PATCH] drm/cirrus: rewrite and modernize driver.

2019-04-04 Thread Gerd Hoffmann
Hi, > > Speaking of wayland: Seems at least gnome-shell insists on using XR24. > > Yeah XR24 is pretty much mandatory. Noralf added a few helpers to > convert XR24 to other formats, for display not supporting anything > else. Because userspace. Ah, right, that is an option too. Given we

Re: [PATCH] drm/cirrus: rewrite and modernize driver.

2019-04-04 Thread Daniel Vetter
On Thu, Apr 4, 2019 at 7:51 AM Gerd Hoffmann wrote: > > On Thu, Apr 04, 2019 at 12:58:09PM +1000, David Airlie wrote: > > On Wed, Apr 3, 2019 at 5:23 PM Gerd Hoffmann wrote: > > > > > > Time to kill some bad sample code people are copying from ;) > > > > > > This is a complete rewrite of the

Re: [PATCH v4 5/5] xfs: disable map_sync for async flush

2019-04-04 Thread Pankaj Gupta
> > On Thu, Apr 04, 2019 at 09:09:12AM +1100, Dave Chinner wrote: > > On Wed, Apr 03, 2019 at 04:10:18PM +0530, Pankaj Gupta wrote: > > > Virtio pmem provides asynchronous host page cache flush > > > mechanism. we don't support 'MAP_SYNC' with virtio pmem > > > and xfs. > > > > > >

Re: [PATCH v4 5/5] xfs: disable map_sync for async flush

2019-04-04 Thread Pankaj Gupta
Hi Dave, > > Virtio pmem provides asynchronous host page cache flush > > mechanism. we don't support 'MAP_SYNC' with virtio pmem > > and xfs. > > > > Signed-off-by: Pankaj Gupta > > --- > > fs/xfs/xfs_file.c | 8 > > 1 file changed, 8 insertions(+) > > > > diff --git