From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1M9E8K-0005YS-Pm for qemu-devel@nongnu.org; Wed, 27 May 2009 04:05:08 -0400 Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1M9E8G-0005VN-0W for qemu-devel@nongnu.org; Wed, 27 May 2009 04:05:08 -0400 Received: from [199.232.76.173] (port=47757 helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1M9E8F-0005VK-TT for qemu-devel@nongnu.org; Wed, 27 May 2009 04:05:03 -0400 Received: from mx20.gnu.org ([199.232.41.8]:30521) by monty-python.gnu.org with esmtps (TLS-1.0:RSA_AES_256_CBC_SHA1:32) (Exim 4.60) (envelope-from ) id 1M9E8F-0001dt-52 for qemu-devel@nongnu.org; Wed, 27 May 2009 04:05:03 -0400 Received: from gecko.sbs.de ([194.138.37.40]) by mx20.gnu.org with esmtp (Exim 4.60) (envelope-from ) id 1M9E8D-0000ED-Cr for qemu-devel@nongnu.org; Wed, 27 May 2009 04:05:01 -0400 Message-ID: <4A1CF428.2000104@siemens.com> Date: Wed, 27 May 2009 10:04:56 +0200 From: Jan Kiszka MIME-Version: 1.0 References: <1243335807-9495-1-git-send-email-agraf@suse.de> <4A1CE497.4070507@web.de> <4A1CE9FC.7000601@web.de> <8493542A-A2A1-4AC0-B4F0-3A2139C02D49@suse.de> In-Reply-To: <8493542A-A2A1-4AC0-B4F0-3A2139C02D49@suse.de> Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Subject: [Qemu-devel] Re: [PATCH] User Networking: Enable removal of redirections List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Alexander Graf Cc: Blue Swirl , qemu-devel@nongnu.org Alexander Graf wrote: > > On 27.05.2009, at 09:21, Jan Kiszka wrote: > >> Jan Kiszka wrote: >>> Alexander Graf wrote: >>>> Using the new host_net_redir command you can easily create redirections >>>> on the fly while your VM is running. >>>> >>>> While that's great, it's missing the removal of redirections, in >>>> case you >>>> want to have a port closed again at a later point in time. >>>> >>>> This patch adds support for removal of redirections. >>> >>> Cool. I was just too lazy to hack on slirp for this so far, but now you >>> did it. >>> >>>> Signed-off-by: Alexander Graf >>>> --- >>>> monitor.c | 5 +++-- >>>> net.c | 43 ++++++++++++++++++++++++++++++++++++++++++- >>>> net.h | 2 +- >>>> slirp/libslirp.h | 1 + >>>> slirp/slirp.c | 23 +++++++++++++++++++++++ >>>> vl.c | 2 +- >>>> 6 files changed, 71 insertions(+), 5 deletions(-) >>>> >>>> diff --git a/monitor.c b/monitor.c >>>> index 0f38c71..dbab3de 100644 >>>> --- a/monitor.c >>>> +++ b/monitor.c >>>> @@ -1759,8 +1759,9 @@ static const mon_cmd_t mon_cmds[] = { >>>> { "host_net_remove", "is", net_host_device_remove, >>>> "vlan_id name", "remove host VLAN client" }, >>>> #ifdef CONFIG_SLIRP >>>> - { "host_net_redir", "s", net_slirp_redir, >>>> - "[tcp|udp]:host-port:[guest-host]:guest-port", "redirect TCP >>>> or UDP connections from host to guest (requires -net user)" }, >>>> + { "host_net_redir", "ss?", net_slirp_redir, >>>> + "[tcp|udp]:host-port:[guest-host]:guest-port", "redirect TCP >>>> or UDP connections from host to guest (requires -net user)\n" >>> >>> If going for a single command, this should rather look like this: >>> >>> "[tcp|udp]:host-port:[guest-host]:guest-port | >>> remove [tcp|udp]:host-port" >>> >>> BTW, I didn't refactor this interface along with the command line >>> changes, but I should actually. To remain consistent, we should call it >>> host_net_fwd (-redir will be replaced with hostfwd=) or even take >>> the chance and split it into host_net_fwd_add and host_net_fwd_remove. >>> >>>> + "host_net_redir remove [tcp:|udp:]host-port -- remove >>>> redirection" }, >>>> #endif >>>> { "balloon", "i", do_balloon, >>>> "target", "request VM to change it's memory allocation (in >>>> MB)" }, >> >> Before I forget: You also have to update to monitor-related >> documentation when adding/changing commands. > > What monitor-related documentation? I did a grep -R host_net_redir qemu > and didn't find any other occurence of that command :o. Yeah, I also failed in the discipline of properly updating qemu-doc.texi when adding host_net_redir. :-> I think Blueswirl recently posted patch to fold monitor options and documentations into one file just like we already have for the command line. Would be helpful if that get merged soon Jan -- Siemens AG, Corporate Technology, CT SE 2 Corporate Competence Center Embedded Linux