4

CPU負荷を定期的に表示するGUIアプリケーションがあります。負荷は StateReader クラスによって読み取られます。

public class StateReader
{
    ManagementObjectSearcher searcher;

    public StateReader()
    {
        ManagementScope scope = new ManagementScope("\\\\localhost\\root\\cimv2");
        ObjectQuery query = new ObjectQuery("select Name,PercentProcessorTime from Win32_PerfFormattedData_PerfOS_Processor where not Name='_Total'");
        searcher = new ManagementObjectSearcher(scope, query);
    }

    // give the maximum load over all cores
    public UInt64 CPULoad()
    {
        List<UInt64> list = new List<UInt64>();
        ManagementObjectCollection results = searcher.Get();
        foreach (ManagementObject result in results)
        {
            list.Add((UInt64)result.Properties["PercentProcessorTime"].Value); 
        }
        return list.Max();
    }
}

GUI は、リアクティブ拡張機能を使用して更新されます。

var gui = new GUI();
var reader = new StateReader();

var sub = Observable.Interval(TimeSpan.FromSeconds(0.5))
                    .Select(_ => reader.CPULoad())
                    .ObserveOn(gui)
                    .Subscribe(gui.ShowCPUState);

Application.Run(gui);
sub.Dispose();

アプリケーションを終了すると、次のエラーが表示されます

RaceOnRCWCleanup was detected. 
An attempt has been mad to free an RCW that is in use. The RCW is use on the 
active thread or another thread. Attempting to free an in-use RCW can cause 
corruption or data loss.

このエラーは、CPU 負荷を読み取らずにランダムな値を指定した場合には表示されないため、負荷の読み取りにエラーが何らかの形で関連しています。また、ブレークポイントを置いてしばらくApplication.Run(gui)待つと、エラーはそれほど頻繁には発生しないようです。

これと私のグーグル検索から、管理名前空間でクラスを使用すると、Runtime Callable Wrapper でラップされた COM オブジェクトを参照するバックグラウンド スレッドが作成され、アプリケーションを終了すると、そのスレッドは適切に閉じる時間がないと思います。 RCW、私のエラーにつながります。これは正しいですか、どうすればこの問題を解決できますか?

受け取った応答を反映するようにコードを編集しましたが、それでも同じエラーが発生します。コードは次の 3 点で更新されます。

  • StateReader は Disposable であり、その ManagementObjectSearcher を Dispose メソッドで破棄し、メイン メソッドで Application.Run の後に StateReader オブジェクトで Dispose を呼び出します。
  • CPULoad では、ManagementCollection とその中の各 ManagementObject を破棄します
  • メイン メソッドでは、GUI の FormClosing のイベント ハンドラーでサブスクリプション オブジェクトを破棄
    します。これにより、GUI を閉じた後にその GUI に対してイベントが生成されないようにする必要があります。

コードの関連部分は現在、StateReader にあります。

// give the maximum load over all cores
public UInt64 CPULoad()
{
    List<UInt64> list = new List<UInt64>();
    using (ManagementObjectCollection results = searcher.Get())
    {
        foreach (ManagementObject result in results)
        {
            list.Add((UInt64)result.Properties["PercentProcessorTime"].Value); 
            result.Dispose();
        }
    }
    return list.Max();
}

public void Dispose()
{
    searcher.Dispose();
}

そして私のメインでは：

gui.FormClosing += (a1, a2) => sub.Dispose();

Application.Run(gui);
reader.Dispose();

エラーを回避するために他にできることはありますか?

4

2 に答える 2

1

StateReaderアプリケーションを終了する前に、使い捨てにして処分する必要があると思います。StateReader処分する必要がありsearcherます。ManagementObjectただし、本当の問題は、あなたが処分していないことだと思いますCPULoad。CPULoadRCW が解放された後に GC が実行される場合。ただし、GC の前に終了すると、例外が発生する可能性があります。

管理名前空間のクラスを使用すると、Runtime Callable Wrapper にラップされた COM オブジェクトを参照するバックグラウンド スレッドが作成されると思います。

Observable.Intervalバックグラウンド スレッドを作成し、CPULoadそのスレッドで実行します。

于 2012-03-07T10:27:06.420 に答える
0

それを避けるために、バックグラウンド スレッドの実行中にアプリケーションを終了させないでくださいCPULoad。

私が思いつく最も簡単な解決策は、新しいフォアグラウンド スレッドから CPU 負荷を取得してからスレッドに参加することです。

public UInt64 CPULoad()
{
    List<UInt64> list = new List<UInt64>();
    Thread thread = new Thread(() =>
    {
        ManagementObjectCollection results = searcher.Get();
        foreach (ManagementObject result in results)
        {
            list.Add((UInt64)result.Properties["PercentProcessorTime"].Value);
        }
    });
    thread.Start();
    thread.Join();
    return list.Max();
}

毎回新しいスレッドを開始するオーバーヘッドは、遅い WMI 呼び出しと比較して無視できます。

コメントで言及されている同期タイマーは、実質的に同じ動作を実現しますが、UI スレッドをブロックし、遅い WMI ではほとんど使用できません。

于 2014-10-20T16:31:15.360 に答える