From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:38864) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1c2090-0005ax-M8 for qemu-devel@nongnu.org; Wed, 02 Nov 2016 14:20:15 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1c208z-00039J-Rn for qemu-devel@nongnu.org; Wed, 02 Nov 2016 14:20:14 -0400 Date: Wed, 2 Nov 2016 14:20:04 -0400 From: Jeff Cody Message-ID: <20161102182004.GB15677@localhost.localdomain> References: <20161102175539.4375-1-mreitz@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20161102175539.4375-1-mreitz@redhat.com> Subject: Re: [Qemu-devel] [PATCH for-2.8? 0/3] block/curl: Drop TFTP "support" List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Max Reitz Cc: qemu-block@nongnu.org, qemu-devel@nongnu.org, Kevin Wolf , Markus Armbruster , Eric Blake On Wed, Nov 02, 2016 at 06:55:36PM +0100, Max Reitz wrote: > See patch 3 for the reason why we have actually never supported TFTP at > all (except for very small files (i.e. below 256 kB or so)). > > I would consider this series a bug fix because, well, it doesn't really > change any functionality, and the bug is "We don't support TFTP but we > pretend we do". > I tend to agree. I'm willing to pull this in through my branch for 2.8, unless there arises some outcry with good reason to keep tftp. > > Alternatives to this approach: > > - Deprecate TFTP first. Wait one version, then drop it. > > We could do this, but I personally don't think it's necessary. We have > done this for host_floppy, but in contrast to host_floppy, TFTP really > has never worked. Thus, I conclude that nobody is actually using it or > has ever used it for real work. > > Still, if you think otherwise, we can still do this, of course. > > > - Don't remove TFTP altogether, but just emit a run-time error like we > do for HTTP servers that do not support range-based requests. > > Seems dirty and not like the real solution to me. Also, we have > removed other block drivers in the past, so I don't think we should > keep TFTP. > Since it is broken by nature, I like your original approach of just removing it. > > Max Reitz (3): > qemu-options: Drop mentions of curl's TFTP support > qapi: Drop curl's TFTP protocol > block/curl: Drop TFTP "support" > > block/curl.c | 20 +------------------- > docs/qmp-commands.txt | 2 +- > qapi/block-core.json | 7 +++---- > qemu-options.hx | 6 +++--- > 4 files changed, 8 insertions(+), 27 deletions(-) A > 3:1 delete to insert ratio, that is an ideal patch series ;-)