CEPH filesystem development
 help / color / mirror / Atom feed
From: Josh Durgin <josh.durgin@inktank.com>
To: Stefan Priebe <s.priebe@profihost.ag>
Cc: "ceph-devel@vger.kernel.org" <ceph-devel@vger.kernel.org>
Subject: Re: librbd discard bug problems -> i got it
Date: Mon, 19 Nov 2012 14:48:53 -0800	[thread overview]
Message-ID: <50AAB755.7060805@inktank.com> (raw)
In-Reply-To: <50AAB682.7070506@profihost.ag>

On 11/19/2012 02:45 PM, Stefan Priebe wrote:
> Hi Josh,
>
> how can i log in AioRequest.h?
>
> I need to add some debugging into:
>      void complete(int r)
>      {
>        if (should_complete(r)) {
>          if (m_hide_enoent && r == -ENOENT)
>            r = 0;
>          m_completion->complete(r);
>          delete this;
>        }
>      }

Move the implementation of the method out of the header and into
AioRequest.cc. Then you can use ldout like in the rest of that file.

Josh

> Am 19.11.2012 23:38, schrieb Josh Durgin:
>> On 11/19/2012 01:19 PM, Stefan Priebe wrote:
>>> Hello Josh,
>>>
>>> after digging three days around i got it.
>>
>> That's great!
>>
>>> The problem is in aio_discard in internal.cc. The i/o fails when AioZero
>>> or AioTruncate is used.
>>
>> Interesting, I wouldn't have expected the bug to be there. Is
>> it failing with a different error code, or just taking too long
>> so the guest kernel tries to cancel it?
>>
>>> It works fine with AioRemove. It seems to depend on overlapping.
>>> Hopefully i'm able to provide a patch this nicht.
>>
>> Looking forward to it!
>>
>> Josh
> --
> To unsubscribe from this list: send the line "unsubscribe ceph-devel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html


  reply	other threads:[~2012-11-19 22:49 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-11-19 21:19 librbd discard bug problems -> i got it Stefan Priebe
2012-11-19 22:38 ` Josh Durgin
2012-11-19 22:45   ` Stefan Priebe
2012-11-19 22:48     ` Josh Durgin [this message]
2012-11-19 23:16   ` Stefan Priebe
2012-11-19 23:33     ` Josh Durgin
2012-11-19 23:42       ` Stefan Priebe
2012-11-19 23:47         ` Josh Durgin
2012-11-20  0:00           ` Stefan Priebe
2012-11-20  0:28             ` Josh Durgin
2012-11-20  9:50               ` Stefan Priebe - Profihost AG
2012-11-20 12:46               ` librbd discard bug problems -> i REALLY " Stefan Priebe - Profihost AG
2012-11-20 17:09                 ` Josh Durgin

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=50AAB755.7060805@inktank.com \
    --to=josh.durgin@inktank.com \
    --cc=ceph-devel@vger.kernel.org \
    --cc=s.priebe@profihost.ag \
    /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