2023-05-07 05:43:22

by Randy Dunlap

[permalink] [raw]
Subject: [PATCH] Documentation/block: request: delete the unused FIXME comment

This FIXME comment looks like it could be a local file contents
attempt, but this file is short and does not needs its own contents
block, so drop it completely.

Signed-off-by: Randy Dunlap <[email protected]>
Cc: Jens Axboe <[email protected]>
Cc: [email protected]
Cc: Jonathan Corbet <[email protected]>
Cc: [email protected]
---
Documentation/block/request.rst | 17 -----------------
1 file changed, 17 deletions(-)

diff -- a/Documentation/block/request.rst b/Documentation/block/request.rst
--- a/Documentation/block/request.rst
+++ b/Documentation/block/request.rst
@@ -5,23 +5,6 @@ struct request documentation
Jens Axboe <[email protected]> 27/05/02


-.. FIXME:
- No idea about what does mean - seems just some noise, so comment it
-
- 1.0
- Index
-
- 2.0 Struct request members classification
-
- 2.1 struct request members explanation
-
- 3.0
-
-
- 2.0
-
-
-
Short explanation of request members
====================================


2023-05-07 12:40:39

by Jens Axboe

[permalink] [raw]
Subject: Re: [PATCH] Documentation/block: request: delete the unused FIXME comment

On 5/6/23 10:50?PM, Randy Dunlap wrote:
> This FIXME comment looks like it could be a local file contents
> attempt, but this file is short and does not needs its own contents
> block, so drop it completely.

I had a quick look at that file (which I didn't even remember existed),
and let's just burn it down. It's way out dated at this point, should
just be removed.

--
Jens Axboe