From: Philipp Reisner <philipp.reisner@linbit.com>
To: drbd-dev@lists.linbit.com
Subject: Re: [Drbd-dev] 8.2.6 Peer disk state handling issue when attaching
Date: Thu, 30 Oct 2008 12:23:20 +0100 [thread overview]
Message-ID: <200810301223.20773.philipp.reisner@linbit.com> (raw)
In-Reply-To: <DA0E7D869C862D4095C265233CD1D41E0E3F55@EXNA.corp.stratus.com>
Am Montag 15 September 2008 20:26:15 schrieb Petrakis, Peter:
> Hi All,
>
> We've evolved the original patch to the following, which seems to solve
> our problem. Can you see anything wrong with it? Thanks.
>
> diff -r f9aa469f7044 drbd_main.c
> --- a/drbd_main.c Fri Sep 12 13:51:40 2008 -0400
> +++ b/drbd_main.c Mon Sep 15 10:04:48 2008 -0400
> @@ -765,7 +765,7 @@
> ns.conn = Connected;
> }
>
> - if (ns.conn != os.conn && ns.conn >= Connected &&
> + if (ns.conn >= Connected &&
> (ns.disk == Consistent || ns.disk == Outdated)) {
> switch(ns.conn) {
> case WFBitMapT:
> @@ -787,7 +787,7 @@
> WARN("Implicit set disk from Outdate to
> UpToDate\n");
> }
>
> - if (ns.conn != os.conn && ns.conn >= Connected &&
> + if (ns.conn >= Connected &&
> (ns.pdsk == Consistent || ns.pdsk == Outdated)) {
> switch(ns.conn) {
> case Connected:
>
>
Hi Peter,
There is nothing wrong with that patch. It is just my failure to follow
drbd-dev for the last month...
I have committed now nearly that code change upstream. You will
find it there soon as 00f2ce70e0daaa72775b3712863fb29ee99581f3
(in drbd-8.0) and from there it will be propagated to 8.2 of
course.
-phil
--
: Dipl-Ing Philipp Reisner
: LINBIT | Your Way to High Availability
: Tel: +43-1-8178292-50, Fax: +43-1-8178292-82
: http://www.linbit.com
DRBD(R) and LINBIT(R) are registered trademarks of LINBIT, Austria.
prev parent reply other threads:[~2008-10-30 11:23 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-08-12 3:52 [Drbd-dev] 8.2.6 Peer disk state handling issue when attaching Graham, Simon
2008-08-12 8:49 ` Lars Ellenberg
2008-09-15 18:26 ` Petrakis, Peter
2008-10-30 11:23 ` Philipp Reisner [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=200810301223.20773.philipp.reisner@linbit.com \
--to=philipp.reisner@linbit.com \
--cc=drbd-dev@lists.linbit.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox