All of lore.kernel.org
 help / color / mirror / Atom feed
From: Sasha Levin <levinsasha928@gmail.com>
To: Linus Torvalds <torvalds@linux-foundation.org>
Cc: suresh.b.siddha@intel.com, Artem Bityutskiy <dedekind1@gmail.com>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	linux-mm <linux-mm@kvack.org>,
	linux-mtd@lists.infradead.org, Dave Jones <davej@redhat.com>,
	Andrew Morton <akpm@linux-foundation.org>,
	dwmw2@infradead.org
Subject: Re: mtd: kernel BUG at arch/x86/mm/pat.c:279!
Date: Fri, 28 Sep 2012 21:44:41 +0200	[thread overview]
Message-ID: <5065FE29.5080302@gmail.com> (raw)
In-Reply-To: <CA+55aFwDWcQv9DBCQhMrcK-zx75qBN11iQhq5dxt+orosmeXbg@mail.gmail.com>

On 09/28/2012 09:13 PM, Linus Torvalds wrote:
> On Fri, Sep 28, 2012 at 11:05 AM, Artem Bityutskiy <dedekind1@gmail.com> wrote:
>>
>> I am not the maintainer, but please, go ahead an push your fix. I do not
>> have time to test it myself and it does not look like anyone else in the
>> small mtd community does.
> 
> Grr. I told people that patch wasn't tested. I hadn't even *compiled*
> it. It has a typo ("inlint" instead of "inline" - so close).
> 
> Sasha said he had tested it, but nobody even mentioned this thing. Now
> I'm nervous. I had committed it in my tree and was just about to push
> it out when I decided that I should at least compile it despite the
> "tested-by".
> 
> Hmm? Now I really *really* want to know that it's been tested on
> actual hardware too. Sasha, what patch did you actually test? Did you
> just fix the "inlint" thing, or was there something else entirely?

I've just fixed the inlint thing since it was pretty trivial, I didn't bother commenting on it since I figured it would turn into
an actual patch from someone who could test it on actual hardware first.

Note that I've tested it on a KVM guest, and not on real hardware - so I'm not sure how much of a test that is.


Thanks,
Sasha

WARNING: multiple messages have this Message-ID (diff)
From: Sasha Levin <levinsasha928@gmail.com>
To: Linus Torvalds <torvalds@linux-foundation.org>
Cc: Artem Bityutskiy <dedekind1@gmail.com>,
	suresh.b.siddha@intel.com,
	Andrew Morton <akpm@linux-foundation.org>,
	dwmw2@infradead.org,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	linux-mtd@lists.infradead.org, linux-mm <linux-mm@kvack.org>,
	Dave Jones <davej@redhat.com>
Subject: Re: mtd: kernel BUG at arch/x86/mm/pat.c:279!
Date: Fri, 28 Sep 2012 21:44:41 +0200	[thread overview]
Message-ID: <5065FE29.5080302@gmail.com> (raw)
In-Reply-To: <CA+55aFwDWcQv9DBCQhMrcK-zx75qBN11iQhq5dxt+orosmeXbg@mail.gmail.com>

On 09/28/2012 09:13 PM, Linus Torvalds wrote:
> On Fri, Sep 28, 2012 at 11:05 AM, Artem Bityutskiy <dedekind1@gmail.com> wrote:
>>
>> I am not the maintainer, but please, go ahead an push your fix. I do not
>> have time to test it myself and it does not look like anyone else in the
>> small mtd community does.
> 
> Grr. I told people that patch wasn't tested. I hadn't even *compiled*
> it. It has a typo ("inlint" instead of "inline" - so close).
> 
> Sasha said he had tested it, but nobody even mentioned this thing. Now
> I'm nervous. I had committed it in my tree and was just about to push
> it out when I decided that I should at least compile it despite the
> "tested-by".
> 
> Hmm? Now I really *really* want to know that it's been tested on
> actual hardware too. Sasha, what patch did you actually test? Did you
> just fix the "inlint" thing, or was there something else entirely?

I've just fixed the inlint thing since it was pretty trivial, I didn't bother commenting on it since I figured it would turn into
an actual patch from someone who could test it on actual hardware first.

Note that I've tested it on a KVM guest, and not on real hardware - so I'm not sure how much of a test that is.


Thanks,
Sasha

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

WARNING: multiple messages have this Message-ID (diff)
From: Sasha Levin <levinsasha928@gmail.com>
To: Linus Torvalds <torvalds@linux-foundation.org>
Cc: Artem Bityutskiy <dedekind1@gmail.com>,
	suresh.b.siddha@intel.com,
	Andrew Morton <akpm@linux-foundation.org>,
	dwmw2@infradead.org,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	linux-mtd@lists.infradead.org, linux-mm <linux-mm@kvack.org>,
	Dave Jones <davej@redhat.com>
Subject: Re: mtd: kernel BUG at arch/x86/mm/pat.c:279!
Date: Fri, 28 Sep 2012 21:44:41 +0200	[thread overview]
Message-ID: <5065FE29.5080302@gmail.com> (raw)
In-Reply-To: <CA+55aFwDWcQv9DBCQhMrcK-zx75qBN11iQhq5dxt+orosmeXbg@mail.gmail.com>

On 09/28/2012 09:13 PM, Linus Torvalds wrote:
> On Fri, Sep 28, 2012 at 11:05 AM, Artem Bityutskiy <dedekind1@gmail.com> wrote:
>>
>> I am not the maintainer, but please, go ahead an push your fix. I do not
>> have time to test it myself and it does not look like anyone else in the
>> small mtd community does.
> 
> Grr. I told people that patch wasn't tested. I hadn't even *compiled*
> it. It has a typo ("inlint" instead of "inline" - so close).
> 
> Sasha said he had tested it, but nobody even mentioned this thing. Now
> I'm nervous. I had committed it in my tree and was just about to push
> it out when I decided that I should at least compile it despite the
> "tested-by".
> 
> Hmm? Now I really *really* want to know that it's been tested on
> actual hardware too. Sasha, what patch did you actually test? Did you
> just fix the "inlint" thing, or was there something else entirely?

I've just fixed the inlint thing since it was pretty trivial, I didn't bother commenting on it since I figured it would turn into
an actual patch from someone who could test it on actual hardware first.

Note that I've tested it on a KVM guest, and not on real hardware - so I'm not sure how much of a test that is.


Thanks,
Sasha


  reply	other threads:[~2012-09-28 19:44 UTC|newest]

Thread overview: 77+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-06-29  8:48 mtd: kernel BUG at arch/x86/mm/pat.c:279! Sasha Levin
2012-06-29  8:48 ` Sasha Levin
2012-07-30 11:00 ` Sasha Levin
2012-07-30 11:00   ` Sasha Levin
2012-07-30 11:00   ` Sasha Levin
2012-09-07 16:55 ` Sasha Levin
2012-09-07 16:55   ` Sasha Levin
2012-09-07 16:55   ` Sasha Levin
2012-09-07 18:14   ` Linus Torvalds
2012-09-07 18:14     ` Linus Torvalds
2012-09-07 18:14     ` Linus Torvalds
2012-09-07 22:42     ` Suresh Siddha
2012-09-07 22:42       ` Suresh Siddha
2012-09-07 22:42       ` Suresh Siddha
2012-09-07 23:09       ` Linus Torvalds
2012-09-07 23:09         ` Linus Torvalds
2012-09-07 23:09         ` Linus Torvalds
2012-09-07 23:54         ` Suresh Siddha
2012-09-07 23:54           ` Suresh Siddha
2012-09-07 23:54           ` Suresh Siddha
2012-09-08 19:57           ` Linus Torvalds
2012-09-08 19:57             ` Linus Torvalds
2012-09-09 14:56             ` Suresh Siddha
2012-09-09 14:56               ` Suresh Siddha
2012-09-09 14:56               ` Suresh Siddha
2012-09-09 15:31               ` Linus Torvalds
2012-09-09 15:31                 ` Linus Torvalds
2012-09-09 15:31                 ` Linus Torvalds
2012-09-09 17:01                 ` H. Peter Anvin
2012-09-09 17:01                   ` H. Peter Anvin
2012-09-09 17:01                   ` H. Peter Anvin
2012-09-12 10:50               ` Sasha Levin
2012-09-12 10:50                 ` Sasha Levin
2012-09-12 10:50                 ` Sasha Levin
2012-09-12 10:56                 ` Sasha Levin
2012-09-12 10:56                   ` Sasha Levin
2012-09-12 10:56                   ` Sasha Levin
2012-09-28  9:00                   ` Sasha Levin
2012-09-28  9:00                     ` Sasha Levin
2012-09-28  9:00                     ` Sasha Levin
2012-09-28 16:44                     ` Linus Torvalds
2012-09-28 16:44                       ` Linus Torvalds
2012-09-28 16:44                       ` Linus Torvalds
2012-09-28 18:05                       ` Artem Bityutskiy
2012-09-28 18:05                         ` Artem Bityutskiy
2012-09-28 19:13                         ` Linus Torvalds
2012-09-28 19:13                           ` Linus Torvalds
2012-09-28 19:13                           ` Linus Torvalds
2012-09-28 19:44                           ` Sasha Levin [this message]
2012-09-28 19:44                             ` Sasha Levin
2012-09-28 19:44                             ` Sasha Levin
2012-09-28 19:04                       ` David Woodhouse
2012-09-28 19:04                         ` David Woodhouse
2012-09-28 19:15                         ` richard -rw- weinberger
2012-09-28 19:15                           ` richard -rw- weinberger
2012-09-28 19:15                           ` richard -rw- weinberger
2012-09-28 19:18                           ` richard -rw- weinberger
2012-09-28 19:18                             ` richard -rw- weinberger
2012-09-28 19:18                             ` richard -rw- weinberger
2012-09-29 16:11                         ` David Woodhouse
2012-09-29 16:11                           ` David Woodhouse
2012-09-29 16:34                           ` David Woodhouse
2012-09-29 16:34                             ` David Woodhouse
2012-09-09 16:56             ` H. Peter Anvin
2012-09-09 16:56               ` H. Peter Anvin
2012-09-09 16:56               ` H. Peter Anvin
2012-09-09 19:04               ` David Woodhouse
2012-09-09 19:04                 ` David Woodhouse
2012-09-09 20:33                 ` H. Peter Anvin
2012-09-09 20:33                   ` H. Peter Anvin
2012-09-09 20:33                   ` H. Peter Anvin
2012-09-10  5:17               ` Sasha Levin
2012-09-10  5:17                 ` Sasha Levin
2012-09-10  5:17                 ` Sasha Levin
2012-09-08  8:10         ` Sasha Levin
2012-09-08  8:10           ` Sasha Levin
2012-09-08  8:10           ` Sasha Levin

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=5065FE29.5080302@gmail.com \
    --to=levinsasha928@gmail.com \
    --cc=akpm@linux-foundation.org \
    --cc=davej@redhat.com \
    --cc=dedekind1@gmail.com \
    --cc=dwmw2@infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mm@kvack.org \
    --cc=linux-mtd@lists.infradead.org \
    --cc=suresh.b.siddha@intel.com \
    --cc=torvalds@linux-foundation.org \
    /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 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.