FourierFlows / GeophysicalFlows.jl

Geophysical fluid dynamics pseudospectral solvers with Julia and FourierFlows.jl.
https://fourierflows.github.io/GeophysicalFlowsDocumentation/stable/
MIT License
155 stars 32 forks source link

Use KernelAbstractions to accelerate `MultilayerQG.streamfunctionfrompv!` #112

Open glwagner opened 4 years ago

glwagner commented 4 years ago

KernelAbstractions.jl can be used to accelerate the function

https://github.com/FourierFlows/GeophysicalFlows.jl/blob/47a2b51def8af38b727dd64542c87051f169d5ea/src/multilayerqg.jl#L299-L302

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

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

glwagner commented 4 years ago

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 commented 4 years ago

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 commented 4 years ago

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 commented 4 years ago

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

glwagner commented 4 years ago

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 commented 4 years ago

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

glwagner commented 4 years ago

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 commented 4 years ago

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 commented 3 years ago

I should resurrect this..

navidcy commented 3 years ago

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

glwagner commented 3 years ago

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

mpudig commented 3 months ago

Hi @navidcy @glwagner – I am hoping to resurrect this issue. I'd like to run a significant number of high vertical resolution simulations and leveraging GPU capabilities would be hugely beneficial...

I am CUDA/GPU-literate but by no means fluent, and am trying to understand this thread and your thinking behind what a possible fix would be. For my own understanding, the main issue is the scalar indexing of S in the PV inversion step, right?

https://github.com/FourierFlows/GeophysicalFlows.jl/blob/634ef2d1d67e42a4b13b4af298bd475810cf5f98/src/multilayerqg.jl#L536-L548

It seems you've both thought of a few ways to rewrite the calculation of S and the PV inversion matrix-matrix multiply so that performance is optimized for GPUs. I'd be more than happy to have a go at changing the code and running the tests myself, but I might need a bit of guidance around what method proposed in this thread would be a fruitful initial direction to go in...!

Thanks

glwagner commented 3 months ago

The issue is the explicit loop over i, j. To utilize the GPU you have to write a kernel, hopefully using KernelAbstractions like we do in Oceananigans. Check out the docs for KernelAbstractions and write/run a few kernels for the GPU to test your skills. This is a fairly simple kernel so hopefully it will be fairly straightforward.

glwagner commented 3 months ago

Just want to emphasize that it is easy to learn KernelAbstractions, you can try just running the docs examples to start which should take a few minutes, and then spend an hour or two to learn how to write your own. At that point you're ready to solve this problem and run high res simulations.

mpudig commented 3 months ago

Okay, awesome, thanks for the direction, Greg! I'll give this a go and hopefully will run into few difficulties. Famous last words...!

glwagner commented 3 months ago

image

navidcy commented 3 months ago

you made this meme???

glwagner commented 3 months ago

yes

navidcy commented 3 months ago

It's a great one ;)

mpudig commented 3 months ago

I had a go at writing the kernel using KernelAbstractions.jl and modifying the MultiLayerQG.pvfromstreamfunction! call:

@kernel function pvfromstreamfunction_kernel!(qh, ψh, S, nlayers)
  i, j = @index(Global, NTuple)

  @unroll for k = 1:nlayers

      @inbounds qh[i, j, k] = 0

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

  end
end

and

function pvfromstreamfunction!(qh, ψh, params, grid)
  # 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 8
  workgroup = 8, 8

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

  # Instantiates the kernel for relevant backend device
  backend = KernelAbstractions.get_backend(qh)
  kernel! = pvfromstreamfunction_kernel!(backend, workgroup, worksize)

  # Launch the kernel
  S, nlayers = params.S, params.nlayers
  kernel!(qh, ψh, S, nlayers)

  # This will ensure that no other operations occur until the kernel has finished
  KernelAbstractions.synchronize(backend)

  return nothing
end

I ran some simple benchmark tests on a GPU (rtx8000), a CPU with 16 threads and a CPU with 1 thread. For example, with nlayers = 3 and nx = 128 the improvement on the GPU is significant:

Something I was surprised by was that a multi-threaded CPU performed slightly better than the GPU in terms of speed in some cases. For example, with nlayers = 12 and nx = 512

Does this surprise you?

In any case, the rewrite definitely accelerates MultiLayerQG.pvfromstreamfunction! on GPUs for arbitrary nlayers. Let me know if you would like me to open a PR to add the changes!

mpudig commented 2 weeks ago

@navidcy @glwagner Just following up on the above in case you missed it. Let me know if you think I should open a PR to add this to the MultiLayerQG code.

navidcy commented 2 weeks ago

Open a PR I think! Yeah!

glwagner commented 2 weeks ago

For example, with nlayers = 3 and nx = 128 the improvement on the GPU is significant:

The speed up is 1000x so "significant" is an understatement

Does this surprise you?

Yes, if you can show the code or open a PR maybe we will spot something.