All of lore.kernel.org
 help / color / mirror / Atom feed
From: gregory.clement@free-electrons.com (Gregory CLEMENT)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH] ARM: compressed: discard ksym/kcrctab input section
Date: Fri, 20 Oct 2017 17:25:56 +0200	[thread overview]
Message-ID: <878tg56dtn.fsf@free-electrons.com> (raw)
In-Reply-To: <CAKv+Gu_+D+d+5Zi8LRmi3_sHEVx-FLLKHsi9aUUmNB8VvcHANQ@mail.gmail.com> (Ard Biesheuvel's message of "Thu, 12 Oct 2017 20:03:53 +0100")

Hi Ard,
 
 On jeu., oct. 12 2017, Ard Biesheuvel <ard.biesheuvel@linaro.org> wrote:

> On 12 October 2017 at 10:45, Russell King - ARM Linux
> <linux@armlinux.org.uk> wrote:
>> On Thu, Oct 12, 2017 at 11:24:57AM +0200, Gregory CLEMENT wrote:
>>> Hi Ard,
>>>
>>> Can we move forward to fix the booting problem ?
>>>
>>> What about amending your commit log with this new information and then
>>> submit it to Russell patch system?
>>
>> Well, I think there's a choice that needs to be made between this
>> approach and Arnd's approach.
>>
>> I'm not all that thrilled with the need to add explicit alignment to
>> data that is inherently a byte stream, and that invariably results in
>> unaligned data words even if you do align the start of it.  That
>> sounds to me very much like a hack rather than a proper solution.
>> So, right now I'm leaning more towards Arnd's solution than Ard's
>> from what's been said in this thread.
>>
>
> I agree that the struct type unaligned accessors are the best choice
> for ARM in any case, given that it will also prevent hitting the
> alignment fixup handler in the kernel unnecessarily.
>
>> However, I don't recall Arnd's patch, it's probably buried deep in
>> my mailbox.
>>
>
> Well, unless you are considering changing the unaligned accessors from
> access_ok.h to le_struct.h as a bugfix, I think we need both patches.

We will soon reach v4.14-rc6 and the Armada XP and Armada 370 still not
boot. I also didn't see your patch in rmk patch system.

Waiting for you find a agreement an other option is to remove CONFIG_EFI
from multi_v7_defconfig as I don't really see any armv7 base board using
EFI.

Thanks,

Gregory

-- 
Gregory Clement, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com

  reply	other threads:[~2017-10-20 15:25 UTC|newest]

Thread overview: 30+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-09-08 15:31 [PATCH] ARM: compressed: discard ksym/kcrctab input section Ard Biesheuvel
2017-09-08 15:39 ` Gregory CLEMENT
2017-09-08 15:41   ` Ard Biesheuvel
2017-10-04 12:16 ` Gregory CLEMENT
2017-10-04 12:20   ` Ard Biesheuvel
2017-10-04 12:43     ` Russell King - ARM Linux
2017-10-09 12:39       ` Ard Biesheuvel
2017-10-12  9:24         ` Gregory CLEMENT
2017-10-12  9:45           ` Russell King - ARM Linux
2017-10-12 19:03             ` Ard Biesheuvel
2017-10-20 15:25               ` Gregory CLEMENT [this message]
2017-10-20 15:28                 ` Ard Biesheuvel
2017-10-20 16:11                   ` Russell King - ARM Linux
2017-10-20 16:20                     ` Ard Biesheuvel
2017-10-20 16:32                       ` Russell King - ARM Linux
2017-10-20 16:36                         ` Ard Biesheuvel
2017-10-20 16:54                           ` Russell King - ARM Linux
2017-10-20 17:12                             ` Ard Biesheuvel
2017-10-21  7:56 ` Matthias Brugger
2017-10-21  8:14   ` Ard Biesheuvel
2017-10-22 22:00     ` Matthias Brugger
2017-10-23  9:29       ` Russell King - ARM Linux
2017-10-23 11:48         ` Russell King - ARM Linux
2017-10-23 21:17           ` Matthias Brugger
2017-10-23 22:19             ` Russell King - ARM Linux
2017-10-24  6:51               ` Matthias Brugger
2017-10-23 21:15         ` Matthias Brugger
2017-10-23 21:32     ` Matthias Brugger
2017-10-23 22:29       ` Russell King - ARM Linux
2017-10-24  8:36       ` Andrea Adami

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=878tg56dtn.fsf@free-electrons.com \
    --to=gregory.clement@free-electrons.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.