[Piglit] [PATCH 6/7] Add SyncFileWriter for write/close synchronization on results file.

Eoff, Ullysses A ullysses.a.eoff at intel.com
Mon Feb 7 18:47:31 PST 2011


> -----Original Message-----
> From: piglit-bounces+ullysses.a.eoff=intel.com at lists.freedesktop.org
> [mailto:piglit-bounces+ullysses.a.eoff=intel.com at lists.freedesktop.org]
> On Behalf Of U. Artie Eoff
> Sent: Monday, February 07, 2011 6:06 PM
> To: piglit at lists.freedesktop.org
> Subject: [Piglit] [PATCH 6/7] Add SyncFileWriter for write/close
> synchronization on results file.
> 
> +	@synchronized_self
> +	def write(self, *args):
> +		print >> self.file, ' '.join([str(a) for a in args]),
> +		self.file.flush()
> +		os.fsync(self.file.fileno())

Rats!! I need to resubmit this patch series one more time.  The print technique
used here adds a space at the end of each line, which causes a small bug when
--name is not used explicitly.  That is, the name is not auto-filled in the 
results summary file.

Sorry for the resubmissions!

--U. Artie Eoff


More information about the Piglit mailing list