On Fri, Mar 31, 2017 at 8:05 AM, Tsunakawa, Takayuki <[email protected]> wrote: > From: Amit Kapila [mailto:[email protected]] >> The latest patch looks good to me apart from one Debug message, so I have >> marked it as Ready For Committer. > > Thank you so much! > > >> + ereport(DEBUG1, >> + (errmsg("disabling huge pages"))); >> >> I think this should be similar to what we display in sysv_shmem.c as below: >> >> elog(DEBUG1, "mmap(%zu) with MAP_HUGETLB failed, huge pages disabled: %m", >> allocsize); > > I understood you suggested this to make the reason clear for disabling huge > pages. OK, done as follows. > > + elog(DEBUG1, "CreateFileMapping(%llu) with SEC_LARGE_PAGES > failed " > + "due to insufficient large pages, huge pages disabled", > + size); >
You should use %zu instead of %llu to print Size type of variable. -- With Regards, Amit Kapila. EnterpriseDB: http://www.enterprisedb.com -- Sent via pgsql-hackers mailing list ([email protected]) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
