From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <Tim.Bird@sony.com>
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=Sony.onmicrosoft.com; 
	s=selector1-Sony-onmicrosoft-com;
	h=From:Date:Subject:Message-ID:Content-Type:MIME-Version;
	bh=yaXw9p88Lp8LlxmwcOjEtZCKJIPLyAl8xMQiEkSh4+k=;
	b=LXJ77uBi2exHJC/NOE/VJy1qu3SM6WXHQq4hFB7GR+wD/tnCdD1zThmPU9tzTrb58FLdIpRVAN9y0gicMOV+qP/RJNPITRZ6xmG0PiB48fNxEd5Grqm81iIJsHrvmApVNhVUEVWA6Rb+2kMIV7iUVs7BIPDfMGtQm6CO9bEPkB0=
From: "Bird, Timothy" <Tim.Bird@sony.com>
Date: Tue, 31 Oct 2017 17:27:55 +0000
Message-ID: <ECADFF3FD767C149AD96A924E7EA6EAF40ACCEFC@USCULXMSG01.am.sony.com>
References: <1509438715-14039-1-git-send-email-daniel.sangorrin@toshiba.co.jp>
	<006f01d35223$d1415890$73c409b0$@toshiba.co.jp>
In-Reply-To: <006f01d35223$d1415890$73c409b0$@toshiba.co.jp>
Content-Language: en-US
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
Subject: Re: [Fuego] cyclictest patches
List-Id: Mailing list for the Fuego test framework
	<fuego.lists.linuxfoundation.org>
List-Unsubscribe: <https://lists.linuxfoundation.org/mailman/options/fuego>,
	<mailto:fuego-request@lists.linuxfoundation.org?subject=unsubscribe>
List-Archive: <http://lists.linuxfoundation.org/pipermail/fuego/>
List-Post: <mailto:fuego@lists.linuxfoundation.org>
List-Help: <mailto:fuego-request@lists.linuxfoundation.org?subject=help>
List-Subscribe: <https://lists.linuxfoundation.org/mailman/listinfo/fuego>,
	<mailto:fuego-request@lists.linuxfoundation.org?subject=subscribe>
To: Daniel Sangorrin <daniel.sangorrin@toshiba.co.jp>, "fuego@lists.linuxfoundation.org" <fuego@lists.linuxfoundation.org>



> -----Original Message-----
> From: Daniel Sangorrin on Tuesday, October 31, 2017 9:40 AM
> > -----Original Message-----
> > From: fuego-bounces@lists.linuxfoundation.org [mailto:fuego-
> bounces@lists.linuxfoundation.org] On Behalf Of Daniel Sangorrin
> > I have modified the cyclictest benchmark test.
> >
> > - cyclictest: update source code tarball
> >
> > The previous source code was too old and lacked some important
> > flags. I added a patch to build the test for toolchains based on
> > old kernels (maybe this should be upstream)
> >
> > TODO: rt-tests contains other tests that are already in Fuego
> > and can share the same tarball.
> > TODO: cyclictest requires some disturbance running in
> > background.
>=20
> Note: this patch is on my master branch (it was too big for the mailing l=
ist).

Is this tarball made on your system, or does it come from some public sourc=
e?
In general, I'd like to avoid custom-made tarballs in Fuego.  If it's
normal cyclictest source, with some extra custom sauce, I think it would
be better to keep the 2 parts separate, so it's easier to update.

I'll take a look and let you know what I think.
 -- Tim