First off, I know there are some people here who would swear that you should get one 1GB stick instead of two 512MB sticks. This way you would have more RAM slots open for future expansion. I agree with them, but it's not a horribly huge deal, IMO. Both have their ups and downs, because you can run two in dual-channel mode for better performance, but you can't have (overall) as much memory. If you get a 1GB stick now, and another later then you can have 2GB in dual-channel. Think about getting a 1GB stick instead of two halves, so that way you won't max out your RAM at 2GB in the future. This is something to think about for games that love RAM (see: Battlefront 2).
Also, some places I've seen list the Corsair TWINX sticks that you've chosen run at 1.9v instead of the 1.8v that the motherboard says it needs. From what I understand, this is okay, but from what I've heard you might need to buy another (cheap) stick of, say, 128 or 256MB ram that can run at 1.8v in order to start up the computer. Once it starts booting, you'll have to go into the BIOS and change the RAM voltages so that the 1.9v sticks will work on your motherboard, shut it down again and then you can put these sticks in.
Here's one that seems to have good reviews over at trusty newegg, and looks like it runs at 1.8v (so you won't have to hassle with the paragraph above):
http://www.tigerdirect.ca/applications/SearchTools/item-details.asp?EdpNo=2458715&Tab=0&NoMapp=0
It's a little more expensive than the one you've picked out, so if you want to look around for another 2x512MB kit that has 1.8v, that would work just as well!
Cheers.