prova
- togliendo gli apici alle immagini tra le parentesi
- definendo un display: block per il link (che è un elemento inline)
- assicurandoti che il percorso sia giusto e sia una path relativa a partire da dove si trova il il file con il tuo css

poi semplificherei il markup... un div per contenere un solo link mi sembra poco pratico, quindi:

Markup
codice:
<div id="contenitore">
  
</div>
CSS
codice:
<style type="text/css">
#roll  {
   background-image:url(percorso/immagine1) ;
   width:250px; 
   height:500px;
   display: block;
}

#roll:hover {
  background-image:url(percorso/immagine2) ;
}
</style>