开发者

Segmentation fault while pushing elements to a queue

开发者 https://www.devze.com 2023-04-11 21:03 出处:网络
I am programming in Cpp after a long time, and I am stuck at a place. the code is given below (its long so be patient):

I am programming in Cpp after a long time, and I am stuck at a place. the code is given below (its long so be patient):

bool mazerouter( int xs, int ys, int ps, int xt, int yt, int pt )
{
    int n = gn; int w = gw;
    // The expansion list as maintained by the maze router. Could be declared as a class variable
    // but then, we need to find a way to empty the queue every time mazerouter is exited.
     queue <Wire_seg> explist;

    // Step 1: mark all the track segments that are available and adjacent to target as "t".
    // this depends on pin number.
     vector <  vector<int> > target = getWireSegments( xt, yt, pt );
    //int skip = 1/fc_in;
    int i,j;

    std::vector < std::vector <Wire_seg> > HorzWire = fpga.getHorzWire();
    std::vector < std::vector <Wire_seg> > VertWire = fpga.getVertWire();       
    for ( i = 0; i < target.size(); i = i+2 )
    {
        if ( pt == 1 || pt == 3 )
        {
            if ( VertWire[ target[i][0]][ target[i][1] ].getTag() == "a" )
                VertWire[ target[i][0] ][ target[i][1] ].setTag("t");
        }
        else if ( pt == 2 )
        {
            if ( HorzWire[ target[i][0]][ target[i][1] ].getTag() == "a" )
                HorzWire[ target[i][0] ][ target[i][1] ].setTag("t");
        }
    } cout<<"Completed routing step 1\n";

    // Step 2: mark all the track segments that are available and adjacent to source as "o". 
    // Push all the wire_segs in expansion list.
     vector <  vector<int> > source = getWireSegments( xs, ys, ps );
    //skip = 1/fc_out;
    len = 0; // for initialization of the source wire segments
    for ( i = 0; i < source.size(); i = i+1 )
    {
        if ( HorzWire[ source[i][0] ][ source[i][1] ].getTag() == "a" )
        {
            HorzWire[ source[i][0] ][ source[i][1] ].setNtag( 0 );
            HorzWire[ source[i][0] ][ source[i][1] ].setTag( "u" );
            explist.push( HorzWire[ source[i][0] ][ source[i][1] ] );
        }
    } cout<<"Completed routing step 2 and expansion list size: "<<explist.size()<<"\n";

    // The while loop

    bool pathfound = false;     
     vector <  vector <int> > h;
     vector <  vector <int> > v;
    while ( !explist.empty() )
    {
        Wire_seg ws = explist.front();
         cout<<"Completed routing step 3 substep 1 with front's tag:"<<explist.front().getTag()<<"\n";
        explist.pop();
        h = ws.getHorzCon();
        v = ws.getVertCon();
        len = ws.getNtag() + 1;
        int a;
        for ( a = 0; a < h.size(); a++ )
        {
            if ( HorzWire[ h[a][0] ][ h[a][1] ].getTag() == "t" )
            {
                 cout<<"target hit in horzwire\n";
                pathfound = true; 

                //path.push_back(  vector <  string > () ); pl++;   
                //path.at(pl).push_back( fpga.getHorzWire( h[a][0], h[a][1] ).getUid() );   
                break;
            }   
            else if ( HorzWire[ h[a][0] ][ h[a][1] ].getTag() == "a" )
            {
                HorzWire[ h[a][0] ][ h[a][1] ].setTag( "u" );HorzWire[ h[a][0] ][ h[a][1] ].setNtag( len );
                 cout<<"target not found, pushing horzwire("<<h[a][0]<<","<<h[a][1]<<") in explist with new tag:"<<HorzWire[ h[a][0] ][ h[a][1] ].getTag()<<"\n";
                explist.push( HorzWire[ h[a][0] ][ h[a][1] ] );
            }
        }
         cout<<"Completed routing step 3 substep 2\n";
        for ( a = 0; a < v.size(); a++ )
        {
            if ( VertWire[ v[a][0] ][ v[a][1] ].getTag() == "t" )
            {
                pathfound = true; 
                //path.push_back(  vector <  string > () ); pl++;   
                //path.at(pl).push_back( fpga.getVertWire( v[a][0], v[a][1] ).getUid() );
                break;
            }
            else if ( VertWire[ v[a][0] ][ v[a][1] ].getTag() == "a" )
            {
                VertWire[ v[a][0] ][ v[a][1] ].setTag( "u" ); VertWire[ v[a][0] ][ v[a][1] ].setNtag( len );
                                    // the following is the line causing trouble
                explist.push( VertWire[ v[a][0] ][ v[a][1] ] ); <=================================================================
                cout<<"target not found, pushing vertwire("<<v[a][0]<开发者_开发技巧<","<<v[a][1]<<") in explist with new tag:"<<VertWire[ v[a][0] ][ v[a][1] ].getTag()<<"\n";
            }
        }
        h.clear();
        v.clear();
         cout<<"Completed routing step 3 substep 3 and expansion list size: "<<explist.size()<<"\n";
    }// end while
     cout<<"Completed routing step 3\n";
    return pathfound;
}// end mazerouter

I am getting a Segmentation fault for the line 'explist.push( VertWire[ v[a][0] ][ v[a][1] ] );" which has been marked as "<==" in the code. Once I comment the line and run the code, i get no fault, but of-course, i find no solution to my problem. Any idea what I am doing wrong? Anything would be of great help. thanks in advance

what this function does: Given source block (xs,ys) and incoming direction(ps, has four values corresponding to east, west, north, south) and target block (xs, ys) and outgoing direction (pt, same as ps), i need to find a route to connect them. Actually, I am implementing routing in FPGA where Wire_seg is a class for wire segments between Logic blocks (which is implemented as Logic_Block class) and i need to route between logic blocks.

As far as printing value of 'VertWire[ v[a][0] ][ v[a][1] ]' is concerned, it is a class, and hence i am accessing it's member variable tag to ensure that it exists.

@daniel: i am already using a g++ compiler.

The code for "Wire_seg.h" is:

class Wire_seg
{
// the track number for each wire segment, varies from 0 to w-1.
int tno;

// the orientation of the wire segment. 
char ornt;      

// vector to store the indices of all possible connections that can be made from
// this wire segment. 
std::vector < std::vector<int> > horz_con;
std::vector < std::vector<int> > vert_con;

//an unique id given to each wire of the following format "ornt:row:col"
std::string uid;

// The tag that helps determine whether this wire is availabel for a path or not.
// initially all wires are available.
std::string tag;

int ntag;

 public:    

Wire_seg()
{
    tag = "a";
}

    void setHorzCon( std::vector < std::vector<int> > h )
{
    horz_con = h;
}

void setVertCon( std::vector < std::vector<int> > v )
{
    vert_con = v;
}

void setTno( int t )
{
    tno = t;
}

void setOrnt( char orientation )
{
    ornt = orientation;     
}

void setUid( std::string id )
{
    uid = id;
}

void setTag( std::string t )
{
    tag.assign(t);
}

void setNtag( int t )
{
    ntag = t;
}

int getTno()
{
    return tno;
}

char getOrnt()
{
    return ornt;    
}

std::vector < std::vector<int> > getHorzCon()
{
    return horz_con;
}

std::vector < std::vector<int> > getVertCon()
{
    return vert_con;
}

std::string getUid()
{
    return uid;
}

std::string getTag()
{
    return tag;
}

int getNtag()
{
    return ntag;
}
};

I hope this makes the problem more clear.


Your use of std::deque appears pretty safe, so if you're getting a segmentation fault, you may be compiling with an implementation of std::vector that does not perform bounds-checking. Switch that on if you can.

If your environment doesn't provide bounds-checking, try using g++ (which does). Or write a simple wrapper class around std::vector that gives you the required interface: size() and operator[] appear to be all you need. And so do your own bounds checking. Then you'll be able to see where you're accessing past the end of a vector.

0

精彩评论

暂无评论...
验证码 换一张
取 消

关注公众号