* ima measurement carrying on -mm
@ 2016-09-29 20:44 Thiago Jung Bauermann
2016-09-29 21:02 ` Andrew Morton
0 siblings, 1 reply; 6+ messages in thread
From: Thiago Jung Bauermann @ 2016-09-29 20:44 UTC (permalink / raw)
To: Andrew Morton; +Cc: Stephen Rothwell, Eric Biederman, Mimi Zohar, LKML
Hello Andrew,
You have in the -mm tree a version of the "kexec handover buffer" and "ima
carry measurement list" patches that were NAKed by Eric Biederman. I would
just like to double-check that there's no risk of that version reaching
v4.9.
Mimi posted v5 of a merged patch set that addresses Eric's concern:
https://lists.ozlabs.org/pipermail/linuxppc-dev/2016-September/149183.html
There are no separate kexec handover patches anymore. They were folded into
the series above. The kexec code is simplified now, it doesn't support
updating the buffer and recalculating the hash on reboot, and is now IMA-
specific instead of a generic kexec feature.
--
[]'s
Thiago Jung Bauermann
IBM Linux Technology Center
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: ima measurement carrying on -mm
2016-09-29 20:44 ima measurement carrying on -mm Thiago Jung Bauermann
@ 2016-09-29 21:02 ` Andrew Morton
2016-09-29 21:29 ` Thiago Jung Bauermann
2016-09-29 21:47 ` Eric W. Biederman
0 siblings, 2 replies; 6+ messages in thread
From: Andrew Morton @ 2016-09-29 21:02 UTC (permalink / raw)
To: Thiago Jung Bauermann; +Cc: Stephen Rothwell, Eric Biederman, Mimi Zohar, LKML
On Thu, 29 Sep 2016 17:44:10 -0300 Thiago Jung Bauermann <bauerman@linux.vnet.ibm.com> wrote:
> Hello Andrew,
>
> You have in the -mm tree a version of the "kexec handover buffer" and "ima
> carry measurement list" patches that were NAKed by Eric Biederman. I would
> just like to double-check that there's no risk of that version reaching
> v4.9.
>
> Mimi posted v5 of a merged patch set that addresses Eric's concern:
>
> https://lists.ozlabs.org/pipermail/linuxppc-dev/2016-September/149183.html
>
> There are no separate kexec handover patches anymore. They were folded into
> the series above. The kexec code is simplified now, it doesn't support
> updating the buffer and recalculating the hash on reboot, and is now IMA-
> specific instead of a generic kexec feature.
Yup, thanks.
I wasn't thinking any of this material is suitable for 4.9. Seems that
a bit more consideration will be needed. Am I wrong about that?
Are all of these -mm patches up to date?
kexec_file-allow-arch-specific-memory-walking-for-kexec_add_buffer.patch
kexec_file-change-kexec_add_buffer-to-take-kexec_buf-as-argument.patch
kexec_file-factor-out-kexec_locate_mem_hole-from-kexec_add_buffer.patch
powerpc-change-places-using-config_kexec-to-use-config_kexec_core-instead.patch
powerpc-factor-out-relocation-code-from-module_64c-to-elf_util_64c.patch
powerpc-generalize-elf64_apply_relocate_add.patch
powerpc-adapt-elf64_apply_relocate_add-for-kexec_file_load.patch
powerpc-add-functions-to-read-elf-files-of-any-endianness.patch
powerpc-implement-kexec_file_load.patch
powerpc-add-code-to-work-with-device-trees-in-kexec_file_load.patch
powerpc-add-support-for-loading-elf-kernels-with-kexec_file_load.patch
powerpc-add-support-for-loading-elf-kernels-with-kexec_file_load-fix.patch
powerpc-add-purgatory-for-kexec_file_load-implementation.patch
powerpc-add-purgatory-for-kexec_file_load-implementation-fix.patch
powerpc-enable-config_kexec_file-in-powerpc-server-defconfigs.patch
#
kexec_file-include-the-purgatory-segment-in-the-kexec-image-checksum.patch
kexec_file-add-buffer-hand-over-support-for-the-next-kernel.patch
powerpc-kexec_file-add-buffer-hand-over-support-for-the-next-kernel.patch
kexec_file-add-mechanism-to-update-kexec-segments.patch
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: ima measurement carrying on -mm
2016-09-29 21:02 ` Andrew Morton
@ 2016-09-29 21:29 ` Thiago Jung Bauermann
2016-09-29 21:53 ` Eric W. Biederman
2016-09-29 21:47 ` Eric W. Biederman
1 sibling, 1 reply; 6+ messages in thread
From: Thiago Jung Bauermann @ 2016-09-29 21:29 UTC (permalink / raw)
To: Andrew Morton
Cc: Stephen Rothwell, Eric Biederman, Mimi Zohar, LKML,
Michael Ellerman
Am Donnerstag, 29 September 2016, 14:02:06 schrieb Andrew Morton:
> On Thu, 29 Sep 2016 17:44:10 -0300 Thiago Jung Bauermann
<bauerman@linux.vnet.ibm.com> wrote:
> > Hello Andrew,
> >
> > You have in the -mm tree a version of the "kexec handover buffer" and
> > "ima carry measurement list" patches that were NAKed by Eric Biederman.
> > I would just like to double-check that there's no risk of that version
> > reaching v4.9.
> >
> > Mimi posted v5 of a merged patch set that addresses Eric's concern:
> >
> > https://lists.ozlabs.org/pipermail/linuxppc-dev/2016-September/149183.ht
> > ml
> >
> > There are no separate kexec handover patches anymore. They were folded
> > into the series above. The kexec code is simplified now, it doesn't
> > support updating the buffer and recalculating the hash on reboot, and
> > is now IMA- specific instead of a generic kexec feature.
>
> Yup, thanks.
>
> I wasn't thinking any of this material is suitable for 4.9. Seems that
> a bit more consideration will be needed. Am I wrong about that?
Yes regarding the "ima carry measurement list" patches, but I was hoping
that at least the kexec_file_load patches would be upstreamed.
> Are all of these -mm patches up to date?
>
> kexec_file-allow-arch-specific-memory-walking-for-kexec_add_buffer.patch
> kexec_file-change-kexec_add_buffer-to-take-kexec_buf-as-argument.patch
> kexec_file-factor-out-kexec_locate_mem_hole-from-kexec_add_buffer.patch
> powerpc-change-places-using-config_kexec-to-use-config_kexec_core-instead.
> patch
> powerpc-factor-out-relocation-code-from-module_64c-to-elf_util_64c.patch
> powerpc-generalize-elf64_apply_relocate_add.patch
> powerpc-adapt-elf64_apply_relocate_add-for-kexec_file_load.patch
> powerpc-add-functions-to-read-elf-files-of-any-endianness.patch
> powerpc-implement-kexec_file_load.patch
> powerpc-add-code-to-work-with-device-trees-in-kexec_file_load.patch
> powerpc-add-support-for-loading-elf-kernels-with-kexec_file_load.patch
> powerpc-add-support-for-loading-elf-kernels-with-kexec_file_load-fix.patch
> powerpc-add-purgatory-for-kexec_file_load-implementation.patch
> powerpc-add-purgatory-for-kexec_file_load-implementation-fix.patch
> powerpc-enable-config_kexec_file-in-powerpc-server-defconfigs.patch
Yes, the above are the latest version (v8).
> kexec_file-include-the-purgatory-segment-in-the-kexec-image-checksum.patch
> kexec_file-add-buffer-hand-over-support-for-the-next-kernel.patch
> powerpc-kexec_file-add-buffer-hand-over-support-for-the-next-kernel.patch
> kexec_file-add-mechanism-to-update-kexec-segments.patch
The above were superseded by the version Mimi posted this week I referred
to.
Thanks for the quick response.
--
[]'s
Thiago Jung Bauermann
IBM Linux Technology Center
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: ima measurement carrying on -mm
2016-09-29 21:02 ` Andrew Morton
2016-09-29 21:29 ` Thiago Jung Bauermann
@ 2016-09-29 21:47 ` Eric W. Biederman
1 sibling, 0 replies; 6+ messages in thread
From: Eric W. Biederman @ 2016-09-29 21:47 UTC (permalink / raw)
To: Andrew Morton; +Cc: Thiago Jung Bauermann, Stephen Rothwell, Mimi Zohar, LKML
Andrew Morton <akpm@linux-foundation.org> writes:
> On Thu, 29 Sep 2016 17:44:10 -0300 Thiago Jung Bauermann <bauerman@linux.vnet.ibm.com> wrote:
>
>> Hello Andrew,
>>
>> You have in the -mm tree a version of the "kexec handover buffer" and "ima
>> carry measurement list" patches that were NAKed by Eric Biederman. I would
>> just like to double-check that there's no risk of that version reaching
>> v4.9.
>>
>> Mimi posted v5 of a merged patch set that addresses Eric's concern:
>>
>> https://lists.ozlabs.org/pipermail/linuxppc-dev/2016-September/149183.html
>>
>> There are no separate kexec handover patches anymore. They were folded into
>> the series above. The kexec code is simplified now, it doesn't support
>> updating the buffer and recalculating the hash on reboot, and is now IMA-
>> specific instead of a generic kexec feature.
>
> Yup, thanks.
>
> I wasn't thinking any of this material is suitable for 4.9. Seems that
> a bit more consideration will be needed. Am I wrong about that?
With Mimi's v5 version of these patches the kexec portions look reasonable
and the rest is pretty much just ima code and a little bit of powerpc
specific glue (as this is only currently supported for the powerpc
way of passing boot parameters).
> Are all of these -mm patches up to date?
No. There was a v5 that mimi posted. I think the patches listed below are
obsolete at this point.
> kexec_file-allow-arch-specific-memory-walking-for-kexec_add_buffer.patch
> kexec_file-change-kexec_add_buffer-to-take-kexec_buf-as-argument.patch
> kexec_file-factor-out-kexec_locate_mem_hole-from-kexec_add_buffer.patch
> powerpc-change-places-using-config_kexec-to-use-config_kexec_core-instead.patch
> powerpc-factor-out-relocation-code-from-module_64c-to-elf_util_64c.patch
> powerpc-generalize-elf64_apply_relocate_add.patch
> powerpc-adapt-elf64_apply_relocate_add-for-kexec_file_load.patch
> powerpc-add-functions-to-read-elf-files-of-any-endianness.patch
> powerpc-implement-kexec_file_load.patch
> powerpc-add-code-to-work-with-device-trees-in-kexec_file_load.patch
> powerpc-add-support-for-loading-elf-kernels-with-kexec_file_load.patch
> powerpc-add-support-for-loading-elf-kernels-with-kexec_file_load-fix.patch
> powerpc-add-purgatory-for-kexec_file_load-implementation.patch
> powerpc-add-purgatory-for-kexec_file_load-implementation-fix.patch
> powerpc-enable-config_kexec_file-in-powerpc-server-defconfigs.patch
> #
> kexec_file-include-the-purgatory-segment-in-the-kexec-image-checksum.patch
> kexec_file-add-buffer-hand-over-support-for-the-next-kernel.patch
> powerpc-kexec_file-add-buffer-hand-over-support-for-the-next-kernel.patch
> kexec_file-add-mechanism-to-update-kexec-segments.patch
Eric
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: ima measurement carrying on -mm
2016-09-29 21:29 ` Thiago Jung Bauermann
@ 2016-09-29 21:53 ` Eric W. Biederman
2016-09-29 22:16 ` Thiago Jung Bauermann
0 siblings, 1 reply; 6+ messages in thread
From: Eric W. Biederman @ 2016-09-29 21:53 UTC (permalink / raw)
To: Thiago Jung Bauermann
Cc: Andrew Morton, Stephen Rothwell, Mimi Zohar, LKML,
Michael Ellerman
Thiago Jung Bauermann <bauerman@linux.vnet.ibm.com> writes:
> Am Donnerstag, 29 September 2016, 14:02:06 schrieb Andrew Morton:
>> On Thu, 29 Sep 2016 17:44:10 -0300 Thiago Jung Bauermann
> <bauerman@linux.vnet.ibm.com> wrote:
>> > Hello Andrew,
>> >
>> > You have in the -mm tree a version of the "kexec handover buffer" and
>> > "ima carry measurement list" patches that were NAKed by Eric Biederman.
>> > I would just like to double-check that there's no risk of that version
>> > reaching v4.9.
>> >
>> > Mimi posted v5 of a merged patch set that addresses Eric's concern:
>> >
>> > https://lists.ozlabs.org/pipermail/linuxppc-dev/2016-September/149183.ht
>> > ml
>> >
>> > There are no separate kexec handover patches anymore. They were folded
>> > into the series above. The kexec code is simplified now, it doesn't
>> > support updating the buffer and recalculating the hash on reboot, and
>> > is now IMA- specific instead of a generic kexec feature.
>>
>> Yup, thanks.
>>
>> I wasn't thinking any of this material is suitable for 4.9. Seems that
>> a bit more consideration will be needed. Am I wrong about that?
>
> Yes regarding the "ima carry measurement list" patches, but I was hoping
> that at least the kexec_file_load patches would be upstreamed.
Oh bah. I was confused about that straight forward adding of kexec_file
support to powerpc. I thought that was already in existence.
In that case let me say I am concerned about modifying the flattened
device tree, especially in the kexec_file. I would think that the
flattened device tree would be something that it would be desirable to
keep intact.
I know in the x86 boot protocol we have some variables that are purely
passed by the bootloader (like the command line) and some that just
representations of firmware provided information. Does powerpc not have
that separation.
I would think being able to pass the flattened device tree through
unchanged would be very desirable in the kexec case as it removes the
possibility of error.
>> Are all of these -mm patches up to date?
>>
>> kexec_file-allow-arch-specific-memory-walking-for-kexec_add_buffer.patch
>> kexec_file-change-kexec_add_buffer-to-take-kexec_buf-as-argument.patch
>> kexec_file-factor-out-kexec_locate_mem_hole-from-kexec_add_buffer.patch
>> powerpc-change-places-using-config_kexec-to-use-config_kexec_core-instead.
>> patch
>> powerpc-factor-out-relocation-code-from-module_64c-to-elf_util_64c.patch
>> powerpc-generalize-elf64_apply_relocate_add.patch
>> powerpc-adapt-elf64_apply_relocate_add-for-kexec_file_load.patch
>> powerpc-add-functions-to-read-elf-files-of-any-endianness.patch
>> powerpc-implement-kexec_file_load.patch
>> powerpc-add-code-to-work-with-device-trees-in-kexec_file_load.patch
>> powerpc-add-support-for-loading-elf-kernels-with-kexec_file_load.patch
>> powerpc-add-support-for-loading-elf-kernels-with-kexec_file_load-fix.patch
>> powerpc-add-purgatory-for-kexec_file_load-implementation.patch
>> powerpc-add-purgatory-for-kexec_file_load-implementation-fix.patch
>> powerpc-enable-config_kexec_file-in-powerpc-server-defconfigs.patch
>
> Yes, the above are the latest version (v8).
Eric
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: ima measurement carrying on -mm
2016-09-29 21:53 ` Eric W. Biederman
@ 2016-09-29 22:16 ` Thiago Jung Bauermann
0 siblings, 0 replies; 6+ messages in thread
From: Thiago Jung Bauermann @ 2016-09-29 22:16 UTC (permalink / raw)
To: Eric W. Biederman
Cc: Andrew Morton, Stephen Rothwell, Mimi Zohar, LKML,
Michael Ellerman
Am Donnerstag, 29 September 2016, 16:53:50 schrieb Eric W. Biederman:
> Thiago Jung Bauermann <bauerman@linux.vnet.ibm.com> writes:
> > Am Donnerstag, 29 September 2016, 14:02:06 schrieb Andrew Morton:
> >> On Thu, 29 Sep 2016 17:44:10 -0300 Thiago Jung Bauermann
> >
> > <bauerman@linux.vnet.ibm.com> wrote:
> >> > Hello Andrew,
> >> >
> >> > You have in the -mm tree a version of the "kexec handover buffer" and
> >> > "ima carry measurement list" patches that were NAKed by Eric
> >> > Biederman.
> >> > I would just like to double-check that there's no risk of that
> >> > version
> >> > reaching v4.9.
> >> >
> >> > Mimi posted v5 of a merged patch set that addresses Eric's concern:
> >> >
> >> > https://lists.ozlabs.org/pipermail/linuxppc-dev/2016-September/149183
> >> > .ht
> >> > ml
> >> >
> >> > There are no separate kexec handover patches anymore. They were
> >> > folded
> >> > into the series above. The kexec code is simplified now, it doesn't
> >> > support updating the buffer and recalculating the hash on reboot, and
> >> > is now IMA- specific instead of a generic kexec feature.
> >>
> >> Yup, thanks.
> >>
> >> I wasn't thinking any of this material is suitable for 4.9. Seems that
> >> a bit more consideration will be needed. Am I wrong about that?
> >
> > Yes regarding the "ima carry measurement list" patches, but I was hoping
> > that at least the kexec_file_load patches would be upstreamed.
>
> Oh bah. I was confused about that straight forward adding of kexec_file
> support to powerpc. I thought that was already in existence.
>
> In that case let me say I am concerned about modifying the flattened
> device tree, especially in the kexec_file. I would think that the
> flattened device tree would be something that it would be desirable to
> keep intact.
>
> I know in the x86 boot protocol we have some variables that are purely
> passed by the bootloader (like the command line) and some that just
> representations of firmware provided information. Does powerpc not have
> that separation.
>
> I would think being able to pass the flattened device tree through
> unchanged would be very desirable in the kexec case as it removes the
> possibility of error.
As far as I know, that is not possible. The device tree always needs to be
modified to add or update the properties that indicate where the initrd is
loaded and, as you mentioned, the kernel command line. The IMA buffer
patches just adds another property.
--
[]'s
Thiago Jung Bauermann
IBM Linux Technology Center
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2016-09-29 22:16 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-09-29 20:44 ima measurement carrying on -mm Thiago Jung Bauermann
2016-09-29 21:02 ` Andrew Morton
2016-09-29 21:29 ` Thiago Jung Bauermann
2016-09-29 21:53 ` Eric W. Biederman
2016-09-29 22:16 ` Thiago Jung Bauermann
2016-09-29 21:47 ` Eric W. Biederman
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox