Project

General

Profile

Bug #10152

bge debugging always on

Added by Arne Jansen 10 months ago. Updated 9 months ago.

Status:
New
Priority:
Normal
Assignee:
-
Category:
-
Start date:
2018-12-20
Due date:
% Done:

0%

Estimated time:
Difficulty:
Bite-size
Tags:
needs-triage

Description

commit e9bd258d56101ab78fb969d00fe79c53ceb1aa47
Author: David Gwynne <david@gwynne.id.au>
Date:   Fri Jan 30 02:08:46 2015 +0000

    3446 Update bge to support missing 57xx/577xx devices
    Reviewed by: Dan McDonald <danmcd@omniti.com>
    Reviewed by: Josef 'Jeff' Sipek <josef.sipek@nexenta.com>
    Approved by: Garrett D'Amore <garrett@damore.org>

introduced the following change:

-#define        BGE_DEBUGGING           0
+#define        BGE_DEBUGGING           1

which leads to performance problems due to the enabled debugging code.


Related issues

Related to illumos gate - Feature #3446: Update bge to support missing 57xx/577xx devicesClosed2015-04-06

Actions

History

#1

Updated by Marcel Telka 10 months ago

  • Related to Feature #3446: Update bge to support missing 57xx/577xx devices added
#2

Updated by Marcel Telka 10 months ago

  • Description updated (diff)
#3

Updated by Carlos Neira 9 months ago

Arne Jansen wrote:

[...]

introduced the following change:

[...]

which leads to performance problems due to the enabled debugging code.

I just checked the code and this is still hardcoding BGE_DEBUGGING to 1,
to test this I created a PR to illumos-omnios maybe some has a bge card just to test this change.

https://github.com/omniosorg/illumos-omnios/pull/374

Also available in: Atom PDF