chore(uffd): fix stale comment and minor cleanups

- userfaultfd.go: correct the `case faulted` branch comment to refer to
  the `removed` page state (previously claimed REMOVE marks the page as
  `missing`) and drop the duplicated "a a" typo.
- userfaultfd.go: restore the colon in the "failed uffdio copy: %w"
  error message that was lost when faultPage was refactored to return
  (bool, error).
- fcversion: simplify HasFreePageReporting to a single boolean return.

Author: ValentaTomas

packages/orchestrator/pkg/sandbox/uffd/userfaultfd/userfaultfd.go

@@ -317,8 +317,8 @@ func (u *Userfaultfd) Serve(
 			case faulted:
 				// Skip faulting the page. This has already been faulted, either during pre-faulting
 				// or because we handled another page fault on the same address in the current
-				// iteration. It can only be removed via a a UFFD_EVENT_REMOVE, which will mark the
-				// page as `missing`.
+				// iteration. It can only transition out of `faulted` via a UFFD_EVENT_REMOVE, which
+				// will mark the page as `removed`.
 				// For this to work correctly, the used pages cannot be swappable.
 				continue
 			case removed:
@@ -512,7 +512,7 @@ func (u *Userfaultfd) faultPage(
 		span.RecordError(joinedErr)
 		u.logger.Error(ctx, "UFFD serve uffdio copy error", zap.Error(joinedErr))
 
-		return false, fmt.Errorf("failed uffdio copy %w", joinedErr)
+		return false, fmt.Errorf("failed uffdio copy: %w", joinedErr)
 	}
 
 	return true, nil

packages/shared/pkg/fcversion/sandbox_features.go

@@ -9,9 +9,5 @@ func (v *Info) HasHugePages() bool {
 }
 
 func (v *Info) HasFreePageReporting() bool {
-	if v.lastReleaseVersion.Major() > 1 || (v.lastReleaseVersion.Major() == 1 && v.lastReleaseVersion.Minor() >= 14) {
-		return true
-	}
-
-	return false
+	return v.lastReleaseVersion.Major() > 1 || (v.lastReleaseVersion.Major() == 1 && v.lastReleaseVersion.Minor() >= 14)
 }