Re: Make pg_stat_io view count IOs as bytes instead of blocks
От | Nazir Bilal Yavuz |
---|---|
Тема | Re: Make pg_stat_io view count IOs as bytes instead of blocks |
Дата | |
Msg-id | CAN55FZ3fXzNhqMvPrDF_MPcyyO536j5xovTZKKZVn8dpCusYZA@mail.gmail.com обсуждение исходный текст |
Ответ на | Re: Make pg_stat_io view count IOs as bytes instead of blocks (Bertrand Drouvot <bertranddrouvot.pg@gmail.com>) |
Список | pgsql-hackers |
Hi, On Thu, 16 Jan 2025 at 10:12, Bertrand Drouvot <bertranddrouvot.pg@gmail.com> wrote: > > Hi, > > On Thu, Jan 16, 2025 at 12:47:17AM -0500, Tom Lane wrote: > > Michael Paquier <michael@paquier.xyz> writes: > > > Not completely sure about the number of parenthesis, but I hope that > > > this should be enough (extra set around io_op): > > > +#define pgstat_is_ioop_tracked_in_bytes(io_op) \ > > > + (((unsigned int) (io_op)) < IOOP_NUM_TYPES && \ > > > + ((unsigned int) (io_op)) >= IOOP_EXTEND) > > > > Yeah, that's safe parenthesis-wise. Whether it'll silence > > the warning from those old clangs remains to be seen. > > Thanks for the report and the proposed "fix". > > From what I can see, the above proposal does (at least) silent the warning > here (clang 5.0.1 and same as demoiselle): https://godbolt.org/z/cGosfzGne (we > can see the warning by using the current define and that the warning is gone > with the new define). Thanks all! I checked clang 4 as well on the link you sent and it also fixes the warning there. -- Regards, Nazir Bilal Yavuz Microsoft
В списке pgsql-hackers по дате отправления: