Re: IGT conventions

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Jan 16, 2014 at 12:55 AM, Daniel Vetter <daniel@xxxxxxxx> wrote:
> Anything you put out to stderr will be tracked as a "warn" in piglit. Atm
> we don't have any such use-case though I think, mostly since keeping
> unbuffer stderr and buffered stdout in sync is a pain ;-) But I guess we
> could formalize this a bit if you see it useful for you with a
>
> #define igt_warn(a...) fprintf(stderr, a)
>
> or something like that. Some of the checks in kms_flip.c might benefit
> from this, since on a lot of our platforms the rather stringent timing
> checks often fail randomly. But besides such corner-cases I kinda prefer
> if we just split up testcases more instead of trying to be really clever
> with the level of fail encounter and reported.

Actually if we put an fflush(stdout); before the fprintf then we would
not have any issues with buffered vs. unbuffered. And for consistency
maybe we could define igt_warn as just fputs and igt_warn_f as the
full printf thing.

If you think this is useful for your tests then I'll happily merge a
patch to add igt_warn*

Cheers, Daniel
-- 
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux