* [PATCH] erofs-utils: fix multiple definition of `sbi'
[not found] <20201208105741.9614-1-hsiangkao.ref@aol.com>
@ 2020-12-08 10:57 ` Gao Xiang via Linux-erofs
2020-12-08 16:16 ` nl6720
0 siblings, 1 reply; 4+ messages in thread
From: Gao Xiang via Linux-erofs @ 2020-12-08 10:57 UTC (permalink / raw)
To: linux-erofs, nl6720
As nl6720 reported [1], lib/inode.o (mkfs) and lib/super.o (erofsfuse)
could be compiled together by some options. Fix it now.
[1] https://lore.kernel.org/r/10789285.Na0ui7I3VY@walnut
Reported-by: nl6720 <nl6720@gmail.com>
Signed-off-by: Gao Xiang <hsiangkao@aol.com>
---
Hi nl6720,
could you verify this patch? Thanks in advance!
Thanks,
Gao Xiang
lib/config.c | 1 +
lib/inode.c | 2 --
lib/super.c | 2 --
3 files changed, 1 insertion(+), 4 deletions(-)
diff --git a/lib/config.c b/lib/config.c
index 315511284871..3ecd48140cfd 100644
--- a/lib/config.c
+++ b/lib/config.c
@@ -11,6 +11,7 @@
#include "erofs/internal.h"
struct erofs_configure cfg;
+struct erofs_sb_info sbi;
void erofs_init_configure(void)
{
diff --git a/lib/inode.c b/lib/inode.c
index 3d634fc92852..0c4839dc7152 100644
--- a/lib/inode.c
+++ b/lib/inode.c
@@ -22,8 +22,6 @@
#include "erofs/xattr.h"
#include "erofs/exclude.h"
-struct erofs_sb_info sbi;
-
#define S_SHIFT 12
static unsigned char erofs_ftype_by_mode[S_IFMT >> S_SHIFT] = {
[S_IFREG >> S_SHIFT] = EROFS_FT_REG_FILE,
diff --git a/lib/super.c b/lib/super.c
index 2d366928e12b..025cefee3aac 100644
--- a/lib/super.c
+++ b/lib/super.c
@@ -11,8 +11,6 @@
#include "erofs/io.h"
#include "erofs/print.h"
-struct erofs_sb_info sbi;
-
static bool check_layout_compatibility(struct erofs_sb_info *sbi,
struct erofs_super_block *dsb)
{
--
2.24.0
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] erofs-utils: fix multiple definition of `sbi'
2020-12-08 10:57 ` [PATCH] erofs-utils: fix multiple definition of `sbi' Gao Xiang via Linux-erofs
@ 2020-12-08 16:16 ` nl6720
2020-12-08 23:24 ` Gao Xiang
0 siblings, 1 reply; 4+ messages in thread
From: nl6720 @ 2020-12-08 16:16 UTC (permalink / raw)
To: Gao Xiang, linux-erofs
On Tuesday, 8 December 2020 12:57:41 EET Gao Xiang wrote:
> As nl6720 reported [1], lib/inode.o (mkfs) and lib/super.o (erofsfuse)
> could be compiled together by some options. Fix it now.
>
> [1] https://lore.kernel.org/r/10789285.Na0ui7I3VY@walnut
> Reported-by: nl6720 <nl6720@gmail.com>
> Signed-off-by: Gao Xiang <hsiangkao@aol.com>
> ---
> Hi nl6720,
> could you verify this patch? Thanks in advance!
>
> Thanks,
> Gao Xiang
This patch fixes the build issue for me. Thanks!
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] erofs-utils: fix multiple definition of `sbi'
2020-12-08 16:16 ` nl6720
@ 2020-12-08 23:24 ` Gao Xiang
2020-12-12 9:54 ` Li GuiFu via Linux-erofs
0 siblings, 1 reply; 4+ messages in thread
From: Gao Xiang @ 2020-12-08 23:24 UTC (permalink / raw)
To: nl6720; +Cc: linux-erofs
On Tue, Dec 08, 2020 at 06:16:57PM +0200, nl6720 wrote:
> On Tuesday, 8 December 2020 12:57:41 EET Gao Xiang wrote:
> > As nl6720 reported [1], lib/inode.o (mkfs) and lib/super.o (erofsfuse)
> > could be compiled together by some options. Fix it now.
> >
> > [1] https://lore.kernel.org/r/10789285.Na0ui7I3VY@walnut
> > Reported-by: nl6720 <nl6720@gmail.com>
> > Signed-off-by: Gao Xiang <hsiangkao@aol.com>
> > ---
> > Hi nl6720,
> > could you verify this patch? Thanks in advance!
> >
> > Thanks,
> > Gao Xiang
>
> This patch fixes the build issue for me. Thanks!
Thanks for verification, I'll push out this later...
Fixes: 5e35b75ad499 ("erofs-utils: introduce fuse implementation")
Thanks,
Gao Xiang
>
>
>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] erofs-utils: fix multiple definition of `sbi'
2020-12-08 23:24 ` Gao Xiang
@ 2020-12-12 9:54 ` Li GuiFu via Linux-erofs
0 siblings, 0 replies; 4+ messages in thread
From: Li GuiFu via Linux-erofs @ 2020-12-12 9:54 UTC (permalink / raw)
To: Gao Xiang, nl6720; +Cc: linux-erofs
On 2020/12/9 7:24, Gao Xiang wrote:
> On Tue, Dec 08, 2020 at 06:16:57PM +0200, nl6720 wrote:
>> On Tuesday, 8 December 2020 12:57:41 EET Gao Xiang wrote:
>>> As nl6720 reported [1], lib/inode.o (mkfs) and lib/super.o (erofsfuse)
>>> could be compiled together by some options. Fix it now.
>>>
>>> [1] https://lore.kernel.org/r/10789285.Na0ui7I3VY@walnut
>>> Reported-by: nl6720 <nl6720@gmail.com>
>>> Signed-off-by: Gao Xiang <hsiangkao@aol.com>
>>> ---
It looks good
Reviewed-by: Li Guifu <bluce.lee@aliyun.com>
Thanks,
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2020-12-12 9:54 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <20201208105741.9614-1-hsiangkao.ref@aol.com>
2020-12-08 10:57 ` [PATCH] erofs-utils: fix multiple definition of `sbi' Gao Xiang via Linux-erofs
2020-12-08 16:16 ` nl6720
2020-12-08 23:24 ` Gao Xiang
2020-12-12 9:54 ` Li GuiFu via Linux-erofs
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).