Re: [PATCH] drivers/block/drbd: remove erroneous semicolon
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 >>> > > --- >>> > > 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 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] drivers/block/drbd: remove erroneous semicolon
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 > > --- > > 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; > > } > > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] drivers/block/drbd: remove erroneous semicolon
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 > --- > 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 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/