All of lore.kernel.org
 help / color / mirror / Atom feed
From: "H. Peter Anvin" <hpa@zytor.com>
To: Roland Kletzing <devzero@web.de>, kernel-team@lists.ubuntu.com
Cc: Linux Kernel Mailing List <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH] - add "fixpae" bootparam to fix/workaround #930447
Date: Tue, 26 Nov 2013 15:33:45 -0800	[thread overview]
Message-ID: <52952FD9.1060906@zytor.com> (raw)
In-Reply-To: <trinity-1045dd8c-e43f-4f8c-94e4-c0f0f15f611d-1385507362267@3capp-webde-bs36>

On 11/26/2013 03:09 PM, Roland Kletzing wrote:
> Hello kernel-team, 
> 
> regarding https://bugs.launchpad.net/ubuntu/+source/syslinux/+bug/930447 , i have developed a simple patch to (hopefully) provide an easy and elegant workaround/fix for the issue. I assume some hundred users (if not some thousand, as entries in bugreports are only the tip of an iceberg) are affected by this issue.
> 
> Please take a review and feel free to merge.
> 
> As defaulting to PAE kernels is specific for some distros only, i think this is no patch for mainline (yet).
> 
> I did not often do patches, nor am i a good programmer - so sorry if the patch or the submission style is not perfect.
> 
> regards
> Roland

No, this is not an acceptable patch per se.

The right way to do this is to have the option, if it is found, simply
set the bit in the CPU array.  However, it is important to verify that
we are on the affected type of CPU, specifically the affected Pentium Ms.

x86 patches should be sent to <x86@kernel.org> or equivalently myself,
<mingo@kernel.org> and <tglx@linutronix.de> and Cc:
<linux-kernel@vger.kernel.org>.  I also encourage you to read the file
Documentation/SubmittingPatches before trying again.

	-hpa


       reply	other threads:[~2013-11-26 23:33 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <trinity-1045dd8c-e43f-4f8c-94e4-c0f0f15f611d-1385507362267@3capp-webde-bs36>
2013-11-26 23:33 ` H. Peter Anvin [this message]
2013-11-27 20:41   ` [PATCH] - add "fixpae" bootparam to fix/workaround #930447 Roland
2013-11-29 12:49     ` Andy Whitcroft

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=52952FD9.1060906@zytor.com \
    --to=hpa@zytor.com \
    --cc=devzero@web.de \
    --cc=kernel-team@lists.ubuntu.com \
    --cc=linux-kernel@vger.kernel.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.