On Fri, 4 Jan 2019 19:57:51 +0000
Yordan Karadzhov <ykaradzhov@xxxxxxxxxx> wrote:
> The "Filter" top menu contains only "Show Task" and "Show CPU"
> menu actions ("Hide" versions are removed). The code that adds
> the "Hide" actions is only commented out, because we may change
> our opinion in the future.
Let's avoid just commenting it out. I've done this before and just
regretted it (it makes the code look messy).
If we decided to add it in the future, it's trivial to add those lines
and not to mention, the code is archived in git.
-- Steve
>
> Signed-off-by: Yordan Karadzhov <ykaradzhov@xxxxxxxxxx>
> ---
> kernel-shark-qt/src/KsMainWindow.cpp | 42 ++++++++++++++++++++++++++--
> kernel-shark-qt/src/KsMainWindow.hpp | 4 +++
> 2 files changed, 44 insertions(+), 2 deletions(-)
>
> diff --git a/kernel-shark-qt/src/KsMainWindow.cpp b/kernel-shark-qt/src/KsMainWindow.cpp
> index a375126..3eb3692 100644
> --- a/kernel-shark-qt/src/KsMainWindow.cpp
> +++ b/kernel-shark-qt/src/KsMainWindow.cpp
> @@ -54,6 +54,7 @@ KsMainWindow::KsMainWindow(QWidget *parent)
> _showEventsAction("Show events", this),
> _showTasksAction("Show tasks", this),
> _hideTasksAction("Hide tasks", this),
> + _showCPUsAction("Show CPUs", this),
> _hideCPUsAction("Hide CPUs", this),
> _advanceFilterAction("Advance Filtering", this),
> _clearAllFilters("Clear all filters", this),
> @@ -208,6 +209,9 @@ void KsMainWindow::_createActions()
> connect(&_hideTasksAction, &QAction::triggered,
> this, &KsMainWindow::_hideTasks);
>
> + connect(&_showCPUsAction, &QAction::triggered,
> + this, &KsMainWindow::_showCPUs);
> +
> connect(&_hideCPUsAction, &QAction::triggered,
> this, &KsMainWindow::_hideCPUs);
>
> @@ -322,8 +326,9 @@ void KsMainWindow::_createMenus()
>
> filter->addAction(&_showEventsAction);
> filter->addAction(&_showTasksAction);
> - filter->addAction(&_hideTasksAction);
> - filter->addAction(&_hideCPUsAction);
> +// filter->addAction(&_hideTasksAction);
> + filter->addAction(&_showCPUsAction);
> +// filter->addAction(&_hideCPUsAction);
> filter->addAction(&_advanceFilterAction);
> filter->addAction(&_clearAllFilters);
>
> @@ -621,6 +626,39 @@ void KsMainWindow::_hideTasks()
> dialog->show();
> }
>
> +void KsMainWindow::_showCPUs()
> +{
> + kshark_context *kshark_ctx(nullptr);
> + KsCheckBoxWidget *cpu_cbd;
> + KsCheckBoxDialog *dialog;
> +
> + if (!kshark_instance(&kshark_ctx))
> + return;
> +
> + cpu_cbd = new KsCPUCheckBoxWidget(_data.tep(), this);
> + dialog = new KsCheckBoxDialog(cpu_cbd, this);
> +
> + if (!kshark_ctx->show_cpu_filter ||
> + !kshark_ctx->show_cpu_filter->count) {
> + cpu_cbd->setDefault(true);
> + } else {
> + int nCPUs = tep_get_cpus(_data.tep());
> + QVector<bool> v(nCPUs, false);
> +
> + for (int i = 0; i < nCPUs; ++i) {
> + if (tracecmd_filter_id_find(kshark_ctx->show_cpu_filter, i))
> + v[i] = true;
> + }
> +
> + cpu_cbd->set(v);
> + }
> +
> + connect(dialog, &KsCheckBoxDialog::apply,
> + &_data, &KsDataStore::applyPosCPUFilter);
> +
> + dialog->show();
> +}
> +
> void KsMainWindow::_hideCPUs()
> {
> kshark_context *kshark_ctx(nullptr);
> diff --git a/kernel-shark-qt/src/KsMainWindow.hpp b/kernel-shark-qt/src/KsMainWindow.hpp
> index 301acc9..c29829a 100644
> --- a/kernel-shark-qt/src/KsMainWindow.hpp
> +++ b/kernel-shark-qt/src/KsMainWindow.hpp
> @@ -120,6 +120,8 @@ private:
>
> QAction _hideTasksAction;
>
> + QAction _showCPUsAction;
> +
> QAction _hideCPUsAction;
>
> QAction _advanceFilterAction;
> @@ -173,6 +175,8 @@ private:
>
> void _hideTasks();
>
> + void _showCPUs();
> +
> void _hideCPUs();
>
> void _advancedFiltering();
|