Per le performance non saprei risponderti, la differenza credo sia talmente piccola che proprio non so quale potrebbe essere piu' veloce...
Ti posso pero' dire questo:
- con "(byref ds as dataset)" passi un oggetto esistente
- con "private fuction ottieniDataSet() as dataset" crei un nuovo oggetto e quinid utilizzi sicuramente piu' risorse

Quindi, io personalmente ti consiglio di utilizzare "byref"... ma e' solo il mio parere personale...