Sem STP, sim, você teria um loop.
Verifique se o STP está habilitado e configurado corretamente, e você não precisa se preocupar com loops.
Terei problemas de loop com essa topologia porque S2 e S3 estão conectados a S1 e s4?
Tudo ficará perfeitamente bem, desde que o STP esteja ATIVADO. STP protegerá você contra loop físico.
Olhando para a sua imagem - no entanto - o que mais está ligado a S1 e S4? Se nada por que não removê-los? O STP desativará um dos links - vamos assumir o S2-S4. Então você vai sair da PC1-S2-S1-S3-S4-somehting (Se "alguma coisa" estiver lá).
Você provavelmente não tem um problema de loop, mas você pode ter um problema de caminho ideal.
referência: link
"The Spanning Tree Protocol (STP) is a network protocol that ensures a loop-free topology"