From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stefan Raspl Subject: Re: [PATCH v2 00/17] tools/kvm_stat: Misc Patches Date: Fri, 17 Mar 2017 08:08:03 +0100 Message-ID: References: <20170310124016.96319-1-raspl@linux.vnet.ibm.com> <20170315213635.GG14081@potion> Reply-To: raspl@linux.vnet.ibm.com Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Cc: kvm@vger.kernel.org, pbonzini@redhat.com, frankja@linux.vnet.ibm.com To: =?UTF-8?B?UmFkaW0gS3LEjW3DocWZ?= Return-path: Received: from mx0b-001b2d01.pphosted.com ([148.163.158.5]:32941 "EHLO mx0a-001b2d01.pphosted.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751035AbdCQHIO (ORCPT ); Fri, 17 Mar 2017 03:08:14 -0400 Received: from pps.filterd (m0098414.ppops.net [127.0.0.1]) by mx0b-001b2d01.pphosted.com (8.16.0.20/8.16.0.20) with SMTP id v2H73nKg059622 for ; Fri, 17 Mar 2017 03:08:12 -0400 Received: from e06smtp10.uk.ibm.com (e06smtp10.uk.ibm.com [195.75.94.106]) by mx0b-001b2d01.pphosted.com with ESMTP id 297w4aggt8-1 (version=TLSv1.2 cipher=AES256-SHA bits=256 verify=NOT) for ; Fri, 17 Mar 2017 03:08:12 -0400 Received: from localhost by e06smtp10.uk.ibm.com with IBM ESMTP SMTP Gateway: Authorized Use Only! Violators will be prosecuted for from ; Fri, 17 Mar 2017 07:08:11 -0000 In-Reply-To: <20170315213635.GG14081@potion> Sender: kvm-owner@vger.kernel.org List-ID: On 15.03.2017 22:36, Radim Krčmář wrote: > 2017-03-10 13:39+0100, Stefan Raspl: >> Assortment of various fixes and improvements for the kvm_stat utility. >> Patches 1-5 are bugfixes. >> Patch 6 provides full PEP8 compliance by inserting the required empty >> lines. It is included as an extra for easy disposal, since that kind of >> patch might be a bit controversial. >> There are some subtle aspects regarding patch 7, as its effects might >> differ per platform. However, I didn't notice any negative effects, only >> improvements. >> Patch 8 is a simple doc update. >> Patches 9-13 provide usability improvements. >> Patches 14-17 add new features. >> >> Changes in v2: >> - 04/17: Removed chunk that eliminated the only once used libc variable, >> and changed '3.' -> '3.0' >> - 07/17: Changed '3.' -> '3.0' >> - 16/17: Fixed accidental removal of description for interactive command 'x' > > Very nice! > > Applied, thanks. > > (I will point out few other minor things, but they are not worth > attention unless you have absolutely nothing to do. :]) I took a note, and I'll see what I can do - thx! Ciao, Stefan