|
From: | Wenchao Xia |
Subject: | Re: [Qemu-devel] [PATCH V3 3/3] qemu-iotests: add tests for runtime fd passing via SCM rights |
Date: | Tue, 03 Sep 2013 10:09:53 +0800 |
User-agent: | Mozilla/5.0 (Windows NT 5.1; rv:17.0) Gecko/20130801 Thunderbird/17.0.8 |
δΊ 2013-9-2 22:51, Stefan Hajnoczi ει:
On Mon, Sep 02, 2013 at 05:18:13PM +0800, Wenchao Xia wrote:@@ -125,5 +125,39 @@ class TestFdSets(iotests.QMPTestCase): 'No file descriptor supplied via SCM_RIGHTS') self.vm.shutdown() +#Add fd at runtime, there are two ways: monitor related or fdset related +class TestSCMFd(iotests.QMPTestCase): + def setUp(self): + self.vm = iotests.VM() + qemu_img('create', '-f', iotests.imgfmt, image0, '128K') + #Add a unused monitor, to verify it works fine when two minitor present + self.vm.add_monitor_telnet("0",4445)Sorry to be picky: please use the same whitespace style as the existing code in these patches. # Comments have a space after the hash function(args, have, space)
OK, will check all the comments and fix.
+ self.vm.launch() + + def tearDown(self): + self.vm.shutdown() + os.remove(image0) + + def _send_fd_by_SCM(self): + ret = self.vm.send_fd_scm(image0) + self.assertEqual(ret, 0, 'Failed to send fd with UNIX SCM') + + def test_add_fd(self): + self._send_fd_by_SCM() + result = self.vm.qmp('add-fd', fdset_id=2, opaque='image0:r') + self.assert_qmp(result, 'return/fdset-id', 2) + + def test_getfd(self): + self._send_fd_by_SCM() + result = self.vm.qmp('getfd', fdname='image0:r') + self.assert_qmp(result, 'return', {}) + + def test_closefd(self): + self._send_fd_by_SCM() + result = self.vm.qmp('getfd', fdname='image0:r') + self.assert_qmp(result, 'return', {}) + result = self.vm.qmp('closefd', fdname='image0:r') + self.assert_qmp(result, 'return', {})It would be good to also check the error cases like the existing tests do (e.g. getfd fdname=asdf -> error).
OK. I'll wait a few days and respin with fix. -- Best Regards Wenchao Xia
[Prev in Thread] | Current Thread | [Next in Thread] |