From: Abdiel Janulgue <abdiel.janulgue@linux.intel.com>
To: Petri Latvala <petri.latvala@intel.com>
Cc: intel-gfx@lists.freedesktop.org
Subject: Re: [i-g-t PATCH 1/4] lib/igt_core: Add igt_exec helpers
Date: Wed, 10 May 2017 15:30:59 +0300 [thread overview]
Message-ID: <bad1cba9-bd3c-e2ed-c056-35cc52653d94@linux.intel.com> (raw)
In-Reply-To: <20170510121444.ixhv5l4a5dk3ot4c@platvala-desk.ger.corp.intel.com>
On 10.05.2017 15:14, Petri Latvala wrote:
> On Wed, May 10, 2017 at 03:10:16PM +0300, Abdiel Janulgue wrote:
>> In case the stream output is more than the pipe buf, the read loop above
>> would just unblock the rest of the entries. From pipe(2) page:
>
>
> But you don't reach that read loop until system() returns. Which
> happens after the program has terminated. Which never happens.
>
Experimented with setting the write end of the pipe as O_NONBLOCK, I
actually didn't notice any difference when dumping tons of stdout
compared with just setting only the read end.
Maybe it's because we are not using write() directly? Not sure if this
is expected behavior or not...
>
>
> --
> Petri Latvala
>
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
prev parent reply other threads:[~2017-05-10 12:31 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-04-20 8:13 [i-g-t PATCH 1/4] lib/igt_core: Add igt_exec helpers Abdiel Janulgue
2017-04-20 8:13 ` [i-g-t PATCH 2/4] igt/igt_core: Provide an option to check for the log buffer contents Abdiel Janulgue
2017-04-20 8:13 ` [i-g-t PATCH 3/4] lib/igt_debugfs: Add helper to return path to device Abdiel Janulgue
2017-05-09 10:21 ` Petri Latvala
2017-04-20 8:13 ` [i-g-t PATCH 4/4] Convert shell script tests to C version Abdiel Janulgue
2017-04-25 9:54 ` [i-g-t PATCH] Convert ddx_intel_after_fbdev to C Abdiel Janulgue
2017-05-09 10:55 ` [i-g-t PATCH 4/4] Convert shell script tests to C version Petri Latvala
2017-05-09 10:18 ` [i-g-t PATCH 1/4] lib/igt_core: Add igt_exec helpers Petri Latvala
2017-05-10 12:10 ` Abdiel Janulgue
2017-05-10 12:14 ` Petri Latvala
2017-05-10 12:30 ` Abdiel Janulgue [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=bad1cba9-bd3c-e2ed-c056-35cc52653d94@linux.intel.com \
--to=abdiel.janulgue@linux.intel.com \
--cc=intel-gfx@lists.freedesktop.org \
--cc=petri.latvala@intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox