With commit 987f20a9dcce ("a.out: Remove the a.out implementation"), the
use of the special taso flag for alpha architectures in the linux_binprm
struct is gone.
Remove the definition of taso in the linux_binprm struct.
No functional change.
Signed-off-by: Lukas Bulwahn <[email protected]>
---
Kees, please pick this patch on top of the commit above.
include/linux/binfmts.h | 3 ---
1 file changed, 3 deletions(-)
diff --git a/include/linux/binfmts.h b/include/linux/binfmts.h
index 3dc20c4f394c..8d51f69f9f5e 100644
--- a/include/linux/binfmts.h
+++ b/include/linux/binfmts.h
@@ -43,9 +43,6 @@ struct linux_binprm {
* original userspace.
*/
point_of_no_return:1;
-#ifdef __alpha__
- unsigned int taso:1;
-#endif
struct file *executable; /* Executable to pass to the interpreter */
struct file *interpreter;
struct file *file;
--
2.17.1
Lukas Bulwahn <[email protected]> writes:
> With commit 987f20a9dcce ("a.out: Remove the a.out implementation"), the
> use of the special taso flag for alpha architectures in the linux_binprm
> struct is gone.
>
> Remove the definition of taso in the linux_binprm struct.
>
> No functional change.
Reviewed-by: "Eric W. Biederman" <[email protected]>
Alphas binfmt_loader is the only use I can find of that variable
so let's kill it as well.
> Signed-off-by: Lukas Bulwahn <[email protected]>
> ---
> Kees, please pick this patch on top of the commit above.
>
> include/linux/binfmts.h | 3 ---
> 1 file changed, 3 deletions(-)
>
> diff --git a/include/linux/binfmts.h b/include/linux/binfmts.h
> index 3dc20c4f394c..8d51f69f9f5e 100644
> --- a/include/linux/binfmts.h
> +++ b/include/linux/binfmts.h
> @@ -43,9 +43,6 @@ struct linux_binprm {
> * original userspace.
> */
> point_of_no_return:1;
> -#ifdef __alpha__
> - unsigned int taso:1;
> -#endif
> struct file *executable; /* Executable to pass to the interpreter */
> struct file *interpreter;
> struct file *file;
On Thu, Sep 29, 2022 at 05:17:28PM -0500, Eric W. Biederman wrote:
> Lukas Bulwahn <[email protected]> writes:
>
> > With commit 987f20a9dcce ("a.out: Remove the a.out implementation"), the
> > use of the special taso flag for alpha architectures in the linux_binprm
> > struct is gone.
> >
> > Remove the definition of taso in the linux_binprm struct.
> >
> > No functional change.
>
> Reviewed-by: "Eric W. Biederman" <[email protected]>
>
> Alphas binfmt_loader is the only use I can find of that variable
> so let's kill it as well.
I can't find anything using "taso". Where are you looking?
--
Kees Cook
On Thu, 29 Sep 2022 22:39:03 +0200, Lukas Bulwahn wrote:
> With commit 987f20a9dcce ("a.out: Remove the a.out implementation"), the
> use of the special taso flag for alpha architectures in the linux_binprm
> struct is gone.
>
> Remove the definition of taso in the linux_binprm struct.
>
> No functional change.
>
> [...]
Applied to for-next/execve, thanks!
[1/1] binfmt: remove taso from linux_binprm struct
https://git.kernel.org/kees/c/9f4beead610c
--
Kees Cook
On Thu, Sep 29, 2022 at 05:17:28PM -0500, Eric W. Biederman wrote:
> Lukas Bulwahn <[email protected]> writes:
>
> > With commit 987f20a9dcce ("a.out: Remove the a.out implementation"), the
> > use of the special taso flag for alpha architectures in the linux_binprm
> > struct is gone.
> >
> > Remove the definition of taso in the linux_binprm struct.
> >
> > No functional change.
>
> Reviewed-by: "Eric W. Biederman" <[email protected]>
>
> Alphas binfmt_loader is the only use I can find of that variable
> so let's kill it as well.
Ah, sorry, misparsed this -- you mean, alpha's use (now removed) was the
only place it was accessed. Agreed. :)
--
Kees Cook
Kees Cook <[email protected]> writes:
> On Thu, Sep 29, 2022 at 05:17:28PM -0500, Eric W. Biederman wrote:
>> Lukas Bulwahn <[email protected]> writes:
>>
>> > With commit 987f20a9dcce ("a.out: Remove the a.out implementation"), the
>> > use of the special taso flag for alpha architectures in the linux_binprm
>> > struct is gone.
>> >
>> > Remove the definition of taso in the linux_binprm struct.
>> >
>> > No functional change.
>>
>> Reviewed-by: "Eric W. Biederman" <[email protected]>
>>
>> Alphas binfmt_loader is the only use I can find of that variable
>> so let's kill it as well.
>
> Ah, sorry, misparsed this -- you mean, alpha's use (now removed) was the
> only place it was accessed. Agreed. :)
Yes. I was looking in a tree without my previous change merged.
The code in binfmt_loader (which is removed in the my change) was the
only user of that variable I could find.
Eric