public inbox for kvm@vger.kernel.org
 help / color / mirror / Atom feed
From: Asias He <asias.hejun@gmail.com>
To: Pekka Enberg <penberg@kernel.org>
Cc: Cyrill Gorcunov <gorcunov@gmail.com>, Ingo Molnar <mingo@elte.hu>,
	Sasha Levin <levinsasha928@gmail.com>,
	Prasad Joshi <prasadjoshi124@gmail.com>,
	kvm@vger.kernel.org, Asias He <asias.hejun@gmail.com>
Subject: [PATCH 2/6] kvm tools: Block SIGALRM for both rx and tx thread for virtio net
Date: Thu, 16 Jun 2011 00:48:07 +0800	[thread overview]
Message-ID: <1308156491-18488-2-git-send-email-asias.hejun@gmail.com> (raw)
In-Reply-To: <1308156491-18488-1-git-send-email-asias.hejun@gmail.com>

In the user mode network development process, SIGALRM makes getaddrinfo()
fails in virtio net thread. Blocking SIGALRM solves this problem.

SIGALRM is used by the console code to inject interrupt periodically.

Signed-off-by: Asias He <asias.hejun@gmail.com>
---
 tools/kvm/virtio/net.c |    4 ++++
 1 files changed, 4 insertions(+), 0 deletions(-)

diff --git a/tools/kvm/virtio/net.c b/tools/kvm/virtio/net.c
index 6916af6..ad5fa3f 100644
--- a/tools/kvm/virtio/net.c
+++ b/tools/kvm/virtio/net.c
@@ -91,6 +91,8 @@ static void *virtio_net_rx_thread(void *p)
 	u16 head;
 	int len;
 
+	sig_block(SIGALRM);
+
 	kvm	= p;
 	vq	= &ndev.vqs[VIRTIO_NET_RX_QUEUE];
 
@@ -125,6 +127,8 @@ static void *virtio_net_tx_thread(void *p)
 	u16 head;
 	int len;
 
+	sig_block(SIGALRM);
+
 	kvm	= p;
 	vq	= &ndev.vqs[VIRTIO_NET_TX_QUEUE];
 
-- 
1.7.5.4


  reply	other threads:[~2011-06-15 16:50 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-06-15 16:48 [PATCH 1/6] kvm tools: Introduce a helper to block a singal for a thread Asias He
2011-06-15 16:48 ` Asias He [this message]
2011-06-15 16:48 ` [PATCH 3/6] kvm tools: Block SIGALRM for theadpool thread Asias He
2011-06-15 16:48 ` [PATCH 4/6] kvm tools: Block SIGALRM for ioeventfd thread Asias He
2011-06-15 16:48 ` [PATCH 5/6] kvm tools: Block SIGALRM for vcpu thread using sig_block() helper Asias He
2011-06-15 20:23   ` Ingo Molnar
2011-06-15 16:48 ` [PATCH 6/6] kvm tools: Introduce a helper to show the blocked signals Asias He

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1308156491-18488-2-git-send-email-asias.hejun@gmail.com \
    --to=asias.hejun@gmail.com \
    --cc=gorcunov@gmail.com \
    --cc=kvm@vger.kernel.org \
    --cc=levinsasha928@gmail.com \
    --cc=mingo@elte.hu \
    --cc=penberg@kernel.org \
    --cc=prasadjoshi124@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox