From mboxrd@z Thu Jan 1 00:00:00 1970 From: sudhir kumar Subject: Re: [Autotest] [AUTOTEST] [PATCH 1/2] Add latest LTP test in autotest Date: Wed, 8 Jul 2009 09:52:27 +0530 Message-ID: References: <1246863519.2865.18.camel@localhost.localdomain> <33307c790907061137h3da12536q47517b1662498793@mail.gmail.com> <6ac58f4f0907070831h381a405bqdd58fdfaf132333e@mail.gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: QUOTED-PRINTABLE Cc: Martin Bligh , Autotest mailing list , Lucas Meneghel Rodrigues , Uri Lublin , kvm-devel To: Lucas Meneghel Rodrigues Return-path: Received: from mail-pz0-f193.google.com ([209.85.222.193]:34416 "EHLO mail-pz0-f193.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750758AbZGHEW2 convert rfc822-to-8bit (ORCPT ); Wed, 8 Jul 2009 00:22:28 -0400 Received: by pzk31 with SMTP id 31so3381441pzk.33 for ; Tue, 07 Jul 2009 21:22:27 -0700 (PDT) In-Reply-To: <6ac58f4f0907070831h381a405bqdd58fdfaf132333e@mail.gmail.com> Sender: kvm-owner@vger.kernel.org List-ID: On Tue, Jul 7, 2009 at 9:01 PM, Lucas Meneghel Rodrigues wrote: > On Tue, Jul 7, 2009 at 4:24 AM, sudhir kumar wro= te: >>> OK, Can we encapsulate this into the wrapper though, rather than ma= king >>> people do it manually? in the existing ltp.patch or something? >>> >> definitely we can do that, but that needs to know about all the corn= er >> cases of failure. So may be we can continue enhancing the patch as p= er >> the failure reports on different OSes. > > For the most immediate needs, we could try =A0building LTP with make = -k. > Plain re-package of LTP kinda goes against our own rules. The > preferred way to do testsuite modifications is patching before the > execution. So let's strive to use the approach 'upstream package > unmodified, patch if needed'. That's how distro package does it, make= s > sense for us too. I will request you to merge the patches if they do not appear to see any major changes at the very early. let people see the failures and we can quickly patch autotest to fix it. I can volunteer to look into the ltp issues reported by people or found by me. > >> 1 more thing I wanted to start a discussion on LTP mailing list is t= o >> make aware the testcase if it is running on a physical host or on a >> guest(say KVM guest). Testcases like power management, group >> scheduling fairness etc do not make much sense to run on a guest(as >> they will fail or break). So It is better for the test to recognise >> the environment and not execute if it is under virtualization and it >> is supposed to fail or break under that environment. Does that make >> sense to you also ? > > We need to make an assessment of what we would expect to see failing > under a guest. LTP has a fairly large codebase, so it will be a fair > amount of work. Yeah, as Martin also points the same. At the very least we can expect the new cases to be virtualization aware. For the existing ones we can take it forward gradually, may be catching the test developers individually :) ATM I will suggest to merge the patches in and let get tested so that we can collect failures/breakages if any. > > Lucas > --=20 Sudhir Kumar