Has it worked like this before?
I only ask because I've dealt with some ISPs who filter traffic by MAC address,, forcing you to call your ISP to clear the mac table in the modem, making doing what you are trying to do not optional. Best idea would to get a router to handle the sharing of the internet connection.