* [PATCH 1/2] kernel-shark: Enforce update of the OpenGL widget when loading new session
@ 2019-05-02 12:45 Yordan Karadzhov
2019-05-02 12:45 ` [PATCH 2/2] kernel-shark: Add "Report bug" button to "Help" Yordan Karadzhov
2019-05-03 8:25 ` [PATCH 1/2] kernel-shark: Enforce update of the OpenGL widget when loading new session Slavomir Kaslev
0 siblings, 2 replies; 3+ messages in thread
From: Yordan Karadzhov @ 2019-05-02 12:45 UTC (permalink / raw)
To: rostedt; +Cc: linux-trace-devel, y.karadz, Yordan Karadzhov
This is needed in the case when the session was saved at different screen
resolution.
Reported-by: Steven Rostedt (VMware) <rostedt@goodmis.org>
Signed-off-by: Yordan Karadzhov <ykaradzhov@vmware.com>
---
kernel-shark/src/KsGLWidget.hpp | 2 ++
kernel-shark/src/KsMainWindow.cpp | 3 +--
kernel-shark/src/KsTraceGraph.cpp | 2 ++
3 files changed, 5 insertions(+), 2 deletions(-)
diff --git a/kernel-shark/src/KsGLWidget.hpp b/kernel-shark/src/KsGLWidget.hpp
index 3bcecf9..bf68be3 100644
--- a/kernel-shark/src/KsGLWidget.hpp
+++ b/kernel-shark/src/KsGLWidget.hpp
@@ -41,6 +41,8 @@ public:
void reset();
+ void update() {resizeGL(width(), height());}
+
void mousePressEvent(QMouseEvent *event);
void mouseMoveEvent(QMouseEvent *event);
diff --git a/kernel-shark/src/KsMainWindow.cpp b/kernel-shark/src/KsMainWindow.cpp
index bb593bd..3cf10bc 100644
--- a/kernel-shark/src/KsMainWindow.cpp
+++ b/kernel-shark/src/KsMainWindow.cpp
@@ -435,7 +435,6 @@ QString KsMainWindow::lastSessionFile()
void KsMainWindow::_restoreSession()
{
loadSession(lastSessionFile());
- _graph.updateGeom();
}
void KsMainWindow::_importSession()
@@ -450,7 +449,6 @@ void KsMainWindow::_importSession()
return;
loadSession(fileName);
- _graph.updateGeom();
}
void KsMainWindow::_updateSession()
@@ -1100,6 +1098,7 @@ void KsMainWindow::loadSession(const QString &fileName)
_session.loadTable(&_view);
_colorPhaseSlider.setValue(_session.getColorScheme() * 100);
+ _graph.updateGeom();
}
void KsMainWindow::_initCapture()
diff --git a/kernel-shark/src/KsTraceGraph.cpp b/kernel-shark/src/KsTraceGraph.cpp
index 6087e96..da2c6aa 100644
--- a/kernel-shark/src/KsTraceGraph.cpp
+++ b/kernel-shark/src/KsTraceGraph.cpp
@@ -538,6 +538,8 @@ void KsTraceGraph::updateGeom()
* allow the scroll bar to disappear when the
* widget is extended to maximum.
*/
+
+ _glWindow.update();
}
void KsTraceGraph::_updateGraphLegends()
--
2.20.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* [PATCH 2/2] kernel-shark: Add "Report bug" button to "Help"
2019-05-02 12:45 [PATCH 1/2] kernel-shark: Enforce update of the OpenGL widget when loading new session Yordan Karadzhov
@ 2019-05-02 12:45 ` Yordan Karadzhov
2019-05-03 8:25 ` [PATCH 1/2] kernel-shark: Enforce update of the OpenGL widget when loading new session Slavomir Kaslev
1 sibling, 0 replies; 3+ messages in thread
From: Yordan Karadzhov @ 2019-05-02 12:45 UTC (permalink / raw)
To: rostedt; +Cc: linux-trace-devel, y.karadz, Yordan Karadzhov
When clicked, the Bugzilla will be opened in the appropriate Web browser
for the user's desktop environment.
Suggested-by: Steven Rostedt (VMware) <rostedt@goodmis.org>
Signed-off-by: Yordan Karadzhov <ykaradzhov@vmware.com>
---
kernel-shark/src/KsMainWindow.cpp | 13 +++++++++++++
kernel-shark/src/KsMainWindow.hpp | 4 ++++
2 files changed, 17 insertions(+)
diff --git a/kernel-shark/src/KsMainWindow.cpp b/kernel-shark/src/KsMainWindow.cpp
index 3cf10bc..c7c7f6b 100644
--- a/kernel-shark/src/KsMainWindow.cpp
+++ b/kernel-shark/src/KsMainWindow.cpp
@@ -67,6 +67,7 @@ KsMainWindow::KsMainWindow(QWidget *parent)
_fullScreenModeAction("Full Screen Mode", this),
_aboutAction("About", this),
_contentsAction("Contents", this),
+ _bugReportAction("Report a bug", this),
_deselectShortcut(this),
_settings("kernelshark.org", "Kernel Shark") // organization , application
{
@@ -288,6 +289,9 @@ void KsMainWindow::_createActions()
_contentsAction.setIcon(QIcon::fromTheme("help-contents"));
connect(&_contentsAction, &QAction::triggered,
this, &KsMainWindow::_contents);
+
+ connect(&_bugReportAction, &QAction::triggered,
+ this, &KsMainWindow::_bugReport);
}
void KsMainWindow::_createMenus()
@@ -365,6 +369,7 @@ void KsMainWindow::_createMenus()
help = menuBar()->addMenu("Help");
help->addAction(&_aboutAction);
help->addAction(&_contentsAction);
+ help->addAction(&_bugReportAction);
}
void KsMainWindow::_open()
@@ -925,6 +930,14 @@ void KsMainWindow::_contents()
QUrl::TolerantMode));
}
+void KsMainWindow::_bugReport()
+{
+ QUrl bugs("https://bugzilla.kernel.org/buglist.cgi?component=Trace-cmd%2FKernelshark&product=Tools&resolution=---",
+ QUrl::TolerantMode);
+
+ QDesktopServices::openUrl(bugs);
+}
+
/** Load trace data for file. */
void KsMainWindow::loadDataFile(const QString& fileName)
{
diff --git a/kernel-shark/src/KsMainWindow.hpp b/kernel-shark/src/KsMainWindow.hpp
index 962de2d..22700d9 100644
--- a/kernel-shark/src/KsMainWindow.hpp
+++ b/kernel-shark/src/KsMainWindow.hpp
@@ -151,6 +151,8 @@ private:
QAction _contentsAction;
+ QAction _bugReportAction;
+
QShortcut _deselectShortcut;
QString _lastDataFilePath, _lastConfFilePath, _lastPluginFilePath;
@@ -205,6 +207,8 @@ private:
void _contents();
+ void _bugReport();
+
void _captureStarted();
void _captureError(QProcess::ProcessError error);
--
2.20.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH 1/2] kernel-shark: Enforce update of the OpenGL widget when loading new session
2019-05-02 12:45 [PATCH 1/2] kernel-shark: Enforce update of the OpenGL widget when loading new session Yordan Karadzhov
2019-05-02 12:45 ` [PATCH 2/2] kernel-shark: Add "Report bug" button to "Help" Yordan Karadzhov
@ 2019-05-03 8:25 ` Slavomir Kaslev
1 sibling, 0 replies; 3+ messages in thread
From: Slavomir Kaslev @ 2019-05-03 8:25 UTC (permalink / raw)
To: Yordan Karadzhov
Cc: Steven Rostedt, linux-trace-devel, Yordan Karadzhov (VMware)
On Thu, May 2, 2019 at 3:46 PM Yordan Karadzhov <ykaradzhov@vmware.com> wrote:
>
> This is needed in the case when the session was saved at different screen
> resolution.
>
> Reported-by: Steven Rostedt (VMware) <rostedt@goodmis.org>
> Signed-off-by: Yordan Karadzhov <ykaradzhov@vmware.com>
> ---
> kernel-shark/src/KsGLWidget.hpp | 2 ++
> kernel-shark/src/KsMainWindow.cpp | 3 +--
> kernel-shark/src/KsTraceGraph.cpp | 2 ++
> 3 files changed, 5 insertions(+), 2 deletions(-)
Both patches look good to me
Reviewed-by: Slavomir Kaslev <kaslevs@vmware.com>
Cheers,
-- Slavi
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2019-05-03 8:26 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-05-02 12:45 [PATCH 1/2] kernel-shark: Enforce update of the OpenGL widget when loading new session Yordan Karadzhov
2019-05-02 12:45 ` [PATCH 2/2] kernel-shark: Add "Report bug" button to "Help" Yordan Karadzhov
2019-05-03 8:25 ` [PATCH 1/2] kernel-shark: Enforce update of the OpenGL widget when loading new session Slavomir Kaslev
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).