Linux PARISC architecture development
 help / color / mirror / Atom feed
From: "Joel Soete" <soete.joel@tiscali.be>
To: "Grant Grundler" <grundler@parisc-linux.org>
Cc: "Matthew Wilcox" <willy@debian.org>,
	parisc-linux@lists.parisc-linux.org,
	"Grant Grundler" <grundler@parisc-linux.org>
Subject: Re: [parisc-linux] fdisk problems 2.4 <-> 2.6
Date: Mon, 10 Nov 2003 10:51:32 +0100	[thread overview]
Message-ID: <3F969FFE00009D32@ocpmta2.freegates.net> (raw)
In-Reply-To: <20031110044556.GB16478@colo.lackof.org>

That is Ok for me, Do you still have the patch to revert it?

Joel

>-- Original Message --
>From: Grant Grundler <grundler@parisc-linux.org>
>To: Joel Soete <soete.joel@tiscali.be>
>Cc: Matthew Wilcox <willy@debian.org>,
>	parisc-linux@lists.parisc-linux.org,
>	Grant Grundler <grundler@parisc-linux.org>
>Date: Sun, 9 Nov 2003 21:45:56 -0700
>Subject: Re: [parisc-linux] fdisk problems 2.4 <-> 2.6
>
>
>On Sun, Nov 09, 2003 at 06:59:48PM +0000, Joel Soete wrote:
> Just for remainder here is the alignement of uaccess.h (i just finished

> to test on my c110 :)):
> ----------><----------
> --- uaccess.h.orig      2004-04-20 21:03:59.000000000 +0200

>> +++ uaccess.h   2003-11-09 18:47:06.000000000 +0100
> @@ -42,8 +42,8 @@
>  #if BITS_PER_LONG == 32
>  #define LDD_KERNEL(ptr) __get_kernel_bad();
>  #define LDD_USER(ptr) __get_user_bad();
> -#define STD_KERNEL(x, ptr) __put_kernel_asm64((u32)x,
>tr)
> -#define STD_USER(x, ptr) __put_user_asm64((u32)x,ptr)
> +#define STD_KERNEL(x, ptr) __put_kernel_asm64((u64)x,ptr)
> +#define STD_USER(x, ptr) __put_user_asm64((u64)x,ptr)

willy just removed the (u32) cast and things seem to work for him.

>Do you need the (u64) cast?

...
> -static inline void __put_kernel_asm64(u64 x, void *ptr)
...
> +#define __put_kernel_asm64(x, ptr)                         \
...

Why replace the static inline with a macro?
static inline provides type checki
>g and
type "coercion" when it's not exactly right.

> +       __asm__ __volatile__ (                              \
> +               "\n1:\tstw\t%2,0(%1)\n"                     \
> +               "2:\tstw\t%R2,4(%1)\n"                      \


>hat is "%R2" intended to be?
Randolph pointed out this is broken before and I was wrong
to commit this chunk to the 2.4 tree (I'm working on
removing it now).

thanks,
grant
_______________________________________________
parisc-linux mailing l
>st
parisc-linux@lists.parisc-linux.org
http://lists.parisc-linux.org/mailman/listinfo/parisc-linux



-------------------------------------------------------------------------
Tiscali ADSL: 3 mois GRATUITS! L'Internet rapide, c'est pour tout le monde.

http://reg.tiscali.be/default.asp?lg=fr

  parent reply	other threads:[~2003-11-10  9:51 UTC|newest]

Thread overview: 23+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2003-11-05 20:21 [parisc-linux] fdisk problems 2.4 <-> 2.6 Ruediger Scholz
2003-11-06 22:29 ` Matthew Wilcox
2003-11-07 17:52   ` Joel Soete
2003-11-08  1:59     ` Grant Grundler
2003-11-08 10:59       ` Joel Soete
2003-11-08 19:48         ` Grant Grundler
2003-11-08 22:15           ` Joel Soete
2003-11-09 13:50             ` Joel Soete
2003-11-09 17:57               ` Matthew Wilcox
2003-11-09 18:59                 ` Joel Soete
2003-11-10  4:45                   ` Grant Grundler
2003-11-10  4:55                     ` Grant Grundler
2003-11-10  5:01                     ` John David Anglin
2003-11-10  5:07                     ` Randolph Chung
2003-11-10  5:25                       ` John David Anglin
2003-11-10  9:16                         ` Joel Soete
2003-11-10  9:47                         ` Joel Soete
2003-11-10 18:35                       ` Joel Soete
2003-11-11 11:21                         ` Joel Soete
2003-11-10  9:51                     ` Joel Soete [this message]
2003-11-10 16:58                       ` Grant Grundler
2003-11-07 20:32   ` Ruediger Scholz
2003-11-10 21:58 ` Ruediger Scholz

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=3F969FFE00009D32@ocpmta2.freegates.net \
    --to=soete.joel@tiscali.be \
    --cc=grundler@parisc-linux.org \
    --cc=parisc-linux@lists.parisc-linux.org \
    --cc=willy@debian.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox