Bug 1262561

Summary: moving virt-manager window causes frequent writes with dconf
Product: [Community] Virtualization Tools Reporter: Miroslav <mspehar>
Component: virt-managerAssignee: Cole Robinson <crobinso>
Status: CLOSED UPSTREAM QA Contact:
Severity: medium Docs Contact:
Priority: unspecified    
Version: unspecifiedCC: berrange, crobinso, gscrivan, mspehar, rbalakri
Target Milestone: ---   
Target Release: ---   
Hardware: x86_64   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2015-09-13 18:12:00 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Attachments:
Description Flags
video of the problem none

Description Miroslav 2015-09-12 20:38:58 UTC
Created attachment 1072757 [details]
video of the problem

Description of problem:
When moving virt-manager window in Gnome, dconf-service is having ~0.5MB/s write operations. It seems that with every pixel move, width and height gets updated.

Version-Release number of selected component (if applicable): 1.2.1

How reproducible:
Reproducible on arch linux with gnome.

Steps to Reproduce:
1. run 'dconf watch /org/virt-manager/'
2. with the same user that ran command in point 1., start virt-manager
3. move the virt-manager window left-right

Actual results:
Significant i/o (~0.5MB/s)

Expected results:
No frequent writes.


Additional info:
VM do not have to be started.

Comment 1 Cole Robinson 2015-09-13 18:12:00 UTC
Thanks for the clear bug report. Indeed we weren't being very smart here, fixed upstream now:

commit bcc13dca3188446e79f20eefce74346846e5ecac
Author: Cole Robinson <crobinso>
Date:   Sun Sep 13 14:07:43 2015 -0400

    virt-manager: Only save window dimensions on app exit (bz 1262561)
    
    Rather than on every window change event which spams dconf and hits
    the disk repeatedly. The new method is roughly what is suggested by
    https://wiki.gnome.org/HowDoI/SaveWindowState
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1262561