You are both right, it should definitely be done in a cleaner way. Thomas or I will address it when we get the time. It is the case that most uses should call diskfs_check_readonly rather than using the variable.