diff mbox series

[3/4] kernel-shark-qt: Remove TODO comment in kshark_load_data_entries()

Message ID 20180829162927.20200-3-y.karadz@gmail.com (mailing list archive)
State Accepted
Headers show
Series [1/4] kernel-shark-qt: Change "pevent_" to "tep_" in Doxygen. | expand

Commit Message

Yordan Karadzhov Aug. 29, 2018, 4:29 p.m. UTC
The problem described by the TODO message was fixed, but we forgot
to remove the message itself.

Signed-off-by: Yordan Karadzhov (VMware) <y.karadz@gmail.com>
---
 kernel-shark-qt/src/libkshark.c | 7 -------
 1 file changed, 7 deletions(-)
diff mbox series

Patch

diff --git a/kernel-shark-qt/src/libkshark.c b/kernel-shark-qt/src/libkshark.c
index 1b7e544..b4a76ae 100644
--- a/kernel-shark-qt/src/libkshark.c
+++ b/kernel-shark-qt/src/libkshark.c
@@ -712,13 +712,6 @@  ssize_t kshark_load_data_entries(struct kshark_context *kshark_ctx,
 	if (*data_rows)
 		free(*data_rows);
 
-	/*
-	 * TODO: Getting the records separately slows this function
-	 *       down, instead of just accessing the records when
-	 *	 setting up the kernel entries. But this keeps the
-	 *	 code simplier. We should revisit to see if we can
-	 *	 bring back the performance.
-	 */
 	total = get_records(kshark_ctx, &rec_list, type);
 	if (total < 0)
 		goto fail;