bryancall commented on code in PR #13005:
URL: https://github.com/apache/trafficserver/pull/13005#discussion_r2967003782
##########
src/iocore/cache/CacheVC.h:
##########
@@ -289,26 +289,26 @@ struct CacheVC : public CacheVConnection {
union {
uint32_t flags;
struct {
- unsigned int use_first_key : 1;
- unsigned int overwrite : 1; // overwrite first_key Dir if
it exists
- unsigned int close_complete : 1; // WRITE_COMPLETE is final
- unsigned int sync : 1; // write to be committed to
durable storage before WRITE_COMPLETE
- unsigned int evacuator : 1;
- unsigned int single_fragment : 1;
- unsigned int evac_vector : 1;
- unsigned int lookup : 1;
- unsigned int update : 1;
- unsigned int remove : 1;
- unsigned int remove_aborted_writers : 1;
- unsigned int open_read_timeout : 1; // UNUSED
- unsigned int data_done : 1;
- unsigned int read_from_writer_called : 1;
- unsigned int rewrite_resident_alt : 1;
- unsigned int readers : 1;
- unsigned int doc_from_ram_cache : 1;
- unsigned int hit_evacuate : 1;
- unsigned int compressed_in_ram : 1; // compressed state in ram
cache
- unsigned int allow_empty_doc : 1; // used for cache empty http
document
+ unsigned int use_first_key : 1;
+ unsigned int overwrite : 1; // overwrite first_key Dir if
it exists
+ unsigned int close_complete : 1; // WRITE_COMPLETE is final
+ unsigned int sync : 1; // write to be committed to
durable storage before WRITE_COMPLETE
+ unsigned int evacuator : 1;
+ unsigned int single_fragment : 1;
+ unsigned int evac_vector : 1;
+ unsigned int lookup : 1;
+ unsigned int update : 1;
+ unsigned int remove : 1;
+ unsigned int remove_aborted_writers : 1;
+ unsigned int unused_open_read_timeout : 1; // UNUSED, reserved for
bitfield layout
+ unsigned int data_done : 1;
+ unsigned int read_from_writer_called : 1;
+ unsigned int rewrite_resident_alt : 1;
+ unsigned int readers : 1;
Review Comment:
Good catch — updated the PR description. The actual change here is renaming
`open_read_timeout` to `unused_open_read_timeout` and realigning the bitfield
formatting. `rewrite_resident_alt` is still in use and was never intended to be
removed.
##########
include/iocore/aio/AIO.h:
##########
@@ -58,7 +58,6 @@ struct ink_aiocb {
off_t aio_offset = 0; /* file offset */
int aio_lio_opcode = 0; /* listio operation */
- int aio_state = 0; /* state flag for List I/O */
};
Review Comment:
Fixed the PR description — only `aio_state` and padding were removed, not
`aio_nbytes`/`aio_offset`. Those are still actively used in the I/O path.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
