Issue with XPTI subscriber detection for subprocess inheritance

オープン
#96 コメント 3 件 リアクション 0 件 担当者 0 名 GitHub で見る

まだ誰も着手していません。

評価

難易度
3/5
見積もり時間
1〜2日
初心者へのやさしさ
45/100
issue の種類
バグ
明瞭さ
おおむね明確
活発さ
停滞
技術スタック
cpp, python
領域
devtools

調査の方向性

Issue に示されている IsSubscriberToXPTI の実装から始め、PTI ライブラリを初期化する Python subprocess test をたどってください。Linux で継承された libpti.so の値、Windows での pti.dll に対する動作を検証し、unitrace と外部の subscriber の検出を維持してください。子プロセスが PTI 自体について警告しなくなり、実際の外部 subscriber は引き続き識別できれば完了です。

索引モデルが issue の本文から書いたものです。

説明

The last release introduced an issue with subprocess XPTI subscriber detection:

Process 1 (Parent Process):

  1. Python test initializes and loads PTI library
  2. PTI queries XPTI_SUBSCRIBERS environment variable: Result = Not Set
  3. PTI registers as XPTI subscriber by setting: XPTI_SUBSCRIBERS=/path/to/libpti.so

Process 2 (Child Process via subprocess):

  1. Child process inherits env from parent process including: XPTI_SUBSCRIBERS=/path/to/libpti.so (inherited)
  2. Child process loads PTI library
  3. PTI queries XPTI_SUBSCRIBERS environment variable: Result = Already Set
  4. PTI incorrectly interprets parent's PTI subscription as foreign subscriber
  5. PTI outputs warning: "Another subscriber already subscribed to Sycl runtime events..."

Root Cause

The current implementation only checks if XPTI_SUBSCRIBERS is non-empty, without difference between PTI itself and real foreign subscribers.

Current Implementation

inline std::pair<bool, bool> IsSubscriberToXPTI() {
  auto current_xpti_subscriber = utils::GetEnv("XPTI_SUBSCRIBERS");
  // and may be it is unitrace
  return {!current_xpti_subscriber.empty(),
          current_xpti_subscriber.find("unitrace") != std::string::npos};
}

Proposed Fix

inline std::pair<bool, bool> IsSubscriberToXPTI() {
  auto current_xpti_subscriber = utils::GetEnv("XPTI_SUBSCRIBERS");
  bool is_unitrace = current_xpti_subscriber.find("unitrace") != std::string::npos;

  if (current_xpti_subscriber.empty()) {
    return {false, is_unitrace};
  }
  
  // Check if the subscriber is PTI itself (not foreign for linux + win)
  if (current_xpti_subscriber.find("libpti") != std::string::npos ||
      current_xpti_subscriber.find("pti.dll") != std::string::npos) {
    return {false, is_unitrace};
  }
  
  // It's a real foreign subscriber (not PTI)
  return {true, is_unitrace};
}

Related issue : https://github.com/intel/intel-xpu-backend-for-triton/issues/4998

主要言語
C++
スター
278
フォーク
75
PR マージ指標
30日以内にマージされた PR はありません

環境構築

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

intel/pti-gpu のほかの issue

intel/pti-gpu の issue をすべて見る

似ている issue

C++ の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。