diff --git a/go/packages/golist.go b/go/packages/golist.go
index f378242..099207d 100644
--- a/go/packages/golist.go
+++ b/go/packages/golist.go
@@ -25,6 +25,7 @@
 	"golang.org/x/tools/go/internal/packagesdriver"
 	"golang.org/x/tools/internal/gocommand"
 	"golang.org/x/tools/internal/packagesinternal"
+	"golang.org/x/xerrors"
 )
 
 // debug controls verbose logging.
@@ -736,7 +737,7 @@
 		if !ok {
 			// Catastrophic error:
 			// - context cancellation
-			return nil, fmt.Errorf("couldn't run 'go': %v", err)
+			return nil, xerrors.Errorf("couldn't run 'go': %w", err)
 		}
 
 		// Old go version?
diff --git a/go/packages/loadmode_string.go b/go/packages/loadmode_string.go
index aff94a3..7ea37e7 100644
--- a/go/packages/loadmode_string.go
+++ b/go/packages/loadmode_string.go
@@ -38,7 +38,7 @@
 func (mod LoadMode) String() string {
 	m := mod
 	if m == 0 {
-		return fmt.Sprintf("LoadMode(0)")
+		return "LoadMode(0)"
 	}
 	var out []string
 	for i, x := range allModes {
diff --git a/internal/lsp/debug/serve.go b/internal/lsp/debug/serve.go
index 2004716..11364b6 100644
--- a/internal/lsp/debug/serve.go
+++ b/internal/lsp/debug/serve.go
@@ -34,6 +34,7 @@
 	"golang.org/x/tools/internal/telemetry/export/metric"
 	"golang.org/x/tools/internal/telemetry/export/ocagent"
 	"golang.org/x/tools/internal/telemetry/export/prometheus"
+	"golang.org/x/xerrors"
 )
 
 type instanceKeyType int
@@ -543,8 +544,16 @@
 func makeGlobalExporter(stderr io.Writer) event.Exporter {
 	return func(ctx context.Context, ev event.Event, tags event.TagMap) context.Context {
 		i := GetInstance(ctx)
-		if ev.IsLog() && (event.Err.Get(ev.Map()) != nil || i == nil) {
-			fmt.Fprintf(stderr, "%v\n", ev)
+
+		if ev.IsLog() {
+			// Don't log context cancellation errors.
+			if err := event.Err.Get(ev.Map()); xerrors.Is(err, context.Canceled) {
+				return ctx
+			}
+			// Make sure any log messages without an instance go to stderr.
+			if i == nil {
+				fmt.Fprintf(stderr, "%v\n", ev)
+			}
 		}
 		ctx = protocol.LogEvent(ctx, ev, tags)
 		if i == nil {
diff --git a/internal/lsp/generate.go b/internal/lsp/generate.go
index 08840aa..2bcd037 100644
--- a/internal/lsp/generate.go
+++ b/internal/lsp/generate.go
@@ -7,7 +7,6 @@
 import (
 	"context"
 	"io"
-	"log"
 	"math/rand"
 	"strconv"
 
@@ -15,7 +14,7 @@
 	"golang.org/x/tools/internal/lsp/debug/tag"
 	"golang.org/x/tools/internal/lsp/protocol"
 	"golang.org/x/tools/internal/telemetry/event"
-	errors "golang.org/x/xerrors"
+	"golang.org/x/xerrors"
 )
 
 func (s *Server) runGenerate(ctx context.Context, dir string, recursive bool) {
@@ -43,12 +42,14 @@
 	}
 	stderr := io.MultiWriter(er, wc)
 	err := inv.RunPiped(ctx, er, stderr)
-	if err != nil && !errors.Is(ctx.Err(), context.Canceled) {
-		log.Printf("generate: command error: %v", err)
-		s.client.ShowMessage(ctx, &protocol.ShowMessageParams{
-			Type:    protocol.Error,
-			Message: "go generate exited with an error, check gopls logs",
-		})
+	if err != nil {
+		event.Error(ctx, "generate: command error: %v", err, tag.Directory.Of(dir))
+		if !xerrors.Is(err, context.Canceled) {
+			s.client.ShowMessage(ctx, &protocol.ShowMessageParams{
+				Type:    protocol.Error,
+				Message: "go generate exited with an error, check gopls logs",
+			})
+		}
 	}
 }
 
diff --git a/internal/lsp/protocol/context.go b/internal/lsp/protocol/context.go
index dac850d..560de4d 100644
--- a/internal/lsp/protocol/context.go
+++ b/internal/lsp/protocol/context.go
@@ -19,9 +19,6 @@
 }
 
 func LogEvent(ctx context.Context, ev event.Event, tags event.TagMap) context.Context {
-	if ctx.Err() != nil {
-		return ctx
-	}
 	if !ev.IsLog() {
 		return ctx
 	}
diff --git a/internal/lsp/source/references.go b/internal/lsp/source/references.go
index d1cc602..a6d35e8 100644
--- a/internal/lsp/source/references.go
+++ b/internal/lsp/source/references.go
@@ -6,13 +6,13 @@
 
 import (
 	"context"
-	"errors"
 	"go/ast"
 	"go/token"
 	"go/types"
 
 	"golang.org/x/tools/internal/lsp/protocol"
 	"golang.org/x/tools/internal/telemetry/event"
+	"golang.org/x/xerrors"
 )
 
 // ReferenceInfo holds information about reference to an identifier in Go source.
@@ -33,7 +33,7 @@
 
 	qualifiedObjs, err := qualifiedObjsAtProtocolPos(ctx, s, f, pp)
 	// Don't return references for builtin types.
-	if errors.Is(err, errBuiltin) {
+	if xerrors.Is(err, errBuiltin) {
 		return nil, nil
 	}
 	if err != nil {