Skip to content

Commit

Permalink
clean up some synch calls
Browse files Browse the repository at this point in the history
  • Loading branch information
etseidl committed Feb 4, 2023
1 parent cd9bbcf commit 61a64ee
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 4 deletions.
1 change: 0 additions & 1 deletion cpp/src/io/parquet/page_enc.cu
Original file line number Diff line number Diff line change
Expand Up @@ -265,7 +265,6 @@ __global__ void __launch_bounds__(block_size)
__syncthreads();

calculate_frag_size<block_size>(s, t);
__syncthreads();
if (t == 0) { frag[blockIdx.x] = s->frag; }
}

Expand Down
4 changes: 1 addition & 3 deletions cpp/src/io/parquet/writer_impl.cu
Original file line number Diff line number Diff line change
Expand Up @@ -927,8 +927,7 @@ void writer::impl::init_page_fragments(cudf::detail::hostdevice_2dvector<gpu::Pa
void writer::impl::init_page_fragments_1d(hostdevice_vector<gpu::PageFragment>& frag,
host_span<size_type const> frag_sizes)
{
auto d_frag_sz = cudf::detail::make_device_uvector_sync(frag_sizes, stream);

auto d_frag_sz = cudf::detail::make_device_uvector_async(frag_sizes, stream);
InitPageFragments1D(frag, d_frag_sz, stream);
frag.device_to_host(stream, true);
}
Expand Down Expand Up @@ -1693,7 +1692,6 @@ void writer::impl::write(table_view const& table, std::vector<partition_info> co
}
}

// synch will be done in init_page_fragments_1d()
expanded_fragments.host_to_device(stream);
chunks.host_to_device(stream);

Expand Down

0 comments on commit 61a64ee

Please sign in to comment.