Can't achieve minimum setp and hold time error?? :((

Right-click on one of the failing paths and do a List Paths. You'll see detailed information you break out on the path below. 99% of the time when there's an internal hold violation, it's because you have a gated clock(i.e. there is logic on the clock path going to the memory.) If this is the case, you want to remove that so the clock directly feeds all logic, and use synchronous logic to control when the memory is written to.