A minha duvida é, eu realmente preciso abrir e fechar a conexão a cada chamada do controller?
Conexões com bancos de dados costumam ser stateless. Ou seja, abre, faz o que tem que fazer e fecha.
Eu, particularmente, não vejo motivos para segurar uma conexão aberta com o banco, até mesmo porque existem configurações de datasources que justamente verificam se existem conexões ociosas e as fecham.
Quando digo "faz o que tem que fazer" não significa executar uma única consulta. Você pode executar N consultas, N transações e etc.
Se você tiver uma lista de clientes para cadastrar no banco, por exemplo, você não precisa abrir e fechar uma conexão para cada item dessa lista. Você pode muito bem abrir uma conexão, iterar a lista inserindo todos os clientes no banco através dessa conexão e depois fechá-la. O importante é não segurar a conexão aberta "pra sempre".
Do modo que utilizo hoje não ira me trazer problemas de performance?
Se você usar Pool de Conexões não. O pool foi concebido exatamente com o propósito de facilitar/deixar menos custoso resgatar uma conexão com o banco.