Re: [i-g-t PATCH v2 1/4] lib/igt_core: Add igt_exec helpers

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

 



On Tue, May 30, 2017 at 01:31:46PM +0300, Abdiel Janulgue wrote:
> 
> 
> On 29.05.2017 16:01, Chris Wilson wrote:
> > On Mon, May 29, 2017 at 03:08:07PM +0300, Abdiel Janulgue wrote:
> >> Support executing external processes with the goal of capturing its
> >> standard streams to the igt logging infrastructure in addition to its
> >> exit status.
> > 
> > This is not an exec wrapper. This is a system() replacement. That it
> > invokes shell to evaluate the cmd should be front and centre.
> > 
> 
> The problem with system() is that it waits until the process is
> completed. If the process' output is more than PIPE_BUF, it'll block. Do
> you have any suggestions?

I'm just saying the name is misleading. igt_system() and explain the
improvements (concurrency and accessible piped output).
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://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