From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:38897) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1XEfGx-0007MT-04 for qemu-devel@nongnu.org; Tue, 05 Aug 2014 09:59:33 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1XEfGq-0003kv-NP for qemu-devel@nongnu.org; Tue, 05 Aug 2014 09:59:26 -0400 Received: from mx1.redhat.com ([209.132.183.28]:9298) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1XEfGq-0003kP-Gd for qemu-devel@nongnu.org; Tue, 05 Aug 2014 09:59:20 -0400 Date: Tue, 5 Aug 2014 14:59:13 +0100 From: Stefan Hajnoczi Message-ID: <20140805135913.GE12251@stefanha-thinkpad.redhat.com> References: <1407209598-2572-1-git-send-email-ming.lei@canonical.com> <20140805093830.GD3186@stefanha-thinkpad.redhat.com> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="idY8LE8SD6/8DnRI" Content-Disposition: inline In-Reply-To: Subject: Re: [Qemu-devel] [PATCH v1 00/17] dataplane: optimization and multi virtqueue support List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Ming Lei Cc: Kevin Wolf , Peter Maydell , Fam Zheng , "Michael S. Tsirkin" , qemu-devel , Paolo Bonzini --idY8LE8SD6/8DnRI Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Tue, Aug 05, 2014 at 05:50:42PM +0800, Ming Lei wrote: > On Tue, Aug 5, 2014 at 5:38 PM, Stefan Hajnoczi wro= te: > > On Tue, Aug 05, 2014 at 11:33:01AM +0800, Ming Lei wrote: > >> These patches bring up below 4 changes: > >> - introduce object allocation pool and apply it to > >> virtio-blk dataplane for improving its performance > >> > >> - introduce selective coroutine bypass mechanism > >> for improving performance of virtio-blk dataplane with > >> raw format image > >> > >> - linux-aio changes: fixing for cases of -EAGAIN and partial > >> completion, increase max events to 256, and remove one unuseful > >> fields in 'struct qemu_laiocb' > >> > >> - support multi virtqueue for virtio-blk > > > > Please split up this patch series into separate patch series. > > > > These are independent changes and there is no reason to combine them. > > You're doing yourself a disservice because changes that are ready to be > > applied are getting held up by those that still need more discussion. >=20 > Without previous optimization patches, the mq conversion can't > obtain so much improvement, that is why I put them together. You can post two sets of numbers: "independent results" and "together with series X, Y, and Z". > Also mq conversion depends on linux-aio fix too. No problem, just include a note in the cover letter for the mq series stating that this series depends on the linux-aio fix. Maintainers keep an eye out for that and make sure that the dependencies are merged before applying. > Also it becomes a difficult to test these patches if they are splitted, > and describing the dependency is a bit annoying too. I understand that you need to manage extra branches and sometimes rebase between them. But that's life. The reason people are pushing back is that you are throwing a blob at the mailing list and expecting reviewers to dissect it. Reviewers have to put more effort in than necessary. As a result they are scrutinizing your changes and are not comfortable with them in their current form. If you want to get patches merged smoothly, split them up and justify each series with a cover letter and performance results (if it's an optimization). That way you get reviewers on your side; they understand and agree with the benefit of the series. Make them *want* to merge the patches. --idY8LE8SD6/8DnRI Content-Type: application/pgp-signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v1 iQEcBAEBAgAGBQJT4OMxAAoJEJykq7OBq3PIesQH/3RWWwILyMNk6kRXp1ZZfZxC I/ZdY0+8plU8sVYpAHh9ZeUsUcxuFoHV2AUY4W7qWV3rAtnEGZOrYVBJ+01Cmh6c nBSSvfF3aMjHf6lt3OWozbLOLTCy/GxZRGOC6/EY+OOHApZxmFVcottKzI5meRk+ E/8/SeviDU0sGWDOVmDNcuQbV1Bs5ERaaWeyGGRl+azs+m0Bg3dJ99p5svXw1h9/ NjpyaV0Qw8YPzXjzBVPjqRi2gUl5XuS7FWMnsnTRqBlVX0kj+d+zEM1f0o8hYvZI QB6M0Zr/HJX1GSPfOIYvoH8pbePPynaCGXGvkEhx4w/9Fcxpc8UIH2x0cWRNXyI= =B6AF -----END PGP SIGNATURE----- --idY8LE8SD6/8DnRI--