> > suspend;
> > end

Oh, another thing: you only use SUSPEND if you want to return values
to the caller; and even then, only in a SELECTABLE procedure. You
didn't show the proc header but from the body it didn't look like that
kind of SP. (I can imagine that you want to return a status code to
the caller here, e.g. to indicate whether the record already existed
or if you had to insert one, but you wouldn't return multiple rows so
an executable SP - without suspend - looks more logical.)

Grtz,
Paul