* linux-next: manual merge of the block tree with the arm64 tree
@ 2020-12-03 3:25 Stephen Rothwell
2020-12-03 11:01 ` Catalin Marinas
2020-12-14 20:17 ` Stephen Rothwell
0 siblings, 2 replies; 6+ messages in thread
From: Stephen Rothwell @ 2020-12-03 3:25 UTC (permalink / raw)
To: Jens Axboe, Catalin Marinas, Will Deacon
Cc: Linux Kernel Mailing List, Linux Next Mailing List, Mark Rutland
[-- Attachment #1: Type: text/plain, Size: 2177 bytes --]
Hi all,
Today's linux-next merge of the block tree got a conflict in:
arch/arm64/include/asm/thread_info.h
between commit:
b5a5a01d8e9a ("arm64: uaccess: remove addr_limit_user_check()")
from the arm64 tree and commit:
192caabd4dd9 ("arm64: add support for TIF_NOTIFY_SIGNAL")
from the block tree.
I fixed it up (see below) and can carry the fix as necessary. This
is now fixed as far as linux-next is concerned, but any non trivial
conflicts should be mentioned to your upstream maintainer when your tree
is submitted for merging. You may also want to consider cooperating
with the maintainer of the conflicting tree to minimise any particularly
complex conflicts.
--
Cheers,
Stephen Rothwell
diff --cc arch/arm64/include/asm/thread_info.h
index 015beafe58f5,cdcf307764aa..000000000000
--- a/arch/arm64/include/asm/thread_info.h
+++ b/arch/arm64/include/asm/thread_info.h
@@@ -63,7 -66,9 +63,8 @@@ void arch_release_task_struct(struct ta
#define TIF_NOTIFY_RESUME 2 /* callback before returning to user */
#define TIF_FOREIGN_FPSTATE 3 /* CPU's FP state is not current's */
#define TIF_UPROBE 4 /* uprobe breakpoint or singlestep */
- #define TIF_MTE_ASYNC_FAULT 5 /* MTE Asynchronous Tag Check Fault */
-#define TIF_FSCHECK 5 /* Check FS is USER_DS on return */
++#define TIF_NOTIFY_SIGNAL 5 /* signal notifications exist */
+ #define TIF_MTE_ASYNC_FAULT 6 /* MTE Asynchronous Tag Check Fault */
-#define TIF_NOTIFY_SIGNAL 7 /* signal notifications exist */
#define TIF_SYSCALL_TRACE 8 /* syscall trace active */
#define TIF_SYSCALL_AUDIT 9 /* syscall auditing */
#define TIF_SYSCALL_TRACEPOINT 10 /* syscall tracepoint for ftrace */
@@@ -96,7 -103,8 +98,8 @@@
#define _TIF_WORK_MASK (_TIF_NEED_RESCHED | _TIF_SIGPENDING | \
_TIF_NOTIFY_RESUME | _TIF_FOREIGN_FPSTATE | \
- _TIF_UPROBE | _TIF_MTE_ASYNC_FAULT)
- _TIF_UPROBE | _TIF_FSCHECK | _TIF_MTE_ASYNC_FAULT | \
++ _TIF_UPROBE | _TIF_MTE_ASYNC_FAULT | \
+ _TIF_NOTIFY_SIGNAL)
#define _TIF_SYSCALL_WORK (_TIF_SYSCALL_TRACE | _TIF_SYSCALL_AUDIT | \
_TIF_SYSCALL_TRACEPOINT | _TIF_SECCOMP | \
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: linux-next: manual merge of the block tree with the arm64 tree
2020-12-03 3:25 linux-next: manual merge of the block tree with the arm64 tree Stephen Rothwell
@ 2020-12-03 11:01 ` Catalin Marinas
2020-12-03 14:36 ` Jens Axboe
2020-12-14 20:17 ` Stephen Rothwell
1 sibling, 1 reply; 6+ messages in thread
From: Catalin Marinas @ 2020-12-03 11:01 UTC (permalink / raw)
To: Stephen Rothwell
Cc: Jens Axboe, Will Deacon, Linux Kernel Mailing List,
Linux Next Mailing List, Mark Rutland
On Thu, Dec 03, 2020 at 02:25:30PM +1100, Stephen Rothwell wrote:
> diff --cc arch/arm64/include/asm/thread_info.h
> index 015beafe58f5,cdcf307764aa..000000000000
> --- a/arch/arm64/include/asm/thread_info.h
> +++ b/arch/arm64/include/asm/thread_info.h
> @@@ -63,7 -66,9 +63,8 @@@ void arch_release_task_struct(struct ta
> #define TIF_NOTIFY_RESUME 2 /* callback before returning to user */
> #define TIF_FOREIGN_FPSTATE 3 /* CPU's FP state is not current's */
> #define TIF_UPROBE 4 /* uprobe breakpoint or singlestep */
> - #define TIF_MTE_ASYNC_FAULT 5 /* MTE Asynchronous Tag Check Fault */
> -#define TIF_FSCHECK 5 /* Check FS is USER_DS on return */
> ++#define TIF_NOTIFY_SIGNAL 5 /* signal notifications exist */
> + #define TIF_MTE_ASYNC_FAULT 6 /* MTE Asynchronous Tag Check Fault */
> -#define TIF_NOTIFY_SIGNAL 7 /* signal notifications exist */
> #define TIF_SYSCALL_TRACE 8 /* syscall trace active */
> #define TIF_SYSCALL_AUDIT 9 /* syscall auditing */
> #define TIF_SYSCALL_TRACEPOINT 10 /* syscall tracepoint for ftrace */
> @@@ -96,7 -103,8 +98,8 @@@
>
> #define _TIF_WORK_MASK (_TIF_NEED_RESCHED | _TIF_SIGPENDING | \
> _TIF_NOTIFY_RESUME | _TIF_FOREIGN_FPSTATE | \
> - _TIF_UPROBE | _TIF_MTE_ASYNC_FAULT)
> - _TIF_UPROBE | _TIF_FSCHECK | _TIF_MTE_ASYNC_FAULT | \
> ++ _TIF_UPROBE | _TIF_MTE_ASYNC_FAULT | \
> + _TIF_NOTIFY_SIGNAL)
Thanks Stephen. It looks alright to me.
--
Catalin
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: linux-next: manual merge of the block tree with the arm64 tree
2020-12-03 11:01 ` Catalin Marinas
@ 2020-12-03 14:36 ` Jens Axboe
2020-12-03 15:05 ` Catalin Marinas
0 siblings, 1 reply; 6+ messages in thread
From: Jens Axboe @ 2020-12-03 14:36 UTC (permalink / raw)
To: Catalin Marinas, Stephen Rothwell
Cc: Will Deacon, Linux Kernel Mailing List, Linux Next Mailing List,
Mark Rutland
On 12/3/20 4:01 AM, Catalin Marinas wrote:
> On Thu, Dec 03, 2020 at 02:25:30PM +1100, Stephen Rothwell wrote:
>> diff --cc arch/arm64/include/asm/thread_info.h
>> index 015beafe58f5,cdcf307764aa..000000000000
>> --- a/arch/arm64/include/asm/thread_info.h
>> +++ b/arch/arm64/include/asm/thread_info.h
>> @@@ -63,7 -66,9 +63,8 @@@ void arch_release_task_struct(struct ta
>> #define TIF_NOTIFY_RESUME 2 /* callback before returning to user */
>> #define TIF_FOREIGN_FPSTATE 3 /* CPU's FP state is not current's */
>> #define TIF_UPROBE 4 /* uprobe breakpoint or singlestep */
>> - #define TIF_MTE_ASYNC_FAULT 5 /* MTE Asynchronous Tag Check Fault */
>> -#define TIF_FSCHECK 5 /* Check FS is USER_DS on return */
>> ++#define TIF_NOTIFY_SIGNAL 5 /* signal notifications exist */
>> + #define TIF_MTE_ASYNC_FAULT 6 /* MTE Asynchronous Tag Check Fault */
>> -#define TIF_NOTIFY_SIGNAL 7 /* signal notifications exist */
>> #define TIF_SYSCALL_TRACE 8 /* syscall trace active */
>> #define TIF_SYSCALL_AUDIT 9 /* syscall auditing */
>> #define TIF_SYSCALL_TRACEPOINT 10 /* syscall tracepoint for ftrace */
>> @@@ -96,7 -103,8 +98,8 @@@
>>
>> #define _TIF_WORK_MASK (_TIF_NEED_RESCHED | _TIF_SIGPENDING | \
>> _TIF_NOTIFY_RESUME | _TIF_FOREIGN_FPSTATE | \
>> - _TIF_UPROBE | _TIF_MTE_ASYNC_FAULT)
>> - _TIF_UPROBE | _TIF_FSCHECK | _TIF_MTE_ASYNC_FAULT | \
>> ++ _TIF_UPROBE | _TIF_MTE_ASYNC_FAULT | \
>> + _TIF_NOTIFY_SIGNAL)
>
> Thanks Stephen. It looks alright to me.
Agree - I'll rebase my tree when -rc7 is out so we won't have this issue once
the 5.11 merge window opens.
--
Jens Axboe
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: linux-next: manual merge of the block tree with the arm64 tree
2020-12-03 14:36 ` Jens Axboe
@ 2020-12-03 15:05 ` Catalin Marinas
2020-12-03 15:58 ` Jens Axboe
0 siblings, 1 reply; 6+ messages in thread
From: Catalin Marinas @ 2020-12-03 15:05 UTC (permalink / raw)
To: Jens Axboe
Cc: Stephen Rothwell, Will Deacon, Linux Kernel Mailing List,
Linux Next Mailing List, Mark Rutland
On Thu, Dec 03, 2020 at 07:36:10AM -0700, Jens Axboe wrote:
> On 12/3/20 4:01 AM, Catalin Marinas wrote:
> > On Thu, Dec 03, 2020 at 02:25:30PM +1100, Stephen Rothwell wrote:
> >> diff --cc arch/arm64/include/asm/thread_info.h
> >> index 015beafe58f5,cdcf307764aa..000000000000
> >> --- a/arch/arm64/include/asm/thread_info.h
> >> +++ b/arch/arm64/include/asm/thread_info.h
> >> @@@ -63,7 -66,9 +63,8 @@@ void arch_release_task_struct(struct ta
> >> #define TIF_NOTIFY_RESUME 2 /* callback before returning to user */
> >> #define TIF_FOREIGN_FPSTATE 3 /* CPU's FP state is not current's */
> >> #define TIF_UPROBE 4 /* uprobe breakpoint or singlestep */
> >> - #define TIF_MTE_ASYNC_FAULT 5 /* MTE Asynchronous Tag Check Fault */
> >> -#define TIF_FSCHECK 5 /* Check FS is USER_DS on return */
> >> ++#define TIF_NOTIFY_SIGNAL 5 /* signal notifications exist */
> >> + #define TIF_MTE_ASYNC_FAULT 6 /* MTE Asynchronous Tag Check Fault */
> >> -#define TIF_NOTIFY_SIGNAL 7 /* signal notifications exist */
> >> #define TIF_SYSCALL_TRACE 8 /* syscall trace active */
> >> #define TIF_SYSCALL_AUDIT 9 /* syscall auditing */
> >> #define TIF_SYSCALL_TRACEPOINT 10 /* syscall tracepoint for ftrace */
> >> @@@ -96,7 -103,8 +98,8 @@@
> >>
> >> #define _TIF_WORK_MASK (_TIF_NEED_RESCHED | _TIF_SIGPENDING | \
> >> _TIF_NOTIFY_RESUME | _TIF_FOREIGN_FPSTATE | \
> >> - _TIF_UPROBE | _TIF_MTE_ASYNC_FAULT)
> >> - _TIF_UPROBE | _TIF_FSCHECK | _TIF_MTE_ASYNC_FAULT | \
> >> ++ _TIF_UPROBE | _TIF_MTE_ASYNC_FAULT | \
> >> + _TIF_NOTIFY_SIGNAL)
> >
> > Thanks Stephen. It looks alright to me.
>
> Agree - I'll rebase my tree when -rc7 is out so we won't have this issue once
> the 5.11 merge window opens.
I don't think rebasing on -rc7 will help since the arm64 commit
b5a5a01d8e9a is queued for 5.11 (so not in -rc7).
It shouldn't matter much, Linus likes the occasional conflict ;).
Anyway, I can wait for your pull request to go in if you'd prefer (and
if it happens in the first week of the merging window).
--
Catalin
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: linux-next: manual merge of the block tree with the arm64 tree
2020-12-03 15:05 ` Catalin Marinas
@ 2020-12-03 15:58 ` Jens Axboe
0 siblings, 0 replies; 6+ messages in thread
From: Jens Axboe @ 2020-12-03 15:58 UTC (permalink / raw)
To: Catalin Marinas
Cc: Stephen Rothwell, Will Deacon, Linux Kernel Mailing List,
Linux Next Mailing List, Mark Rutland
On 12/3/20 8:05 AM, Catalin Marinas wrote:
> On Thu, Dec 03, 2020 at 07:36:10AM -0700, Jens Axboe wrote:
>> On 12/3/20 4:01 AM, Catalin Marinas wrote:
>>> On Thu, Dec 03, 2020 at 02:25:30PM +1100, Stephen Rothwell wrote:
>>>> diff --cc arch/arm64/include/asm/thread_info.h
>>>> index 015beafe58f5,cdcf307764aa..000000000000
>>>> --- a/arch/arm64/include/asm/thread_info.h
>>>> +++ b/arch/arm64/include/asm/thread_info.h
>>>> @@@ -63,7 -66,9 +63,8 @@@ void arch_release_task_struct(struct ta
>>>> #define TIF_NOTIFY_RESUME 2 /* callback before returning to user */
>>>> #define TIF_FOREIGN_FPSTATE 3 /* CPU's FP state is not current's */
>>>> #define TIF_UPROBE 4 /* uprobe breakpoint or singlestep */
>>>> - #define TIF_MTE_ASYNC_FAULT 5 /* MTE Asynchronous Tag Check Fault */
>>>> -#define TIF_FSCHECK 5 /* Check FS is USER_DS on return */
>>>> ++#define TIF_NOTIFY_SIGNAL 5 /* signal notifications exist */
>>>> + #define TIF_MTE_ASYNC_FAULT 6 /* MTE Asynchronous Tag Check Fault */
>>>> -#define TIF_NOTIFY_SIGNAL 7 /* signal notifications exist */
>>>> #define TIF_SYSCALL_TRACE 8 /* syscall trace active */
>>>> #define TIF_SYSCALL_AUDIT 9 /* syscall auditing */
>>>> #define TIF_SYSCALL_TRACEPOINT 10 /* syscall tracepoint for ftrace */
>>>> @@@ -96,7 -103,8 +98,8 @@@
>>>>
>>>> #define _TIF_WORK_MASK (_TIF_NEED_RESCHED | _TIF_SIGPENDING | \
>>>> _TIF_NOTIFY_RESUME | _TIF_FOREIGN_FPSTATE | \
>>>> - _TIF_UPROBE | _TIF_MTE_ASYNC_FAULT)
>>>> - _TIF_UPROBE | _TIF_FSCHECK | _TIF_MTE_ASYNC_FAULT | \
>>>> ++ _TIF_UPROBE | _TIF_MTE_ASYNC_FAULT | \
>>>> + _TIF_NOTIFY_SIGNAL)
>>>
>>> Thanks Stephen. It looks alright to me.
>>
>> Agree - I'll rebase my tree when -rc7 is out so we won't have this issue once
>> the 5.11 merge window opens.
>
> I don't think rebasing on -rc7 will help since the arm64 commit
> b5a5a01d8e9a is queued for 5.11 (so not in -rc7).
Ah indeed, I saw some changes come in yesterday for mainline and assumed
it was those.
> It shouldn't matter much, Linus likes the occasional conflict ;).
> Anyway, I can wait for your pull request to go in if you'd prefer (and
> if it happens in the first week of the merging window).
Right, not an issue, it's a trivial resolve anyway. That branch is
dependent on an x86/core branch, so I'll push it out when that goes in.
But Linus usually pulls those early, so don't think we'll have much of
an issue there.
--
Jens Axboe
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: linux-next: manual merge of the block tree with the arm64 tree
2020-12-03 3:25 linux-next: manual merge of the block tree with the arm64 tree Stephen Rothwell
2020-12-03 11:01 ` Catalin Marinas
@ 2020-12-14 20:17 ` Stephen Rothwell
1 sibling, 0 replies; 6+ messages in thread
From: Stephen Rothwell @ 2020-12-14 20:17 UTC (permalink / raw)
To: Jens Axboe, Catalin Marinas, Will Deacon
Cc: Linux Kernel Mailing List, Linux Next Mailing List, Mark Rutland
[-- Attachment #1: Type: text/plain, Size: 2410 bytes --]
Hi all,
On Thu, 3 Dec 2020 14:25:30 +1100 Stephen Rothwell <sfr@canb.auug.org.au> wrote:
>
> Today's linux-next merge of the block tree got a conflict in:
>
> arch/arm64/include/asm/thread_info.h
>
> between commit:
>
> b5a5a01d8e9a ("arm64: uaccess: remove addr_limit_user_check()")
>
> from the arm64 tree and commit:
>
> 192caabd4dd9 ("arm64: add support for TIF_NOTIFY_SIGNAL")
>
> from the block tree.
>
> I fixed it up (see below) and can carry the fix as necessary. This
> is now fixed as far as linux-next is concerned, but any non trivial
> conflicts should be mentioned to your upstream maintainer when your tree
> is submitted for merging. You may also want to consider cooperating
> with the maintainer of the conflicting tree to minimise any particularly
> complex conflicts.
>
> diff --cc arch/arm64/include/asm/thread_info.h
> index 015beafe58f5,cdcf307764aa..000000000000
> --- a/arch/arm64/include/asm/thread_info.h
> +++ b/arch/arm64/include/asm/thread_info.h
> @@@ -63,7 -66,9 +63,8 @@@ void arch_release_task_struct(struct ta
> #define TIF_NOTIFY_RESUME 2 /* callback before returning to user */
> #define TIF_FOREIGN_FPSTATE 3 /* CPU's FP state is not current's */
> #define TIF_UPROBE 4 /* uprobe breakpoint or singlestep */
> - #define TIF_MTE_ASYNC_FAULT 5 /* MTE Asynchronous Tag Check Fault */
> -#define TIF_FSCHECK 5 /* Check FS is USER_DS on return */
> ++#define TIF_NOTIFY_SIGNAL 5 /* signal notifications exist */
> + #define TIF_MTE_ASYNC_FAULT 6 /* MTE Asynchronous Tag Check Fault */
> -#define TIF_NOTIFY_SIGNAL 7 /* signal notifications exist */
> #define TIF_SYSCALL_TRACE 8 /* syscall trace active */
> #define TIF_SYSCALL_AUDIT 9 /* syscall auditing */
> #define TIF_SYSCALL_TRACEPOINT 10 /* syscall tracepoint for ftrace */
> @@@ -96,7 -103,8 +98,8 @@@
>
> #define _TIF_WORK_MASK (_TIF_NEED_RESCHED | _TIF_SIGPENDING | \
> _TIF_NOTIFY_RESUME | _TIF_FOREIGN_FPSTATE | \
> - _TIF_UPROBE | _TIF_MTE_ASYNC_FAULT)
> - _TIF_UPROBE | _TIF_FSCHECK | _TIF_MTE_ASYNC_FAULT | \
> ++ _TIF_UPROBE | _TIF_MTE_ASYNC_FAULT | \
> + _TIF_NOTIFY_SIGNAL)
>
> #define _TIF_SYSCALL_WORK (_TIF_SYSCALL_TRACE | _TIF_SYSCALL_AUDIT | \
> _TIF_SYSCALL_TRACEPOINT | _TIF_SECCOMP | \
Just a reminder that this conflict still exists.
--
Cheers,
Stephen Rothwell
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2020-12-14 20:18 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-12-03 3:25 linux-next: manual merge of the block tree with the arm64 tree Stephen Rothwell
2020-12-03 11:01 ` Catalin Marinas
2020-12-03 14:36 ` Jens Axboe
2020-12-03 15:05 ` Catalin Marinas
2020-12-03 15:58 ` Jens Axboe
2020-12-14 20:17 ` Stephen Rothwell
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).