Skip to content

GitLab

  • Menu
Projects Groups Snippets
    • Loading...
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Sign in / Register
  • lava lava
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Locked Files
  • Issues 138
    • Issues 138
    • List
    • Boards
    • Service Desk
    • Milestones
    • Iterations
    • Requirements
  • Merge requests 30
    • Merge requests 30
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
    • Test Cases
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Monitor
    • Monitor
    • Incidents
  • Packages & Registries
    • Packages & Registries
    • Container Registry
  • Analytics
    • Analytics
    • CI/CD
    • Code review
    • Insights
    • Issue
    • Repository
    • Value stream
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • lava
  • lavalava
  • Merge requests
  • !1626

Merged
Created Nov 19, 2021 by Paul Sokolovsky@pfalconContributor

lava-outerr: Avoid any chance of buffering on Python side

  • Overview 2
  • Commits 1
  • Pipelines 1
  • Changes 1

This fixes an issue when output on feedback connections of FVP (aka FastModels, for use with which this utility was written in the first place) appeared only after the process terminated. The fix:

  1. Runs Python interpreter with -u (ubuffered) switch.
  2. Instantiates subprocess.Popen with bufsize=0 param.
  3. Reads from process in small chunks, to avoid blocking for long time.

Perhaps only one of these was needed to fix the issue at hand, but be future-proof and apply all.

Signed-off-by: Paul Sokolovsky paul.sokolovsky@linaro.org

Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: lava-outerr-unbuf