Project

General

Profile

Actions
Copy link

Bug #1102

open

TouchProcessor fails on pinet to exit or notify on exit when running on pistore.

Added by Hammel 10 months ago. Updated 5 months ago.

Status:
In Progress
Priority:
Immediate
Assignee:
Hammel
Target version:
PiBox - 3.0 - Corrino
Start date:
04 Mar 2024
Due date:
% Done:

50%

Estimated time:
Severity:
01 - Critical

Description

Failure on pinet shutdown on a pistore:

piboxTouchShutdownProcessor[touchProcessor.c:619] ERROR Timed out waiting on touchProcessor thread to shut down.

Doesn't happen all the time and the timeout eventually clears the problem.

Actions
Copy link
 #1

Updated by Hammel 8 months ago

  • Severity changed from 03 - Medium to 01 - Critical
Actions
Copy link
 #2

Updated by Hammel 6 months ago

  • Status changed from New to In Progress
  • % Done changed from 0 to 30

This might be due to the animation timer interfering with the SIGINT processing. There was no call to g_source_remove() in pinet.c on shutdown.

This is fixed in my sandbox but needs to be tested thoroughly on hardware.

Note that the following all need updates related to the SIGINT call.
  • picam
  • musicfe
  • pipics

This means they all need this code on shutdown.

    if ( isCLIFlagSet(CLI_TOUCH) )
    {   
        /* SIGINT is required to exit the touch processor thread. */
        raise(SIGINT);
        piboxTouchShutdownProcessor();
    }

And the following need updates to disable a timer before the SIGINT call.
  • pistore
  • pinet
Actions
Copy link
 #3

Updated by Hammel 6 months ago

  • Subject changed from TouchProcessor fails to exit or notify on exit on pistore. to TouchProcessor fails on pinet to exit or notify on exit when running on pistore.
Actions
Copy link
 #4

Updated by Hammel 5 months ago

  • % Done changed from 30 to 50

All changes are implemented but all changes need to be tested on hardware.

Media System can be used to test the following:
  • pipics
  • musicfe
  • picam
PiStore system can be used to test the following:
  • pistore
  • pinet
Actions
Copy link

Also available in: Atom PDF