From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:52234) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1YmMQp-0006is-MZ for qemu-devel@nongnu.org; Sun, 26 Apr 2015 09:17:12 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1YmMQm-0001yq-GO for qemu-devel@nongnu.org; Sun, 26 Apr 2015 09:17:11 -0400 Date: Sun, 26 Apr 2015 15:17:03 +0200 From: Thomas Huth Message-ID: <20150426151703.161075bb@tpl530> In-Reply-To: <20150318122235.7eb71f08@oc7435384737.ibm.com> References: <20150318122235.7eb71f08@oc7435384737.ibm.com> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH v2 0/7] Remove more unused functions List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org, qemu-trivial@nongnu.org Cc: Thomas Huth Am Wed, 18 Mar 2015 12:22:35 +0100 schrieb Thomas Huth : > > Here are some more patches to remove completely unused functions > from QEMU. Please review carefully, some of the functions might > still get usefull in the future again, so if you discover one, > please let me know, then I'll remove it from the patch series > again. > > v2: Changed series according to review comment from v1: > - Dropped the patch to remove portio_list_destroy() and > portio_list_del() > - Keep the ARI functions in the pci patch since they might get useful > again > > Thomas Huth (7): > migration: Remove unused functions > vmxnet: Remove unused function vmxnet_rx_pkt_get_num_frags() > pci: Remove unused function ich9_d2pbr_init() > monitor: Remove unused functions > usb: Remove unused functions > util: Remove unused functions > Remove various unused functions Ping! Now that 2.3.0 (congrats!) has been tagged, could somebody (Michael via qemu-trivial tree?) please pick up patches 2 to 7? As far as I know, there haven't been any complaints about them anymore, so I think it should be ok to include them. (patch 1 has already been included via the migration tree, so that's already fine) Cheers, Thomas