CODEWITHSUNDEEP
cudathrust
Vitality
Vitality

Reputation: 21495

Converting Thrust device iterators to raw pointers

I'm considering the following simple code in which I'm converting thrust::host_vector<int>::iterator h_temp_iterator = h_temp.begin(); and thrust::device_vector<int>::iterator d_temp_iterator = d_temp.begin(); to raw pointers.

To this end, I'm passing &(h_temp_iterator[0]) and &(d_temp_iterator[0]) to a function and a kernel, respectively. The former (CPU case) compiles, the latter (GPU case) not. The two cases should be in principle symmetric, so I do not understand the reason for the error message which is:

Error   1   error : no suitable conversion function from "thrust::device_ptr<int>" to "int *" exists

The configurations are:

  1. Windows 7, Visual Studio 2010, CUDA 7.5, compiling for the 3.5 architecture.
  2. Windows 10, Visual Studio 2013, CUDA 8.0, compiling for the 5.2 architecture.

CODE

#include <thrust\host_vector.h>
#include <thrust\device_vector.h>

__global__ void testKernel(int *a, const int N)
{
    int i = threadIdx.x;

    if (i >= N) return;

    a[i] = 2;
}

void testFunction(int *a, const int N)
{
    for (int i = 0; i < N; i++) a[i] = 2;
}

int main()
{
    const int N = 10;

    thrust::host_vector<int> h_temp(N);
    thrust::device_vector<int> d_temp(N);

    thrust::host_vector<int>::iterator h_temp_iterator = h_temp.begin();
    thrust::device_vector<int>::iterator d_temp_iterator = d_temp.begin();

    testFunction(&(h_temp_iterator[0]), N);
    testKernel<<<1, N>>>(&(d_temp_iterator[0]), N);

    for (int i = 0; i < N; i++) printf("%i %i\n", i, h_temp[i]);

    return 0;
}

Upvotes: 4

Views: 1913

Answers (2)

luke8800gts
luke8800gts

Reputation: 428

As you are using thrust, another prettier solution is to get the pointer with data() and cast it to a raw pointer:

thrust::raw_pointer_cast(d_temp_iterator.data())

Upvotes: 4

Vitality
Vitality

Reputation: 21495

Following talonmies' comments, the solution is to pass

thrust::raw_pointer_cast(&d_temp_iterator[0])

and not

&d_temp_iterator[0]

In the following, the fully working code

#include <thrust\host_vector.h>
#include <thrust\device_vector.h>

__global__ void testKernel(int *a, const int N)
{
    int i = threadIdx.x;

    if (i >= N) return;

    a[i] = 2;

    printf("GPU %i %i\n", i, a[i]);
}

void testFunction(int *a, const int N)
{
    for (int i = 0; i < N; i++) {
        a[i] = 2;
        printf("CPU %i %i\n", i, a[i]);
    }
}

int main()
{
    const int N = 10;

    thrust::host_vector<int> h_temp(N);
    thrust::device_vector<int> d_temp(N);

    thrust::host_vector<int>::iterator h_temp_iterator = h_temp.begin();
    thrust::device_vector<int>::iterator d_temp_iterator = d_temp.begin();

    int *temp = thrust::raw_pointer_cast(&d_temp_iterator[0]);

    testFunction(&(h_temp_iterator[0]), N);
    testKernel<<<1, N>>>(temp, N);

    return 0;
}

Upvotes: 5

Related Questions