* [PATCH] ima-evm-utils: Enable large-file support @ 2019-08-19 20:25 Vitaly Chikunov 2019-08-20 15:19 ` Bruno Meneguele 0 siblings, 1 reply; 4+ messages in thread From: Vitaly Chikunov @ 2019-08-19 20:25 UTC (permalink / raw) To: Mimi Zohar, Dmitry Kasatkin, linux-integrity Some architectures require special measures to access large files (LFS). Add `AC_SYS_LARGEFILE' to `configure.ac' to handle this. It seems that ABI is not changed with this. Signed-off-by: Vitaly Chikunov <vt@altlinux.org> --- configure.ac | 1 + 1 file changed, 1 insertion(+) diff --git a/configure.ac b/configure.ac index 3f21ba4..02bd6f8 100644 --- a/configure.ac +++ b/configure.ac @@ -8,6 +8,7 @@ AC_CONFIG_MACRO_DIR([m4]) AC_CANONICAL_HOST AC_USE_SYSTEM_EXTENSIONS +AC_SYS_LARGEFILE # Checks for programs. AC_PROG_CC -- 2.11.0 ^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] ima-evm-utils: Enable large-file support 2019-08-19 20:25 [PATCH] ima-evm-utils: Enable large-file support Vitaly Chikunov @ 2019-08-20 15:19 ` Bruno Meneguele 2019-08-20 19:42 ` Vitaly Chikunov 0 siblings, 1 reply; 4+ messages in thread From: Bruno Meneguele @ 2019-08-20 15:19 UTC (permalink / raw) To: Vitaly Chikunov; +Cc: Mimi Zohar, Dmitry Kasatkin, linux-integrity [-- Attachment #1: Type: text/plain, Size: 951 bytes --] Hi Vitaly, On Mon, Aug 19, 2019 at 11:25:07PM +0300, Vitaly Chikunov wrote: > Some architectures require special measures to access large files (LFS). > Add `AC_SYS_LARGEFILE' to `configure.ac' to handle this. > > It seems that ABI is not changed with this. > > Signed-off-by: Vitaly Chikunov <vt@altlinux.org> > --- > configure.ac | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/configure.ac b/configure.ac > index 3f21ba4..02bd6f8 100644 > --- a/configure.ac > +++ b/configure.ac > @@ -8,6 +8,7 @@ AC_CONFIG_MACRO_DIR([m4]) > > AC_CANONICAL_HOST > AC_USE_SYSTEM_EXTENSIONS > +AC_SYS_LARGEFILE > Should we also add AC_FUNC_FSEEKO? I can't see any use of fseek or ftell in the code, but if we are enabling AC_SYS_LARGEFILE by default we also should check for AC_FUNC_FSEEKO in order to allow the use of fseeko/ftello whenever needed. > # Checks for programs. > AC_PROG_CC > -- > 2.11.0 > [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 488 bytes --] ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] ima-evm-utils: Enable large-file support 2019-08-20 15:19 ` Bruno Meneguele @ 2019-08-20 19:42 ` Vitaly Chikunov 2019-08-21 12:35 ` Bruno Meneguele 0 siblings, 1 reply; 4+ messages in thread From: Vitaly Chikunov @ 2019-08-20 19:42 UTC (permalink / raw) To: Bruno Meneguele; +Cc: Mimi Zohar, Dmitry Kasatkin, linux-integrity Bruno, On Tue, Aug 20, 2019 at 12:19:39PM -0300, Bruno Meneguele wrote: > On Mon, Aug 19, 2019 at 11:25:07PM +0300, Vitaly Chikunov wrote: > > Some architectures require special measures to access large files (LFS). > > Add `AC_SYS_LARGEFILE' to `configure.ac' to handle this. > > > > It seems that ABI is not changed with this. > > > > Signed-off-by: Vitaly Chikunov <vt@altlinux.org> > > --- > > configure.ac | 1 + > > 1 file changed, 1 insertion(+) > > > > diff --git a/configure.ac b/configure.ac > > index 3f21ba4..02bd6f8 100644 > > --- a/configure.ac > > +++ b/configure.ac > > @@ -8,6 +8,7 @@ AC_CONFIG_MACRO_DIR([m4]) > > > > AC_CANONICAL_HOST > > AC_USE_SYSTEM_EXTENSIONS > > +AC_SYS_LARGEFILE > > > > Should we also add AC_FUNC_FSEEKO? > > I can't see any use of fseek or ftell in the code, but if we are > enabling AC_SYS_LARGEFILE by default we also should check for > AC_FUNC_FSEEKO in order to allow the use of fseeko/ftello whenever > needed. I thought about AC_FUNC_FSEEKO, but we don't use fseeko/ftello, so it didn't look useful. Thanks, > > > # Checks for programs. > > AC_PROG_CC > > -- > > 2.11.0 > > ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] ima-evm-utils: Enable large-file support 2019-08-20 19:42 ` Vitaly Chikunov @ 2019-08-21 12:35 ` Bruno Meneguele 0 siblings, 0 replies; 4+ messages in thread From: Bruno Meneguele @ 2019-08-21 12:35 UTC (permalink / raw) To: Mimi Zohar, Dmitry Kasatkin, linux-integrity [-- Attachment #1: Type: text/plain, Size: 1590 bytes --] On Tue, Aug 20, 2019 at 10:42:36PM +0300, Vitaly Chikunov wrote: > Bruno, > > On Tue, Aug 20, 2019 at 12:19:39PM -0300, Bruno Meneguele wrote: > > On Mon, Aug 19, 2019 at 11:25:07PM +0300, Vitaly Chikunov wrote: > > > Some architectures require special measures to access large files (LFS). > > > Add `AC_SYS_LARGEFILE' to `configure.ac' to handle this. > > > > > > It seems that ABI is not changed with this. > > > > > > Signed-off-by: Vitaly Chikunov <vt@altlinux.org> > > > --- > > > configure.ac | 1 + > > > 1 file changed, 1 insertion(+) > > > > > > diff --git a/configure.ac b/configure.ac > > > index 3f21ba4..02bd6f8 100644 > > > --- a/configure.ac > > > +++ b/configure.ac > > > @@ -8,6 +8,7 @@ AC_CONFIG_MACRO_DIR([m4]) > > > > > > AC_CANONICAL_HOST > > > AC_USE_SYSTEM_EXTENSIONS > > > +AC_SYS_LARGEFILE > > > > > > > Should we also add AC_FUNC_FSEEKO? > > > > I can't see any use of fseek or ftell in the code, but if we are > > enabling AC_SYS_LARGEFILE by default we also should check for > > AC_FUNC_FSEEKO in order to allow the use of fseeko/ftello whenever > > needed. > > I thought about AC_FUNC_FSEEKO, but we don't use fseeko/ftello, so it > didn't look useful. > > Thanks, > That's fine by me :) Just was wondering if we should consider now that for possible use in future. But indeed, that'll be useful only if we would use fseeko/ftello. Thanks. Reviewed-by: Bruno Meneguele <bmeneg@redhat.com> > > > > > > # Checks for programs. > > > AC_PROG_CC > > > -- > > > 2.11.0 > > > > > [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 488 bytes --] ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2019-08-21 12:35 UTC | newest] Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2019-08-19 20:25 [PATCH] ima-evm-utils: Enable large-file support Vitaly Chikunov 2019-08-20 15:19 ` Bruno Meneguele 2019-08-20 19:42 ` Vitaly Chikunov 2019-08-21 12:35 ` Bruno Meneguele
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).