From mboxrd@z Thu Jan 1 00:00:00 1970 From: Peter Xu Subject: Re: [kvm-unit-tests PATCH v2 2/2] run_tests: allow run tests in parallel Date: Fri, 6 Jan 2017 11:31:12 +0800 Message-ID: <20170106033112.GB4135@pxdev.xzpeter.org> References: <1483438205-31110-1-git-send-email-peterx@redhat.com> <1483438205-31110-3-git-send-email-peterx@redhat.com> <20170104150938.GA9029@potion> <20170105030746.GN22664@pxdev.xzpeter.org> <20170105194401.GD29430@potion> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Cc: Paolo Bonzini , Andrew Jones , qemu-devel@nongnu.org, kvm@vger.kernel.org To: Radim =?utf-8?B?S3LEjW3DocWZ?= Return-path: Content-Disposition: inline In-Reply-To: <20170105194401.GD29430@potion> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: qemu-devel-bounces+gceq-qemu-devel=gmane.org@nongnu.org Sender: "Qemu-devel" List-Id: kvm.vger.kernel.org On Thu, Jan 05, 2017 at 08:44:02PM +0100, Radim Kr=C4=8Dm=C3=A1=C5=99 wro= te: [...] > > Anyway, IMHO squashing you suggestion of "trap exit SIGINT" at the > > entry of for_each_unittest() is an acceptable solution - it works in > > all cases. >=20 > Seems like the best solution at this time ... > We actually want to "exit 130", as it would when using normal wait does= , > and maybe it could be improved a bit by adding a wait: >=20 > trap 'wait; exit 130' SIGINT Sounds better, thanks! Will repost with this one. -- peterx