the code you have in the thread here is incomplete...here is a full example of a stored procedure, which takes 2 parameters (you can add/remove them at will), just follow the syntax
SET ANSI_NULLS ON
GO
SET QUOTED_IDENTIFIER ON
GO
CREATE PROCEDURE proc_myStoredProcName(@variable1 varchar(20),@variable2 int)
AS
BEGIN
-- SET NOCOUNT ON added to prevent extra result sets from interfering with SELECT statements. u can comment it or delete it if not needed
SET NOCOUNT ON;
--your select,update,insert statement below
SELECT id,user_id,login_time FROM somedatabasename.dbo.user_activity WHERE login_time='this works'
END
GO
Hope this helps..:-)