Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use KernelAbstractions to accelerate MultilayerQG.streamfunctionfrompv! #112

Open
glwagner opened this issue Sep 15, 2020 · 11 comments
Open
Labels
🎮 gpu 🚑 help wanted Extra attention is needed optimization 🏎 ❓ question Further information is requested

Comments

@glwagner
Copy link
Member

KernelAbstractions.jl can be used to accelerate the function

function streamfunctionfrompv!(ψh, qh, params, grid)
for j=1:grid.nl, i=1:grid.nkr
CUDA.@allowscalar @views ψh[i, j, :] .= params.invS[i, j] * qh[i, j, :]
end

A simple example showing how to use KernelAbstractions is the "Naive Transpose":

https://juliagpu.gitlab.io/KernelAbstractions.jl/examples/naive_transpose/

@glwagner
Copy link
Member Author

glwagner commented Sep 15, 2020

The first step is to write a kernel, which will look something like

@kernel invert_column!(ψh, qh, S⁻¹)
    i, j = @index(Global, NTuple)
    @inbounds ψh[i, j] .= S⁻¹[i, j] * qh[i, j]
end

The next step is to create a work layout over which the kernel is launched. If we restrict attention to models that always have more than 32 grid points, we can use something like

# Larger workgroups are generally more efficient. For more generality, we could put an if statement that incurs
# different behavior when either nkl or nl are less than 16
workgroup = 16, 16

# The size determines how many times the kernel is run
worksize = grid.nkr, grid.nl

# This (and its useage below) will ensure the kernel is not run _before_ the data in qh is available
barrier = Event(dev)

# Creates a loop over the specified worksize, using workgroup to organize the computation
loop_invert_column! = invert_column!(dev, workgroup, worksize)

# Launch the kernel
event = loop_invert_column!(ψh, qh, params.invS, dependencies=barrier)

# This will ensure that no other operations occur until the kernel has finished
wait(dev, event)

@glwagner
Copy link
Member Author

glwagner commented Sep 15, 2020

One thing I am not totally sure about is whether KernelAbstractions will compile away the matrix multiplication in @inbounds ψh[i, j] .= S⁻¹[i, j] * qh[i, j]. I think that it will. If not, we may have to unroll our own loop.

@glwagner
Copy link
Member Author

By the way, I think this optimization also requires the columns of ψh[i, j] to be stored as StaticArrays. It looks like ψh is a 3D array right now. Other parts of the code may also have to converted to kernels if this change is made, since broadcasting over the 3D array would no longer work.

@navidcy
Copy link
Member

navidcy commented Sep 15, 2020

With this last suggestion would x, y FFTs work nicely?

@glwagner
Copy link
Member Author

With this last suggestion would x, y FFTs work nicely?

Oof, good point.

Hmm, maybe we need to hand-write the matrix matrix multiply then. Not sure.

@navidcy
Copy link
Member

navidcy commented Sep 15, 2020

yes it's been coming to haunt us either way...
(I remember a similar discussion some months ago...)

@glwagner
Copy link
Member Author

Something like

@kernel invert_column!(ψh, qh, S⁻¹)
    i, j = @index(Global, NTuple)
    ψh_column = view(ψh, i, j, :)
    qh_column = view(qh, i, j, :)
    @inbounds ψh_column .= S⁻¹[i, j] * qh_column
end

might work.

@glwagner
Copy link
Member Author

glwagner commented Sep 16, 2020

Otherwise a kernel along the lines of

using KernelAbstractions.Extras.LoopInfo: @unroll

@kernel invert_column!(ψh, qh, S⁻¹, nz)
    i, j = @index(Global, NTuple)

    @unroll for k = 1:nz

        @inbounds ψh[i, j, k] = 0

        @unroll for m = 1:nz
            @inbounds ψh[i, j, k] += S⁻¹[i, j][k, m] * qh[i, j, m]
        end

    end
end

might work, alternatively. Or maybe my indices are screwed up --- whichever is correct.

Nothing is too difficult, it's just a matter of trying it out.

@navidcy
Copy link
Member

navidcy commented Dec 2, 2020

I should resurrect this..

@navidcy navidcy added 🎮 gpu optimization 🏎 ❓ question Further information is requested labels Dec 2, 2020
@navidcy navidcy added the 🚑 help wanted Extra attention is needed label Jan 28, 2021
@navidcy
Copy link
Member

navidcy commented Mar 18, 2021

What about https://github.com/mcabbott/Tullio.jl to the rescue? (just a random thought)

@glwagner
Copy link
Member Author

There's probably a lot of solutions! I think I gave two, but there might be more.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🎮 gpu 🚑 help wanted Extra attention is needed optimization 🏎 ❓ question Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants