How to deal with unavoidable circular references
A reasonable solution is the second one, that is to remove the MakeId FK from Vehicles, for two reasons:


The information about the Maker of a certain Model is already contained in the Model record, so storing it inside Vehicle is redundant.
Having both attributes in Vehicle is prone to errors, since one could have inconsistent information inside a Vehicle.