Remove PrintBufferDescs() and PrintPinnedBufs().
authorTom Lane <[email protected]>
Sun, 19 Jan 2025 19:00:22 +0000 (14:00 -0500)
committerTom Lane <[email protected]>
Sun, 19 Jan 2025 19:00:22 +0000 (14:00 -0500)
These have been #ifdef'd out for a long time, and in fact have
been uncompilable since commit 48354581a of 2016-04-10.  The
fact that nobody noticed for so long demonstrates their lack of
usefulness, so let's remove them rather than fix them.

Author: Jacob Brazeal <[email protected]>
Discussion: https://postgr.es/m/CA+COZaB+9CN_f63PPRoVhHjYmCwwmb_9CWLxqCJdMWDqs1a-JA@mail.gmail.com

src/backend/storage/buffer/bufmgr.c
src/include/storage/bufmgr.h

index 739daa1153a64080203df26849a99bf53f6eac86..0d8849bf894fb25ff84fb7fb96b94221b82d8a60 100644 (file)
@@ -4412,64 +4412,6 @@ DropDatabaseBuffers(Oid dbid)
    }
 }
 
-/* -----------------------------------------------------------------
- *     PrintBufferDescs
- *
- *     this function prints all the buffer descriptors, for debugging
- *     use only.
- * -----------------------------------------------------------------
- */
-#ifdef NOT_USED
-void
-PrintBufferDescs(void)
-{
-   int         i;
-
-   for (i = 0; i < NBuffers; ++i)
-   {
-       BufferDesc *buf = GetBufferDescriptor(i);
-       Buffer      b = BufferDescriptorGetBuffer(buf);
-
-       /* theoretically we should lock the bufhdr here */
-       elog(LOG,
-            "[%02d] (freeNext=%d, rel=%s, "
-            "blockNum=%u, flags=0x%x, refcount=%u %d)",
-            i, buf->freeNext,
-            relpathbackend(BufTagGetRelFileLocator(&buf->tag),
-                           INVALID_PROC_NUMBER, BufTagGetForkNum(&buf->tag)),
-            buf->tag.blockNum, buf->flags,
-            buf->refcount, GetPrivateRefCount(b));
-   }
-}
-#endif
-
-#ifdef NOT_USED
-void
-PrintPinnedBufs(void)
-{
-   int         i;
-
-   for (i = 0; i < NBuffers; ++i)
-   {
-       BufferDesc *buf = GetBufferDescriptor(i);
-       Buffer      b = BufferDescriptorGetBuffer(buf);
-
-       if (GetPrivateRefCount(b) > 0)
-       {
-           /* theoretically we should lock the bufhdr here */
-           elog(LOG,
-                "[%02d] (freeNext=%d, rel=%s, "
-                "blockNum=%u, flags=0x%x, refcount=%u %d)",
-                i, buf->freeNext,
-                relpathperm(BufTagGetRelFileLocator(&buf->tag),
-                            BufTagGetForkNum(&buf->tag)),
-                buf->tag.blockNum, buf->flags,
-                buf->refcount, GetPrivateRefCount(b));
-       }
-   }
-}
-#endif
-
 /* ---------------------------------------------------------------------
  *     FlushRelationBuffers
  *
index 3fdd29bd0b6eae61e40e61621ab3e167853d0029..46b4e0d90f37b3636824481034011844edadd4be 100644 (file)
@@ -275,10 +275,6 @@ extern void DropDatabaseBuffers(Oid dbid);
 
 extern bool BufferIsPermanent(Buffer buffer);
 extern XLogRecPtr BufferGetLSNAtomic(Buffer buffer);
-
-#ifdef NOT_USED
-extern void PrintPinnedBufs(void);
-#endif
 extern void BufferGetTag(Buffer buffer, RelFileLocator *rlocator,
                         ForkNumber *forknum, BlockNumber *blknum);