linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] drivers/block/drbd: remove erroneous semicolon
@ 2013-03-27  6:56 Chen Gang
  2013-04-03  8:00 ` Chen Gang
  0 siblings, 1 reply; 6+ messages in thread
From: Chen Gang @ 2013-03-27  6:56 UTC (permalink / raw)
  To: drbd-dev; +Cc: drbd-user, linux-kernel


  need remove it, or always return -EBUSY.
  it is found by compiler with 'EXTRA_CFLAG=-W'.

Signed-off-by: Chen Gang <gang.chen@asianux.com>
---
 drivers/block/drbd/drbd_actlog.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/block/drbd/drbd_actlog.c b/drivers/block/drbd/drbd_actlog.c
index 6afe173..6608076 100644
--- a/drivers/block/drbd/drbd_actlog.c
+++ b/drivers/block/drbd/drbd_actlog.c
@@ -389,7 +389,7 @@ int drbd_al_begin_io_nonblock(struct drbd_conf *mdev, struct drbd_interval *i)
 		if (unlikely(tmp != NULL)) {
 			struct bm_extent  *bm_ext = lc_entry(tmp, struct bm_extent, lce);
 			if (test_bit(BME_NO_WRITES, &bm_ext->flags)) {
-				if (!test_and_set_bit(BME_PRIORITY, &bm_ext->flags));
+				if (!test_and_set_bit(BME_PRIORITY, &bm_ext->flags))
 					return -EBUSY;
 				return -EWOULDBLOCK;
 			}
-- 
1.7.7.6

^ permalink raw reply related	[flat|nested] 6+ messages in thread
* Re: [PATCH] drivers/block/drbd: remove erroneous semicolon
  2013-03-27  6:56 [PATCH] drivers/block/drbd: remove erroneous semicolon Chen Gang
@ 2013-04-03  8:00 ` Chen Gang
  2013-04-08 20:45   ` Lars Ellenberg
  0 siblings, 1 reply; 6+ messages in thread
From: Chen Gang @ 2013-04-03  8:00 UTC (permalink / raw)
  To: drbd-dev; +Cc: drbd-user, linux-kernel

Hello maintainers:

  when you have time, please help to check this patch whether is OK.

  thanks.

gchen.

On 2013年03月27日 14:56, Chen Gang wrote:
> 
>   need remove it, or always return -EBUSY.
>   it is found by compiler with 'EXTRA_CFLAG=-W'.
> 
> Signed-off-by: Chen Gang <gang.chen@asianux.com>
> ---
>  drivers/block/drbd/drbd_actlog.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/drivers/block/drbd/drbd_actlog.c b/drivers/block/drbd/drbd_actlog.c
> index 6afe173..6608076 100644
> --- a/drivers/block/drbd/drbd_actlog.c
> +++ b/drivers/block/drbd/drbd_actlog.c
> @@ -389,7 +389,7 @@ int drbd_al_begin_io_nonblock(struct drbd_conf *mdev, struct drbd_interval *i)
>  		if (unlikely(tmp != NULL)) {
>  			struct bm_extent  *bm_ext = lc_entry(tmp, struct bm_extent, lce);
>  			if (test_bit(BME_NO_WRITES, &bm_ext->flags)) {
> -				if (!test_and_set_bit(BME_PRIORITY, &bm_ext->flags));
> +				if (!test_and_set_bit(BME_PRIORITY, &bm_ext->flags))
>  					return -EBUSY;
>  				return -EWOULDBLOCK;
>  			}
> 


-- 
Chen Gang

Asianux Corporation

^ permalink raw reply	[flat|nested] 6+ messages in thread
* Re: [PATCH] drivers/block/drbd: remove erroneous semicolon
  2013-04-03  8:00 ` Chen Gang
@ 2013-04-08 20:45   ` Lars Ellenberg
  2013-04-09  1:22     ` Chen Gang
  0 siblings, 1 reply; 6+ messages in thread
From: Lars Ellenberg @ 2013-04-08 20:45 UTC (permalink / raw)
  To: Chen Gang; +Cc: drbd-dev, drbd-user, linux-kernel

On Wed, Apr 03, 2013 at 04:00:31PM +0800, Chen Gang wrote:
> Hello maintainers:
> 
>   when you have time, please help to check this patch whether is OK.
> 
>   thanks.

Thanks.

It has already been queued in:
http://git.kernel.dk/?p=linux-block.git;a=commit;h=0b6ef4164f50698eee536903d69d086add1a7889

> On 2013年03月27日 14:56, Chen Gang wrote:
> > 
> >   need remove it, or always return -EBUSY.
> >   it is found by compiler with 'EXTRA_CFLAG=-W'.
> > 
> > Signed-off-by: Chen Gang <gang.chen@asianux.com>
> > ---
> >  drivers/block/drbd/drbd_actlog.c |    2 +-
> >  1 files changed, 1 insertions(+), 1 deletions(-)
> > 
> > diff --git a/drivers/block/drbd/drbd_actlog.c b/drivers/block/drbd/drbd_actlog.c
> > index 6afe173..6608076 100644
> > --- a/drivers/block/drbd/drbd_actlog.c
> > +++ b/drivers/block/drbd/drbd_actlog.c
> > @@ -389,7 +389,7 @@ int drbd_al_begin_io_nonblock(struct drbd_conf *mdev, struct drbd_interval *i)
> >  		if (unlikely(tmp != NULL)) {
> >  			struct bm_extent  *bm_ext = lc_entry(tmp, struct bm_extent, lce);
> >  			if (test_bit(BME_NO_WRITES, &bm_ext->flags)) {
> > -				if (!test_and_set_bit(BME_PRIORITY, &bm_ext->flags));
> > +				if (!test_and_set_bit(BME_PRIORITY, &bm_ext->flags))
> >  					return -EBUSY;
> >  				return -EWOULDBLOCK;
> >  			}
> > 

^ permalink raw reply	[flat|nested] 6+ messages in thread
* Re: [PATCH] drivers/block/drbd: remove erroneous semicolon
  2013-04-08 20:45   ` Lars Ellenberg
@ 2013-04-09  1:22     ` Chen Gang
  2013-04-09 15:56       ` [Drbd-dev] " Lars Ellenberg
  0 siblings, 1 reply; 6+ messages in thread
From: Chen Gang @ 2013-04-09  1:22 UTC (permalink / raw)
  To: drbd-dev, drbd-user, linux-kernel

On 2013年04月09日 04:45, Lars Ellenberg wrote:
> On Wed, Apr 03, 2013 at 04:00:31PM +0800, Chen Gang wrote:
>> > Hello maintainers:
>> > 
>> >   when you have time, please help to check this patch whether is OK.
>> > 
>> >   thanks.
> Thanks.
> 
> It has already been queued in:
> http://git.kernel.dk/?p=linux-block.git;a=commit;h=0b6ef4164f50698eee536903d69d086add1a7889
> 

  ok thanks.

  it seems I am a little earlier than the link's patch.
    I found it on Wed 27 Mar 2013 14:56
    link's patch found it on Wed 27 Mar 2013 21:08

  ;-)


  but that's all right, it doesn't matter.

  thanks.

  :-)

gchen.


>> > On 2013年03月27日 14:56, Chen Gang wrote:
>>> > > 
>>> > >   need remove it, or always return -EBUSY.
>>> > >   it is found by compiler with 'EXTRA_CFLAG=-W'.
>>> > > 
>>> > > Signed-off-by: Chen Gang <gang.chen@asianux.com>
>>> > > ---
>>> > >  drivers/block/drbd/drbd_actlog.c |    2 +-
>>> > >  1 files changed, 1 insertions(+), 1 deletions(-)
>>> > > 
>>> > > diff --git a/drivers/block/drbd/drbd_actlog.c b/drivers/block/drbd/drbd_actlog.c
>>> > > index 6afe173..6608076 100644
>>> > > --- a/drivers/block/drbd/drbd_actlog.c
>>> > > +++ b/drivers/block/drbd/drbd_actlog.c
>>> > > @@ -389,7 +389,7 @@ int drbd_al_begin_io_nonblock(struct drbd_conf *mdev, struct drbd_interval *i)
>>> > >  		if (unlikely(tmp != NULL)) {
>>> > >  			struct bm_extent  *bm_ext = lc_entry(tmp, struct bm_extent, lce);
>>> > >  			if (test_bit(BME_NO_WRITES, &bm_ext->flags)) {
>>> > > -				if (!test_and_set_bit(BME_PRIORITY, &bm_ext->flags));
>>> > > +				if (!test_and_set_bit(BME_PRIORITY, &bm_ext->flags))
>>> > >  					return -EBUSY;
>>> > >  				return -EWOULDBLOCK;
>>> > >  			}
>>> > > 
> 


-- 
Chen Gang

Asianux Corporation

^ permalink raw reply	[flat|nested] 6+ messages in thread
* Re: [Drbd-dev] [PATCH] drivers/block/drbd: remove erroneous semicolon
  2013-04-09  1:22     ` Chen Gang
@ 2013-04-09 15:56       ` Lars Ellenberg
  2013-04-10  4:01         ` Chen Gang
  0 siblings, 1 reply; 6+ messages in thread
From: Lars Ellenberg @ 2013-04-09 15:56 UTC (permalink / raw)
  To: Chen Gang; +Cc: drbd-dev, drbd-user, linux-kernel

On Tue, Apr 09, 2013 at 09:22:40AM +0800, Chen Gang wrote:
> On 2013年04月09日 04:45, Lars Ellenberg wrote:
> > On Wed, Apr 03, 2013 at 04:00:31PM +0800, Chen Gang wrote:
> >> > Hello maintainers:
> >> > 
> >> >   when you have time, please help to check this patch whether is OK.
> >> > 
> >> >   thanks.
> > Thanks.
> > 
> > It has already been queued in:
> > http://git.kernel.dk/?p=linux-block.git;a=commit;h=0b6ef4164f50698eee536903d69d086add1a7889
> > 
> 
>   ok thanks.
> 
>   it seems I am a little earlier than the link's patch.
>     I found it on Wed 27 Mar 2013 14:56
>     link's patch found it on Wed 27 Mar 2013 21:08
> 

The original report (by kbuild test robot) is from the week before iirc
(don't have it anymore), the original commit in our drbd repo the day before.

But:

>   ;-)
> 
>   but that's all right, it doesn't matter.
> 
>   thanks.

 ;-)


^ permalink raw reply	[flat|nested] 6+ messages in thread
* Re: [Drbd-dev] [PATCH] drivers/block/drbd: remove erroneous semicolon
  2013-04-09 15:56       ` [Drbd-dev] " Lars Ellenberg
@ 2013-04-10  4:01         ` Chen Gang
  0 siblings, 0 replies; 6+ messages in thread
From: Chen Gang @ 2013-04-10  4:01 UTC (permalink / raw)
  To: drbd-dev, drbd-user, linux-kernel

On 2013年04月09日 23:56, Lars Ellenberg wrote:
> The original report (by kbuild test robot) is from the week before iirc
> (don't have it anymore), the original commit in our drbd repo the day before.


  really it is !!

  thanks.

  :-)



-- 
Chen Gang

Asianux Corporation

^ permalink raw reply	[flat|nested] 6+ messages in thread
end of thread, other threads:[~2013-04-10  4:01 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-03-27  6:56 [PATCH] drivers/block/drbd: remove erroneous semicolon Chen Gang
2013-04-03  8:00 ` Chen Gang
2013-04-08 20:45   ` Lars Ellenberg
2013-04-09  1:22     ` Chen Gang
2013-04-09 15:56       ` [Drbd-dev] " Lars Ellenberg
2013-04-10  4:01         ` Chen Gang

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).