From: Vyacheslav Dubeyko <slava@dubeyko.com>
To: Andrew Morton <akpm@linux-foundation.org>
Cc: Linux FS devel list <linux-fsdevel@vger.kernel.org>,
Al Viro <viro@zeniv.linux.org.uk>,
ChristophHellwig <hch@infradead.org>,
Hin-Tak Leung <htl10@users.sourceforge.net>
Subject: Re: [PATCH v2 00/15] hfsplus: introduce journal replay functionality
Date: Thu, 06 Feb 2014 10:21:19 +0400 [thread overview]
Message-ID: <1391667679.15555.3.camel@ubuntu> (raw)
In-Reply-To: <20140205154833.be220231580431b148c370f3@linux-foundation.org>
On Wed, 2014-02-05 at 15:48 -0800, Andrew Morton wrote:
> On Fri, 24 Jan 2014 19:25:18 +0400 Vyacheslav Dubeyko <slava@dubeyko.com> wrote:
>
> > This second version of patchset that implements journal replay
> > functionality in HFS+ file system driver.
>
> The series introduces a lot of build glitches on i386 (and other 32-bit
> builds)
>
> - tons of printk mismatch warnings
>
> - unresolvable references to __udivdi3 and __umoddi3 (need to use
> do_div() or similar).
>
> - several bisection holes:
>
> - with 4 patches applied, error: implicit declaration of function 'JHDR_SIZE'
>
> - with three patches applied, warning: 'hfsplus_create_journal' used but never defined
>
> and probably others.
>
> I'll drop the series for now. Please, find an i386 compiler and ensure
> that we get a clean build at every point of the patch series.
>
>
> To find the __udivdi3 thing I do something like
>
> make M=fs/hfsplus
> grep __udivdi3 fs/hfsplus/*.o
>
> then for the offending .o file
>
> make fs/hfsplus/foo.s (or foo.lst)
>
> then poke around in the assembly code to locate the offending function.
Ok. Thank you. I'll check patchset building on i386 platform and I'll
fix all found errors.
Thanks,
Vyacheslav Dubeyko.
next prev parent reply other threads:[~2014-02-06 6:21 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-01-24 15:25 [PATCH v2 00/15] hfsplus: introduce journal replay functionality Vyacheslav Dubeyko
2014-01-27 4:22 ` Hin-Tak Leung
2014-01-27 13:25 ` Vyacheslav Dubeyko
2014-02-05 13:16 ` Hin-Tak Leung
2014-02-05 23:48 ` Andrew Morton
2014-02-06 6:21 ` Vyacheslav Dubeyko [this message]
2014-02-07 6:05 ` Vyacheslav Dubeyko
2014-02-07 6:23 ` Andrew Morton
2014-02-11 7:29 ` Hin-Tak Leung
2014-02-11 8:02 ` Vyacheslav Dubeyko
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1391667679.15555.3.camel@ubuntu \
--to=slava@dubeyko.com \
--cc=akpm@linux-foundation.org \
--cc=hch@infradead.org \
--cc=htl10@users.sourceforge.net \
--cc=linux-fsdevel@vger.kernel.org \
--cc=viro@zeniv.linux.org.uk \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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).