diff --git a/pageserver/src/tenant/remote_timeline_client/download.rs b/pageserver/src/tenant/remote_timeline_client/download.rs
index e9230c58b6..b5c3a52583 100644
--- a/pageserver/src/tenant/remote_timeline_client/download.rs
+++ b/pageserver/src/tenant/remote_timeline_client/download.rs
@@ -235,8 +235,6 @@ async fn download_object<'a>(
                             Ok(chunk) => chunk,
                             Err(e) => return Err(e),
                         };
-                        // TODO(yuchen): might have performance issue when using borrowed version?
-                        // Problem: input is Bytes, does not satisify IO alignment requirement.
                         buffered.write_buffered_borrowed(&chunk, ctx).await?;
                     }
                     let inner = buffered.flush_and_into_inner(ctx).await?;
diff --git a/pageserver/src/virtual_file/owned_buffers_io/write/flush.rs b/pageserver/src/virtual_file/owned_buffers_io/write/flush.rs
index ba7ba13e93..9ec49c33a8 100644
--- a/pageserver/src/virtual_file/owned_buffers_io/write/flush.rs
+++ b/pageserver/src/virtual_file/owned_buffers_io/write/flush.rs
@@ -17,7 +17,6 @@ pub struct FlushHandle<Buf, W> {
     pub(super) maybe_flushed: Option<FullSlice<Buf>>,
 }
 
-// TODO(yuchen): special actions in drop to clean up the join handle?
 pub struct FlushHandleInner<Buf, W> {
     /// A bi-directional channel that sends (buffer, offset) for writes,
     /// and receives recyled buffer.