* [PATCH] md: dm-writeback: add __noreturn to BUG-ging function
@ 2020-11-13 22:52 ` Randy Dunlap
0 siblings, 0 replies; 20+ messages in thread
From: Randy Dunlap @ 2020-11-13 22:52 UTC (permalink / raw)
To: linux-kernel
Cc: Randy Dunlap, Mikulas Patocka, Alasdair Kergon, Mike Snitzer,
dm-devel, Heiko Carstens, Vasily Gorbik, Christian Borntraeger,
linux-s390
Building on arch/s390/ flags this as an error, so add the
__noreturn attribute modifier to prevent the build error.
cc1: some warnings being treated as errors
../drivers/md/dm-writecache.c: In function 'persistent_memory_claim':
../drivers/md/dm-writecache.c:323:1: error: no return statement in function returning non-void [-Werror=return-type]
Fixes: 48debafe4f2f ("dm: add writecache target")
Signed-off-by: Randy Dunlap <rdunlap@infradead.org>
Cc: Mikulas Patocka <mpatocka@redhat.com>
Cc: Alasdair Kergon <agk@redhat.com>
Cc: Mike Snitzer <snitzer@redhat.com>
Cc: dm-devel@redhat.com
Cc: Heiko Carstens <hca@linux.ibm.com>
Cc: Vasily Gorbik <gor@linux.ibm.com>
Cc: Christian Borntraeger <borntraeger@de.ibm.com>
Cc: linux-s390@vger.kernel.org
---
drivers/md/dm-writecache.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
--- linux-next-20201113.orig/drivers/md/dm-writecache.c
+++ linux-next-20201113/drivers/md/dm-writecache.c
@@ -317,7 +317,7 @@ err1:
return r;
}
#else
-static int persistent_memory_claim(struct dm_writecache *wc)
+static int __noreturn persistent_memory_claim(struct dm_writecache *wc)
{
BUG();
}
^ permalink raw reply [flat|nested] 20+ messages in thread
* [dm-devel] [PATCH] md: dm-writeback: add __noreturn to BUG-ging function
@ 2020-11-13 22:52 ` Randy Dunlap
0 siblings, 0 replies; 20+ messages in thread
From: Randy Dunlap @ 2020-11-13 22:52 UTC (permalink / raw)
To: linux-kernel
Cc: linux-s390, Vasily Gorbik, Mike Snitzer, Heiko Carstens,
Randy Dunlap, Christian Borntraeger, dm-devel, Mikulas Patocka,
Alasdair Kergon
Building on arch/s390/ flags this as an error, so add the
__noreturn attribute modifier to prevent the build error.
cc1: some warnings being treated as errors
../drivers/md/dm-writecache.c: In function 'persistent_memory_claim':
../drivers/md/dm-writecache.c:323:1: error: no return statement in function returning non-void [-Werror=return-type]
Fixes: 48debafe4f2f ("dm: add writecache target")
Signed-off-by: Randy Dunlap <rdunlap@infradead.org>
Cc: Mikulas Patocka <mpatocka@redhat.com>
Cc: Alasdair Kergon <agk@redhat.com>
Cc: Mike Snitzer <snitzer@redhat.com>
Cc: dm-devel@redhat.com
Cc: Heiko Carstens <hca@linux.ibm.com>
Cc: Vasily Gorbik <gor@linux.ibm.com>
Cc: Christian Borntraeger <borntraeger@de.ibm.com>
Cc: linux-s390@vger.kernel.org
---
drivers/md/dm-writecache.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
--- linux-next-20201113.orig/drivers/md/dm-writecache.c
+++ linux-next-20201113/drivers/md/dm-writecache.c
@@ -317,7 +317,7 @@ err1:
return r;
}
#else
-static int persistent_memory_claim(struct dm_writecache *wc)
+static int __noreturn persistent_memory_claim(struct dm_writecache *wc)
{
BUG();
}
--
dm-devel mailing list
dm-devel@redhat.com
https://www.redhat.com/mailman/listinfo/dm-devel
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH] md: dm-writeback: add __noreturn to BUG-ging function
2020-11-13 22:52 ` [dm-devel] " Randy Dunlap
@ 2020-11-16 7:30 ` Christian Borntraeger
-1 siblings, 0 replies; 20+ messages in thread
From: Christian Borntraeger @ 2020-11-16 7:30 UTC (permalink / raw)
To: Randy Dunlap, linux-kernel
Cc: Mikulas Patocka, Alasdair Kergon, Mike Snitzer, dm-devel,
Heiko Carstens, Vasily Gorbik, linux-s390
On 13.11.20 23:52, Randy Dunlap wrote:
> Building on arch/s390/ flags this as an error, so add the
> __noreturn attribute modifier to prevent the build error.
>
> cc1: some warnings being treated as errors
> ../drivers/md/dm-writecache.c: In function 'persistent_memory_claim':
> ../drivers/md/dm-writecache.c:323:1: error: no return statement in function returning non-void [-Werror=return-type]
ok with me, but I am asking why
the unreachable macro is not good enough. For x86 it obviously is.
form arch/s390/include/asm/bug.h
#define BUG() do { \
__EMIT_BUG(0); \
unreachable(); \
} while (0)
>
> Fixes: 48debafe4f2f ("dm: add writecache target")
> Signed-off-by: Randy Dunlap <rdunlap@infradead.org>
> Cc: Mikulas Patocka <mpatocka@redhat.com>
> Cc: Alasdair Kergon <agk@redhat.com>
> Cc: Mike Snitzer <snitzer@redhat.com>
> Cc: dm-devel@redhat.com
> Cc: Heiko Carstens <hca@linux.ibm.com>
> Cc: Vasily Gorbik <gor@linux.ibm.com>
> Cc: Christian Borntraeger <borntraeger@de.ibm.com>
> Cc: linux-s390@vger.kernel.org
> ---
> drivers/md/dm-writecache.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> --- linux-next-20201113.orig/drivers/md/dm-writecache.c
> +++ linux-next-20201113/drivers/md/dm-writecache.c
> @@ -317,7 +317,7 @@ err1:
> return r;
> }
> #else
> -static int persistent_memory_claim(struct dm_writecache *wc)
> +static int __noreturn persistent_memory_claim(struct dm_writecache *wc)
> {
> BUG();
> }
>
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [dm-devel] [PATCH] md: dm-writeback: add __noreturn to BUG-ging function
@ 2020-11-16 7:30 ` Christian Borntraeger
0 siblings, 0 replies; 20+ messages in thread
From: Christian Borntraeger @ 2020-11-16 7:30 UTC (permalink / raw)
To: Randy Dunlap, linux-kernel
Cc: linux-s390, Mike Snitzer, Vasily Gorbik, Heiko Carstens,
dm-devel, Mikulas Patocka, Alasdair Kergon
On 13.11.20 23:52, Randy Dunlap wrote:
> Building on arch/s390/ flags this as an error, so add the
> __noreturn attribute modifier to prevent the build error.
>
> cc1: some warnings being treated as errors
> ../drivers/md/dm-writecache.c: In function 'persistent_memory_claim':
> ../drivers/md/dm-writecache.c:323:1: error: no return statement in function returning non-void [-Werror=return-type]
ok with me, but I am asking why
the unreachable macro is not good enough. For x86 it obviously is.
form arch/s390/include/asm/bug.h
#define BUG() do { \
__EMIT_BUG(0); \
unreachable(); \
} while (0)
>
> Fixes: 48debafe4f2f ("dm: add writecache target")
> Signed-off-by: Randy Dunlap <rdunlap@infradead.org>
> Cc: Mikulas Patocka <mpatocka@redhat.com>
> Cc: Alasdair Kergon <agk@redhat.com>
> Cc: Mike Snitzer <snitzer@redhat.com>
> Cc: dm-devel@redhat.com
> Cc: Heiko Carstens <hca@linux.ibm.com>
> Cc: Vasily Gorbik <gor@linux.ibm.com>
> Cc: Christian Borntraeger <borntraeger@de.ibm.com>
> Cc: linux-s390@vger.kernel.org
> ---
> drivers/md/dm-writecache.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> --- linux-next-20201113.orig/drivers/md/dm-writecache.c
> +++ linux-next-20201113/drivers/md/dm-writecache.c
> @@ -317,7 +317,7 @@ err1:
> return r;
> }
> #else
> -static int persistent_memory_claim(struct dm_writecache *wc)
> +static int __noreturn persistent_memory_claim(struct dm_writecache *wc)
> {
> BUG();
> }
>
--
dm-devel mailing list
dm-devel@redhat.com
https://www.redhat.com/mailman/listinfo/dm-devel
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH] md: dm-writeback: add __noreturn to BUG-ging function
2020-11-16 7:30 ` [dm-devel] " Christian Borntraeger
@ 2020-11-16 23:00 ` Randy Dunlap
-1 siblings, 0 replies; 20+ messages in thread
From: Randy Dunlap @ 2020-11-16 23:00 UTC (permalink / raw)
To: Christian Borntraeger, linux-kernel
Cc: Mikulas Patocka, Alasdair Kergon, Mike Snitzer, dm-devel,
Heiko Carstens, Vasily Gorbik, linux-s390
On 11/15/20 11:30 PM, Christian Borntraeger wrote:
>
>
> On 13.11.20 23:52, Randy Dunlap wrote:
>> Building on arch/s390/ flags this as an error, so add the
>> __noreturn attribute modifier to prevent the build error.
>>
>> cc1: some warnings being treated as errors
>> ../drivers/md/dm-writecache.c: In function 'persistent_memory_claim':
>> ../drivers/md/dm-writecache.c:323:1: error: no return statement in function returning non-void [-Werror=return-type]
>
> ok with me, but I am asking why
>
> the unreachable macro is not good enough. For x86 it obviously is.
>
> form arch/s390/include/asm/bug.h
> #define BUG() do { \
> __EMIT_BUG(0); \
> unreachable(); \
> } while (0)
>
Hi Christian,
Good question.
I don't see any guidance about when to use one or the other etc.
I see __noreturn being used 109 times and unreachable();
being used 33 times, but only now that I look at them.
That had nothing to do with why I used __noreturn in the patch.
>
>>
>> Fixes: 48debafe4f2f ("dm: add writecache target")
>> Signed-off-by: Randy Dunlap <rdunlap@infradead.org>
>> Cc: Mikulas Patocka <mpatocka@redhat.com>
>> Cc: Alasdair Kergon <agk@redhat.com>
>> Cc: Mike Snitzer <snitzer@redhat.com>
>> Cc: dm-devel@redhat.com
>> Cc: Heiko Carstens <hca@linux.ibm.com>
>> Cc: Vasily Gorbik <gor@linux.ibm.com>
>> Cc: Christian Borntraeger <borntraeger@de.ibm.com>
>> Cc: linux-s390@vger.kernel.org
>> ---
>> drivers/md/dm-writecache.c | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> --- linux-next-20201113.orig/drivers/md/dm-writecache.c
>> +++ linux-next-20201113/drivers/md/dm-writecache.c
>> @@ -317,7 +317,7 @@ err1:
>> return r;
>> }
>> #else
>> -static int persistent_memory_claim(struct dm_writecache *wc)
>> +static int __noreturn persistent_memory_claim(struct dm_writecache *wc)
>> {
>> BUG();
>> }
>>
thanks.
--
~Randy
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [dm-devel] [PATCH] md: dm-writeback: add __noreturn to BUG-ging function
@ 2020-11-16 23:00 ` Randy Dunlap
0 siblings, 0 replies; 20+ messages in thread
From: Randy Dunlap @ 2020-11-16 23:00 UTC (permalink / raw)
To: Christian Borntraeger, linux-kernel
Cc: linux-s390, Mike Snitzer, Vasily Gorbik, Heiko Carstens,
dm-devel, Mikulas Patocka, Alasdair Kergon
On 11/15/20 11:30 PM, Christian Borntraeger wrote:
>
>
> On 13.11.20 23:52, Randy Dunlap wrote:
>> Building on arch/s390/ flags this as an error, so add the
>> __noreturn attribute modifier to prevent the build error.
>>
>> cc1: some warnings being treated as errors
>> ../drivers/md/dm-writecache.c: In function 'persistent_memory_claim':
>> ../drivers/md/dm-writecache.c:323:1: error: no return statement in function returning non-void [-Werror=return-type]
>
> ok with me, but I am asking why
>
> the unreachable macro is not good enough. For x86 it obviously is.
>
> form arch/s390/include/asm/bug.h
> #define BUG() do { \
> __EMIT_BUG(0); \
> unreachable(); \
> } while (0)
>
Hi Christian,
Good question.
I don't see any guidance about when to use one or the other etc.
I see __noreturn being used 109 times and unreachable();
being used 33 times, but only now that I look at them.
That had nothing to do with why I used __noreturn in the patch.
>
>>
>> Fixes: 48debafe4f2f ("dm: add writecache target")
>> Signed-off-by: Randy Dunlap <rdunlap@infradead.org>
>> Cc: Mikulas Patocka <mpatocka@redhat.com>
>> Cc: Alasdair Kergon <agk@redhat.com>
>> Cc: Mike Snitzer <snitzer@redhat.com>
>> Cc: dm-devel@redhat.com
>> Cc: Heiko Carstens <hca@linux.ibm.com>
>> Cc: Vasily Gorbik <gor@linux.ibm.com>
>> Cc: Christian Borntraeger <borntraeger@de.ibm.com>
>> Cc: linux-s390@vger.kernel.org
>> ---
>> drivers/md/dm-writecache.c | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> --- linux-next-20201113.orig/drivers/md/dm-writecache.c
>> +++ linux-next-20201113/drivers/md/dm-writecache.c
>> @@ -317,7 +317,7 @@ err1:
>> return r;
>> }
>> #else
>> -static int persistent_memory_claim(struct dm_writecache *wc)
>> +static int __noreturn persistent_memory_claim(struct dm_writecache *wc)
>> {
>> BUG();
>> }
>>
thanks.
--
~Randy
--
dm-devel mailing list
dm-devel@redhat.com
https://www.redhat.com/mailman/listinfo/dm-devel
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: md: dm-writeback: add __noreturn to BUG-ging function
2020-11-16 23:00 ` [dm-devel] " Randy Dunlap
@ 2020-11-17 16:31 ` Mike Snitzer
-1 siblings, 0 replies; 20+ messages in thread
From: Mike Snitzer @ 2020-11-17 16:31 UTC (permalink / raw)
To: Randy Dunlap
Cc: Christian Borntraeger, linux-kernel, Mikulas Patocka,
Alasdair Kergon, dm-devel, Heiko Carstens, Vasily Gorbik,
linux-s390
On Mon, Nov 16 2020 at 6:00pm -0500,
Randy Dunlap <rdunlap@infradead.org> wrote:
> On 11/15/20 11:30 PM, Christian Borntraeger wrote:
> >
> >
> > On 13.11.20 23:52, Randy Dunlap wrote:
> >> Building on arch/s390/ flags this as an error, so add the
> >> __noreturn attribute modifier to prevent the build error.
> >>
> >> cc1: some warnings being treated as errors
> >> ../drivers/md/dm-writecache.c: In function 'persistent_memory_claim':
> >> ../drivers/md/dm-writecache.c:323:1: error: no return statement in function returning non-void [-Werror=return-type]
> >
> > ok with me, but I am asking why
> >
> > the unreachable macro is not good enough. For x86 it obviously is.
> >
> > form arch/s390/include/asm/bug.h
> > #define BUG() do { \
> > __EMIT_BUG(0); \
> > unreachable(); \
> > } while (0)
> >
>
> Hi Christian,
>
> Good question.
> I don't see any guidance about when to use one or the other etc.
>
> I see __noreturn being used 109 times and unreachable();
> being used 33 times, but only now that I look at them.
> That had nothing to do with why I used __noreturn in the patch.
But doesn't that speak to the proper fix being needed in unreachable()?
Or at a minimum the fix is needed to arch/s390/include/asm/bug.h's BUG.
I really don't think we should be papering over that by sprinkling
__noreturn around the kernel's BUG() callers.
Maybe switch arch/s390/include/asm/bug.h's BUG to be like
arch/mips/include/asm/bug.h? It itself uses __noreturn with a 'static
inline' function definition rather than #define.
Does that fix the issue?
Thanks,
Mike
p.s. you modified dm-writecache.c (not dm-writeback, wich doesn't
exist).
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [dm-devel] md: dm-writeback: add __noreturn to BUG-ging function
@ 2020-11-17 16:31 ` Mike Snitzer
0 siblings, 0 replies; 20+ messages in thread
From: Mike Snitzer @ 2020-11-17 16:31 UTC (permalink / raw)
To: Randy Dunlap
Cc: linux-s390, Vasily Gorbik, Heiko Carstens, linux-kernel,
Christian Borntraeger, dm-devel, Mikulas Patocka,
Alasdair Kergon
On Mon, Nov 16 2020 at 6:00pm -0500,
Randy Dunlap <rdunlap@infradead.org> wrote:
> On 11/15/20 11:30 PM, Christian Borntraeger wrote:
> >
> >
> > On 13.11.20 23:52, Randy Dunlap wrote:
> >> Building on arch/s390/ flags this as an error, so add the
> >> __noreturn attribute modifier to prevent the build error.
> >>
> >> cc1: some warnings being treated as errors
> >> ../drivers/md/dm-writecache.c: In function 'persistent_memory_claim':
> >> ../drivers/md/dm-writecache.c:323:1: error: no return statement in function returning non-void [-Werror=return-type]
> >
> > ok with me, but I am asking why
> >
> > the unreachable macro is not good enough. For x86 it obviously is.
> >
> > form arch/s390/include/asm/bug.h
> > #define BUG() do { \
> > __EMIT_BUG(0); \
> > unreachable(); \
> > } while (0)
> >
>
> Hi Christian,
>
> Good question.
> I don't see any guidance about when to use one or the other etc.
>
> I see __noreturn being used 109 times and unreachable();
> being used 33 times, but only now that I look at them.
> That had nothing to do with why I used __noreturn in the patch.
But doesn't that speak to the proper fix being needed in unreachable()?
Or at a minimum the fix is needed to arch/s390/include/asm/bug.h's BUG.
I really don't think we should be papering over that by sprinkling
__noreturn around the kernel's BUG() callers.
Maybe switch arch/s390/include/asm/bug.h's BUG to be like
arch/mips/include/asm/bug.h? It itself uses __noreturn with a 'static
inline' function definition rather than #define.
Does that fix the issue?
Thanks,
Mike
p.s. you modified dm-writecache.c (not dm-writeback, wich doesn't
exist).
--
dm-devel mailing list
dm-devel@redhat.com
https://www.redhat.com/mailman/listinfo/dm-devel
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: md: dm-writeback: add __noreturn to BUG-ging function
2020-11-17 16:31 ` [dm-devel] " Mike Snitzer
@ 2020-11-18 15:49 ` Mike Snitzer
-1 siblings, 0 replies; 20+ messages in thread
From: Mike Snitzer @ 2020-11-18 15:49 UTC (permalink / raw)
To: Randy Dunlap
Cc: Christian Borntraeger, linux-kernel, Mikulas Patocka,
Alasdair Kergon, dm-devel, Heiko Carstens, Vasily Gorbik,
linux-s390
On Tue, Nov 17 2020 at 11:31am -0500,
Mike Snitzer <snitzer@redhat.com> wrote:
> On Mon, Nov 16 2020 at 6:00pm -0500,
> Randy Dunlap <rdunlap@infradead.org> wrote:
>
> > On 11/15/20 11:30 PM, Christian Borntraeger wrote:
> > >
> > >
> > > On 13.11.20 23:52, Randy Dunlap wrote:
> > >> Building on arch/s390/ flags this as an error, so add the
> > >> __noreturn attribute modifier to prevent the build error.
> > >>
> > >> cc1: some warnings being treated as errors
> > >> ../drivers/md/dm-writecache.c: In function 'persistent_memory_claim':
> > >> ../drivers/md/dm-writecache.c:323:1: error: no return statement in function returning non-void [-Werror=return-type]
> > >
> > > ok with me, but I am asking why
> > >
> > > the unreachable macro is not good enough. For x86 it obviously is.
> > >
> > > form arch/s390/include/asm/bug.h
> > > #define BUG() do { \
> > > __EMIT_BUG(0); \
> > > unreachable(); \
> > > } while (0)
> > >
> >
> > Hi Christian,
> >
> > Good question.
> > I don't see any guidance about when to use one or the other etc.
> >
> > I see __noreturn being used 109 times and unreachable();
> > being used 33 times, but only now that I look at them.
> > That had nothing to do with why I used __noreturn in the patch.
>
> But doesn't that speak to the proper fix being needed in unreachable()?
> Or at a minimum the fix is needed to arch/s390/include/asm/bug.h's BUG.
>
> I really don't think we should be papering over that by sprinkling
> __noreturn around the kernel's BUG() callers.
>
> Maybe switch arch/s390/include/asm/bug.h's BUG to be like
> arch/mips/include/asm/bug.h? It itself uses __noreturn with a 'static
> inline' function definition rather than #define.
>
> Does that fix the issue?
>
> Thanks,
> Mike
>
> p.s. you modified dm-writecache.c (not dm-writeback, wich doesn't
> exist).
I don't think my suggestion will help.. given it'd still leave
persistent_memory_claim() without a return statement.
Think it worthwhile to just add a dummy 'return 0;' after the BUG().
Mike
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [dm-devel] md: dm-writeback: add __noreturn to BUG-ging function
@ 2020-11-18 15:49 ` Mike Snitzer
0 siblings, 0 replies; 20+ messages in thread
From: Mike Snitzer @ 2020-11-18 15:49 UTC (permalink / raw)
To: Randy Dunlap
Cc: linux-s390, Vasily Gorbik, Heiko Carstens, linux-kernel,
Christian Borntraeger, dm-devel, Mikulas Patocka,
Alasdair Kergon
On Tue, Nov 17 2020 at 11:31am -0500,
Mike Snitzer <snitzer@redhat.com> wrote:
> On Mon, Nov 16 2020 at 6:00pm -0500,
> Randy Dunlap <rdunlap@infradead.org> wrote:
>
> > On 11/15/20 11:30 PM, Christian Borntraeger wrote:
> > >
> > >
> > > On 13.11.20 23:52, Randy Dunlap wrote:
> > >> Building on arch/s390/ flags this as an error, so add the
> > >> __noreturn attribute modifier to prevent the build error.
> > >>
> > >> cc1: some warnings being treated as errors
> > >> ../drivers/md/dm-writecache.c: In function 'persistent_memory_claim':
> > >> ../drivers/md/dm-writecache.c:323:1: error: no return statement in function returning non-void [-Werror=return-type]
> > >
> > > ok with me, but I am asking why
> > >
> > > the unreachable macro is not good enough. For x86 it obviously is.
> > >
> > > form arch/s390/include/asm/bug.h
> > > #define BUG() do { \
> > > __EMIT_BUG(0); \
> > > unreachable(); \
> > > } while (0)
> > >
> >
> > Hi Christian,
> >
> > Good question.
> > I don't see any guidance about when to use one or the other etc.
> >
> > I see __noreturn being used 109 times and unreachable();
> > being used 33 times, but only now that I look at them.
> > That had nothing to do with why I used __noreturn in the patch.
>
> But doesn't that speak to the proper fix being needed in unreachable()?
> Or at a minimum the fix is needed to arch/s390/include/asm/bug.h's BUG.
>
> I really don't think we should be papering over that by sprinkling
> __noreturn around the kernel's BUG() callers.
>
> Maybe switch arch/s390/include/asm/bug.h's BUG to be like
> arch/mips/include/asm/bug.h? It itself uses __noreturn with a 'static
> inline' function definition rather than #define.
>
> Does that fix the issue?
>
> Thanks,
> Mike
>
> p.s. you modified dm-writecache.c (not dm-writeback, wich doesn't
> exist).
I don't think my suggestion will help.. given it'd still leave
persistent_memory_claim() without a return statement.
Think it worthwhile to just add a dummy 'return 0;' after the BUG().
Mike
--
dm-devel mailing list
dm-devel@redhat.com
https://www.redhat.com/mailman/listinfo/dm-devel
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: md: dm-writeback: add __noreturn to BUG-ging function
2020-11-18 15:49 ` [dm-devel] " Mike Snitzer
@ 2020-11-18 16:07 ` Mike Snitzer
-1 siblings, 0 replies; 20+ messages in thread
From: Mike Snitzer @ 2020-11-18 16:07 UTC (permalink / raw)
To: Randy Dunlap
Cc: Christian Borntraeger, linux-kernel, Mikulas Patocka,
Alasdair Kergon, dm-devel, Heiko Carstens, Vasily Gorbik,
linux-s390
On Wed, Nov 18 2020 at 10:49am -0500,
Mike Snitzer <snitzer@redhat.com> wrote:
> I don't think my suggestion will help.. given it'd still leave
> persistent_memory_claim() without a return statement.
>
> Think it worthwhile to just add a dummy 'return 0;' after the BUG().
Decided to go with this, now staged for 5.11:
https://git.kernel.org/pub/scm/linux/kernel/git/device-mapper/linux-dm.git/commit/?h=dm-5.11&id=a1e4865b4dda7071f3707f7e551289ead66e38b1
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [dm-devel] md: dm-writeback: add __noreturn to BUG-ging function
@ 2020-11-18 16:07 ` Mike Snitzer
0 siblings, 0 replies; 20+ messages in thread
From: Mike Snitzer @ 2020-11-18 16:07 UTC (permalink / raw)
To: Randy Dunlap
Cc: linux-s390, Vasily Gorbik, Heiko Carstens, linux-kernel,
Christian Borntraeger, dm-devel, Mikulas Patocka,
Alasdair Kergon
On Wed, Nov 18 2020 at 10:49am -0500,
Mike Snitzer <snitzer@redhat.com> wrote:
> I don't think my suggestion will help.. given it'd still leave
> persistent_memory_claim() without a return statement.
>
> Think it worthwhile to just add a dummy 'return 0;' after the BUG().
Decided to go with this, now staged for 5.11:
https://git.kernel.org/pub/scm/linux/kernel/git/device-mapper/linux-dm.git/commit/?h=dm-5.11&id=a1e4865b4dda7071f3707f7e551289ead66e38b1
--
dm-devel mailing list
dm-devel@redhat.com
https://www.redhat.com/mailman/listinfo/dm-devel
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: md: dm-writeback: add __noreturn to BUG-ging function
2020-11-18 16:07 ` [dm-devel] " Mike Snitzer
@ 2020-11-18 16:35 ` Christian Borntraeger
-1 siblings, 0 replies; 20+ messages in thread
From: Christian Borntraeger @ 2020-11-18 16:35 UTC (permalink / raw)
To: Mike Snitzer, Randy Dunlap
Cc: linux-kernel, Mikulas Patocka, Alasdair Kergon, dm-devel,
Heiko Carstens, Vasily Gorbik, linux-s390
On 18.11.20 17:07, Mike Snitzer wrote:
> On Wed, Nov 18 2020 at 10:49am -0500,
> Mike Snitzer <snitzer@redhat.com> wrote:
>
>> I don't think my suggestion will help.. given it'd still leave
>> persistent_memory_claim() without a return statement.
>>
>> Think it worthwhile to just add a dummy 'return 0;' after the BUG().
>
> Decided to go with this, now staged for 5.11:
> https://git.kernel.org/pub/scm/linux/kernel/git/device-mapper/linux-dm.git/commit/?h=dm-5.11&id=a1e4865b4dda7071f3707f7e551289ead66e38b1
>
Looks like a sane solution. Thank you for following up.
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [dm-devel] md: dm-writeback: add __noreturn to BUG-ging function
@ 2020-11-18 16:35 ` Christian Borntraeger
0 siblings, 0 replies; 20+ messages in thread
From: Christian Borntraeger @ 2020-11-18 16:35 UTC (permalink / raw)
To: Mike Snitzer, Randy Dunlap
Cc: linux-s390, Vasily Gorbik, Heiko Carstens, linux-kernel,
dm-devel, Mikulas Patocka, Alasdair Kergon
On 18.11.20 17:07, Mike Snitzer wrote:
> On Wed, Nov 18 2020 at 10:49am -0500,
> Mike Snitzer <snitzer@redhat.com> wrote:
>
>> I don't think my suggestion will help.. given it'd still leave
>> persistent_memory_claim() without a return statement.
>>
>> Think it worthwhile to just add a dummy 'return 0;' after the BUG().
>
> Decided to go with this, now staged for 5.11:
> https://git.kernel.org/pub/scm/linux/kernel/git/device-mapper/linux-dm.git/commit/?h=dm-5.11&id=a1e4865b4dda7071f3707f7e551289ead66e38b1
>
Looks like a sane solution. Thank you for following up.
--
dm-devel mailing list
dm-devel@redhat.com
https://www.redhat.com/mailman/listinfo/dm-devel
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: md: dm-writeback: add __noreturn to BUG-ging function
2020-11-18 16:35 ` [dm-devel] " Christian Borntraeger
@ 2020-11-18 16:38 ` Randy Dunlap
-1 siblings, 0 replies; 20+ messages in thread
From: Randy Dunlap @ 2020-11-18 16:38 UTC (permalink / raw)
To: Christian Borntraeger, Mike Snitzer
Cc: linux-kernel, Mikulas Patocka, Alasdair Kergon, dm-devel,
Heiko Carstens, Vasily Gorbik, linux-s390
On 11/18/20 8:35 AM, Christian Borntraeger wrote:
>
>
> On 18.11.20 17:07, Mike Snitzer wrote:
>> On Wed, Nov 18 2020 at 10:49am -0500,
>> Mike Snitzer <snitzer@redhat.com> wrote:
>>
>>> I don't think my suggestion will help.. given it'd still leave
>>> persistent_memory_claim() without a return statement.
>>>
>>> Think it worthwhile to just add a dummy 'return 0;' after the BUG().
>>
>> Decided to go with this, now staged for 5.11:
>> https://git.kernel.org/pub/scm/linux/kernel/git/device-mapper/linux-dm.git/commit/?h=dm-5.11&id=a1e4865b4dda7071f3707f7e551289ead66e38b1
>>
>
> Looks like a sane solution. Thank you for following up.
Yes, thanks again.
--
~Randy
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [dm-devel] md: dm-writeback: add __noreturn to BUG-ging function
@ 2020-11-18 16:38 ` Randy Dunlap
0 siblings, 0 replies; 20+ messages in thread
From: Randy Dunlap @ 2020-11-18 16:38 UTC (permalink / raw)
To: Christian Borntraeger, Mike Snitzer
Cc: linux-s390, Vasily Gorbik, Heiko Carstens, linux-kernel,
dm-devel, Mikulas Patocka, Alasdair Kergon
On 11/18/20 8:35 AM, Christian Borntraeger wrote:
>
>
> On 18.11.20 17:07, Mike Snitzer wrote:
>> On Wed, Nov 18 2020 at 10:49am -0500,
>> Mike Snitzer <snitzer@redhat.com> wrote:
>>
>>> I don't think my suggestion will help.. given it'd still leave
>>> persistent_memory_claim() without a return statement.
>>>
>>> Think it worthwhile to just add a dummy 'return 0;' after the BUG().
>>
>> Decided to go with this, now staged for 5.11:
>> https://git.kernel.org/pub/scm/linux/kernel/git/device-mapper/linux-dm.git/commit/?h=dm-5.11&id=a1e4865b4dda7071f3707f7e551289ead66e38b1
>>
>
> Looks like a sane solution. Thank you for following up.
Yes, thanks again.
--
~Randy
--
dm-devel mailing list
dm-devel@redhat.com
https://www.redhat.com/mailman/listinfo/dm-devel
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: md: dm-writeback: add __noreturn to BUG-ging function
2020-11-18 16:07 ` [dm-devel] " Mike Snitzer
@ 2020-11-18 21:24 ` Mikulas Patocka
-1 siblings, 0 replies; 20+ messages in thread
From: Mikulas Patocka @ 2020-11-18 21:24 UTC (permalink / raw)
To: Mike Snitzer
Cc: Randy Dunlap, Christian Borntraeger, linux-kernel,
Alasdair Kergon, dm-devel, Heiko Carstens, Vasily Gorbik,
linux-s390
On Wed, 18 Nov 2020, Mike Snitzer wrote:
> On Wed, Nov 18 2020 at 10:49am -0500,
> Mike Snitzer <snitzer@redhat.com> wrote:
>
> > I don't think my suggestion will help.. given it'd still leave
> > persistent_memory_claim() without a return statement.
> >
> > Think it worthwhile to just add a dummy 'return 0;' after the BUG().
>
> Decided to go with this, now staged for 5.11:
> https://git.kernel.org/pub/scm/linux/kernel/git/device-mapper/linux-dm.git/commit/?h=dm-5.11&id=a1e4865b4dda7071f3707f7e551289ead66e38b1
Hi
I would just use "return -EOPNOTSUPP;" and drop the "#ifdef
DM_WRITECACHE_HAS_PMEM" that you added.
That BUG/return -EOPNOTSUPP code can't happen at all - if
DM_WRITECACHE_HAS_PMEM is not defined, WC_MODE_PMEM(wc) always returns
false - so persistent_memory_claim and BUG() can't ever be called. And if
it can't be called, you don't need to add a code that prints an error in
that case.
If we don't have DM_WRITECACHE_HAS_PMEM, the compiler optimizer will
remove all the code guarded with if (WC_MODE_PMEM(wc)) as unreachable.
Mikulas
From: Mikulas Patocka <mpatocka@redhat.com>
Subject: [PATCH] dm writecache: remove BUG() and fail gracefully insteadfor-nextdm-5.11
Building on arch/s390/ results in this build error:
cc1: some warnings being treated as errors
../drivers/md/dm-writecache.c: In function 'persistent_memory_claim':
../drivers/md/dm-writecache.c:323:1: error: no return statement in function returning non-void [-Werror=return-type]
Fix this by replacing the BUG() with a -EOPNOTSUPP return.
Fixes: 48debafe4f2f ("dm: add writecache target")
Cc: stable@vger.kernel.org # v4.18+
Reported-by: Randy Dunlap <rdunlap@infradead.org>
Signed-off-by: Mikulas Patocka <mpatocka@redhat.com>
Index: linux-2.6/drivers/md/dm-writecache.c
===================================================================
--- linux-2.6.orig/drivers/md/dm-writecache.c
+++ linux-2.6/drivers/md/dm-writecache.c
@@ -319,7 +319,7 @@ err1:
#else
static int persistent_memory_claim(struct dm_writecache *wc)
{
- BUG();
+ return -EOPNOTSUPP;
}
#endif
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [dm-devel] md: dm-writeback: add __noreturn to BUG-ging function
@ 2020-11-18 21:24 ` Mikulas Patocka
0 siblings, 0 replies; 20+ messages in thread
From: Mikulas Patocka @ 2020-11-18 21:24 UTC (permalink / raw)
To: Mike Snitzer
Cc: linux-s390, Vasily Gorbik, Heiko Carstens, Randy Dunlap,
linux-kernel, Christian Borntraeger, dm-devel, Alasdair Kergon
On Wed, 18 Nov 2020, Mike Snitzer wrote:
> On Wed, Nov 18 2020 at 10:49am -0500,
> Mike Snitzer <snitzer@redhat.com> wrote:
>
> > I don't think my suggestion will help.. given it'd still leave
> > persistent_memory_claim() without a return statement.
> >
> > Think it worthwhile to just add a dummy 'return 0;' after the BUG().
>
> Decided to go with this, now staged for 5.11:
> https://git.kernel.org/pub/scm/linux/kernel/git/device-mapper/linux-dm.git/commit/?h=dm-5.11&id=a1e4865b4dda7071f3707f7e551289ead66e38b1
Hi
I would just use "return -EOPNOTSUPP;" and drop the "#ifdef
DM_WRITECACHE_HAS_PMEM" that you added.
That BUG/return -EOPNOTSUPP code can't happen at all - if
DM_WRITECACHE_HAS_PMEM is not defined, WC_MODE_PMEM(wc) always returns
false - so persistent_memory_claim and BUG() can't ever be called. And if
it can't be called, you don't need to add a code that prints an error in
that case.
If we don't have DM_WRITECACHE_HAS_PMEM, the compiler optimizer will
remove all the code guarded with if (WC_MODE_PMEM(wc)) as unreachable.
Mikulas
From: Mikulas Patocka <mpatocka@redhat.com>
Subject: [PATCH] dm writecache: remove BUG() and fail gracefully insteadfor-nextdm-5.11
Building on arch/s390/ results in this build error:
cc1: some warnings being treated as errors
../drivers/md/dm-writecache.c: In function 'persistent_memory_claim':
../drivers/md/dm-writecache.c:323:1: error: no return statement in function returning non-void [-Werror=return-type]
Fix this by replacing the BUG() with a -EOPNOTSUPP return.
Fixes: 48debafe4f2f ("dm: add writecache target")
Cc: stable@vger.kernel.org # v4.18+
Reported-by: Randy Dunlap <rdunlap@infradead.org>
Signed-off-by: Mikulas Patocka <mpatocka@redhat.com>
Index: linux-2.6/drivers/md/dm-writecache.c
===================================================================
--- linux-2.6.orig/drivers/md/dm-writecache.c
+++ linux-2.6/drivers/md/dm-writecache.c
@@ -319,7 +319,7 @@ err1:
#else
static int persistent_memory_claim(struct dm_writecache *wc)
{
- BUG();
+ return -EOPNOTSUPP;
}
#endif
--
dm-devel mailing list
dm-devel@redhat.com
https://www.redhat.com/mailman/listinfo/dm-devel
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: md: dm-writeback: add __noreturn to BUG-ging function
2020-11-18 21:24 ` [dm-devel] " Mikulas Patocka
@ 2020-11-20 14:08 ` Mike Snitzer
-1 siblings, 0 replies; 20+ messages in thread
From: Mike Snitzer @ 2020-11-20 14:08 UTC (permalink / raw)
To: Mikulas Patocka
Cc: Randy Dunlap, Christian Borntraeger, linux-kernel,
Alasdair Kergon, dm-devel, Heiko Carstens, Vasily Gorbik,
linux-s390
On Wed, Nov 18 2020 at 4:24pm -0500,
Mikulas Patocka <mpatocka@redhat.com> wrote:
>
>
> On Wed, 18 Nov 2020, Mike Snitzer wrote:
>
> > On Wed, Nov 18 2020 at 10:49am -0500,
> > Mike Snitzer <snitzer@redhat.com> wrote:
> >
> > > I don't think my suggestion will help.. given it'd still leave
> > > persistent_memory_claim() without a return statement.
> > >
> > > Think it worthwhile to just add a dummy 'return 0;' after the BUG().
> >
> > Decided to go with this, now staged for 5.11:
> > https://git.kernel.org/pub/scm/linux/kernel/git/device-mapper/linux-dm.git/commit/?h=dm-5.11&id=a1e4865b4dda7071f3707f7e551289ead66e38b1
>
> Hi
>
> I would just use "return -EOPNOTSUPP;" and drop the "#ifdef
> DM_WRITECACHE_HAS_PMEM" that you added.
>
> That BUG/return -EOPNOTSUPP code can't happen at all - if
> DM_WRITECACHE_HAS_PMEM is not defined, WC_MODE_PMEM(wc) always returns
> false - so persistent_memory_claim and BUG() can't ever be called. And if
> it can't be called, you don't need to add a code that prints an error in
> that case.
>
> If we don't have DM_WRITECACHE_HAS_PMEM, the compiler optimizer will
> remove all the code guarded with if (WC_MODE_PMEM(wc)) as unreachable.
>
> Mikulas
Fair enough.
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [dm-devel] md: dm-writeback: add __noreturn to BUG-ging function
@ 2020-11-20 14:08 ` Mike Snitzer
0 siblings, 0 replies; 20+ messages in thread
From: Mike Snitzer @ 2020-11-20 14:08 UTC (permalink / raw)
To: Mikulas Patocka
Cc: linux-s390, Vasily Gorbik, Heiko Carstens, Randy Dunlap,
linux-kernel, Christian Borntraeger, dm-devel, Alasdair Kergon
On Wed, Nov 18 2020 at 4:24pm -0500,
Mikulas Patocka <mpatocka@redhat.com> wrote:
>
>
> On Wed, 18 Nov 2020, Mike Snitzer wrote:
>
> > On Wed, Nov 18 2020 at 10:49am -0500,
> > Mike Snitzer <snitzer@redhat.com> wrote:
> >
> > > I don't think my suggestion will help.. given it'd still leave
> > > persistent_memory_claim() without a return statement.
> > >
> > > Think it worthwhile to just add a dummy 'return 0;' after the BUG().
> >
> > Decided to go with this, now staged for 5.11:
> > https://git.kernel.org/pub/scm/linux/kernel/git/device-mapper/linux-dm.git/commit/?h=dm-5.11&id=a1e4865b4dda7071f3707f7e551289ead66e38b1
>
> Hi
>
> I would just use "return -EOPNOTSUPP;" and drop the "#ifdef
> DM_WRITECACHE_HAS_PMEM" that you added.
>
> That BUG/return -EOPNOTSUPP code can't happen at all - if
> DM_WRITECACHE_HAS_PMEM is not defined, WC_MODE_PMEM(wc) always returns
> false - so persistent_memory_claim and BUG() can't ever be called. And if
> it can't be called, you don't need to add a code that prints an error in
> that case.
>
> If we don't have DM_WRITECACHE_HAS_PMEM, the compiler optimizer will
> remove all the code guarded with if (WC_MODE_PMEM(wc)) as unreachable.
>
> Mikulas
Fair enough.
--
dm-devel mailing list
dm-devel@redhat.com
https://www.redhat.com/mailman/listinfo/dm-devel
^ permalink raw reply [flat|nested] 20+ messages in thread
end of thread, other threads:[~2020-11-20 14:09 UTC | newest]
Thread overview: 20+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-11-13 22:52 [PATCH] md: dm-writeback: add __noreturn to BUG-ging function Randy Dunlap
2020-11-13 22:52 ` [dm-devel] " Randy Dunlap
2020-11-16 7:30 ` Christian Borntraeger
2020-11-16 7:30 ` [dm-devel] " Christian Borntraeger
2020-11-16 23:00 ` Randy Dunlap
2020-11-16 23:00 ` [dm-devel] " Randy Dunlap
2020-11-17 16:31 ` Mike Snitzer
2020-11-17 16:31 ` [dm-devel] " Mike Snitzer
2020-11-18 15:49 ` Mike Snitzer
2020-11-18 15:49 ` [dm-devel] " Mike Snitzer
2020-11-18 16:07 ` Mike Snitzer
2020-11-18 16:07 ` [dm-devel] " Mike Snitzer
2020-11-18 16:35 ` Christian Borntraeger
2020-11-18 16:35 ` [dm-devel] " Christian Borntraeger
2020-11-18 16:38 ` Randy Dunlap
2020-11-18 16:38 ` [dm-devel] " Randy Dunlap
2020-11-18 21:24 ` Mikulas Patocka
2020-11-18 21:24 ` [dm-devel] " Mikulas Patocka
2020-11-20 14:08 ` Mike Snitzer
2020-11-20 14:08 ` [dm-devel] " Mike Snitzer
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.