From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:42750) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ayezH-0005X0-8b for qemu-devel@nongnu.org; Fri, 06 May 2016 08:36:13 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ayez5-0004sR-Gs for qemu-devel@nongnu.org; Fri, 06 May 2016 08:36:01 -0400 Received: from mx1.redhat.com ([209.132.183.28]:42055) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ayez5-0004nn-7f for qemu-devel@nongnu.org; Fri, 06 May 2016 08:35:55 -0400 Date: Fri, 6 May 2016 13:35:39 +0100 From: "Dr. David Alan Gilbert" Message-ID: <20160506123539.GD2278@work-vm> References: <1461941263-9523-1-git-send-email-dgilbert@redhat.com> <1461941263-9523-5-git-send-email-dgilbert@redhat.com> <5725C206.3020304@redhat.com> <20160503092208.GD2242@work-vm> <572877E9.9010100@redhat.com> <20160503103423.GF2242@work-vm> <572880A3.7040504@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <572880A3.7040504@redhat.com> Subject: Re: [Qemu-devel] [PATCH v2 4/5] test: Postcopy List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Marcel Apfelbaum Cc: qemu-devel@nongnu.org, quintela@redhat.com, amit.shah@redhat.com, aarcange@redhat.com, den@openvz.org, marcel.a@redhat.com, eblake@redhat.com * Marcel Apfelbaum (marcel@redhat.com) wrote: > > > > > > +int main(int argc, char **argv) > > > > > > +{ > > > > > > + char template[] = "/tmp/postcopy-test-XXXXXX"; > > > > > > > > > > I would not explicitly use /tmp/ > > > > > > > > The ivshmem-test, vhost-user-test and test-qga seem to do it this way; > > > > what's your preferred fix? > > > > > > You could use the P_tmpdir macro instead of '/tmp', but again, > > > if all the tests assume /tmp existence maybe is not an issue. > > > > I don't see any use of P_tmpdir in qemu at all; we do have one use of > > g_get_tmp_dir (in util/memfd.c). > > Even better! I like the way it look for it: > On UNIX, this is taken from the TMPDIR environment variable. > If the variable is not set, P_tmpdir is used, as defined by the system C library. > Failing that, a hard-coded default of "/tmp" is returned. Lets go through and do the /tmp changes another time but to all of the tests; lets keep them consistent for now. I've just posted a version with the assert changes you suggested in. Dave > > > > > > > > > + int ret; > > > > > > + > > > > > > + g_test_init(&argc, &argv, NULL); > > > > > > + > > > > > > + if (!ufd_version_check()) { > > > > > > + return 0; > > > > > > + } > > > > > > + > > > > > > + tmpfs = mkdtemp(template); > > > > > > + if (!tmpfs) { > > > > > > + g_test_message("mkdtemp on path (%s): %s\n", template, strerror(errno)); > > > > > > + } > > > > > > + g_assert(tmpfs); > > > > > > + > > > > > > + module_call_init(MODULE_INIT_QOM); > > > > > > + > > > > > > + qtest_add_func("/postcopy", test_migrate); > > > > > > + > > > > > > > > > > How much time does this test takes? If is too long, maybe we should not run it > > > > > automatically as part of make check, but using an environment variable. > > > > > > > > 4 seconds on my laptop; it seems reasonable. > > > > > > > > > > make-check takes about 1 and a half minute for x86_64 configuration, 4 seconds > > > more seems reasonable indeed. > > > > Dave > > > > > Thanks, > > > Marcel > > > > > > > > > > Dave > > > > -- > > > > Dr. David Alan Gilbert / dgilbert@redhat.com / Manchester, UK > > > > > > > > > -- > > Dr. David Alan Gilbert / dgilbert@redhat.com / Manchester, UK > > > -- Dr. David Alan Gilbert / dgilbert@redhat.com / Manchester, UK