C ++内存泄漏,同时试图删除指针链表的第一个元素

时间:2012-01-04 16:11:36

标签: c++ list memory-leaks

我对C ++很陌生,现在我遇到了两天大问题。 我正在尝试使用posix线程进行多线程渲染(光线投射,多重采样,环境约束),并且每次运行程序时,它都会消耗大约5GB的RAM(在线程启动后),直到终止。显然我有记忆泄漏。我的工作线程是这样的:

struct Job
{
    AOSampler sampler;
    Ray ray;
    bool abool;
    int someint;
    .
    .
    //no pointers here
};
//global
//use of C++ STL list
list<Job*> * jobs;


//Part of thread posix function starts here
list<Job*> tjobs;
// Mark 1    

//Pushing and popping between "tjobs" the threadjobs just for this thread and the global jobpool "jobs". Of course threadsafe with mutex locking.
//The thread pops jobs from "jobs" and puts em into "tjobs"

while(!tjobs.empty())
{
    //many calculations but all vars are on stack, besides creating new jobs an pushing them to some other queue, which will be pushed into "jobs" later

    // !!!THE PROBLEM!!!
    delete (tjobs.front());
    tjobs.pop_front();
    // The memory in htop always rises but never decreases!
}
// jumps to Mark 1
// end of multithread while

代码正在编译并运行并在许多内核上终止但性能很差(4好,24坏,它是24核机器)。我认为这可能是因为5GB的内存使用量(占所有内存的四分之一),但操作系统和缓存可能无法很好地处理这个问题。

我很想找到解决问题的方法。我的谷歌搜索根本没有帮助我。我跳你可以做。任何帮助都非常感谢。

谢谢

(对不起我的英文)

Edit1:忘了提,它还没有输出 - &gt;我无法验证它是否有效

EDIT2: 一些标题:

class AOSampler
{
public:
    AOSampler();
    /// constructor that initializes the sampler, just calls init
    AOSampler(vec3 const & normal, vec3 const & color);
    /// initializes the sampler
    void init(vec3 const & normal, vec3 const & color);
    /// returns an importance sampled random direction and the associated weight
    void sample(vec3 & sampledDirection, vec3 & sampleWeight) const;
private:
    /// orthonormal basis
    vec3 m_normal;
    vec3 m_tangent;
    vec3 m_bitangent;
    /// diffuse color
    vec3 m_color;
};

class Ray
{
public:
    Ray() : tMin(0.001f), tMax(FLT_MAX){}
    vec3 origin;
    vec3 direction;
    float tMin, tMax;
};

class vec3
{
public:
    float x,y,z;

    vec3();
    vec3(float a, float b, float c);

    /// assignment operator that assigns a single scalar value to all components
    void operator=(float v);

    /// unsafe element access 
    float operator[](unsigned int i) const
    {
        return (&x)[i];
    }

    /// length of the vector
    float length() const;

    ///Returns a normalized version of the vector
    vec3 normalize() const;

    /// componentwise summation
    vec3 add(const vec3& a) const;

    /// componentwise subtraction
    vec3 subtract(const vec3& a) const;

    ///compute the dot product which is cos(alpha) * this.Length * a.Length
    ///where alpha is the (smaller) angle between the vectors
    float dot(const vec3& a) const;

    float minComponent() const;
    float maxComponent() const;

    ///computes a vector which is orthogonal to both of the input vectors
    static vec3 cross(const vec3& a, const vec3& b);
    static vec3 min(const vec3& a, const vec3& b);
    static vec3 max(const vec3& a, const vec3& b);

    /// add a vector to this vector
    void operator+=( vec3 const & v );
    /// subtract a vector from this vector
    void operator-=( vec3 const & v );
};

2 个答案:

答案 0 :(得分:1)

您的while循环仅在列表为空时循环,然后您尝试删除不存在的第一个元素。当然这会导致奇怪的行为。或者更有可能的是,你已经把物品放入队列中,永远不会将它们拉出来,这会导致它永远增长。

因此,我们假设您没有向我们展示您的真实代码,而是您重新输入的内容并忘记了!字符,而while循环是正确的。

在那种情况下,你确定它实际上是在泄漏吗?你的进程可能正在使用更多的内存,但如果它正在释放它(它暂时似乎是这样),即使操作系统无法在htop中看到内存,该进程也能够重用它。您可以使用valgrind来更好地了解您是否真的泄漏。

答案 1 :(得分:0)

即使您有权访问源代码,也很难检测到内存泄漏。尝试使用泄漏检测工具,例如Valgrind:http://www.cprogramming.com/debugging/valgrind.html 这是一项很好的投资。