netfilter-devel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Sachin Sanap <sachin.sanap.1gb@gmail.com>
To: Jan Engelhardt <jengelh@medozas.de>
Cc: Netfilter Developer Mailing List
	<netfilter-devel@vger.kernel.org>,
	Lennart Buytenhek <buytenh@wantstofly.org>,
	Bart de Schuymer <bdschuym@pandora.be>,
	Patrick McHardy <kaber@trash.net>, Peter Volkov <pva@gentoo.org>
Subject: Re: Fwd: ebtables broken for kernel above 2.6.28
Date: Fri, 19 Jun 2009 11:36:48 +0530	[thread overview]
Message-ID: <17923cc80906182306t2ae2c7dav3611f61b5c27d0b9@mail.gmail.com> (raw)
In-Reply-To: <alpine.LSU.2.00.0906141921430.31733@fbirervta.pbzchgretzou.qr>

On Sun, Jun 14, 2009 at 11:12 PM, Jan Engelhardt<jengelh@medozas.de> wrote:
> On Friday 2009-05-29 13:14, sachin sanap wrote:
>
>>Sorry dint inform you earlier that iam on 32bit ARM machine.
>>The real problem is the kernel is computing the alignment on
>>__alignof__(struct _xt_align) and the userspace ebtables is computing
>>it on __alignof__(struct ebt_entry_target) , which are two different
>>structs.
>>
>>I have the following patch for ebtables-v2.0.8-2 that fixes this.
>>@@ -37,7 +38,7 @@
>> #define EXEC_STYLE_DAEMON      1
>>
>> #ifndef EBT_MIN_ALIGN
>>-#define EBT_MIN_ALIGN (__alignof__(struct ebt_entry_target))
>>+#define EBT_MIN_ALIGN (__alignof__(struct _xt_align))
>> #endif
>> #define EBT_ALIGN(s) (((s) + (EBT_MIN_ALIGN-1)) & ~(EBT_MIN_ALIGN-1))
>> #define ERRORMSG_MAXLEN 128
>
> Can you run this, just to give me a better understanding of ARM32?
>
>
> #include <stdio.h>
> #include <stdint.h>
> #define p(t) \
>        printf(#t ": %zu\n", __alignof__(t));
>
> int main(void)
> {
>        p(uint8_t);
>        p(uint16_t);
>        p(uint32_t);
>        p(uint64_t);
>        p(float);
>        p(double);
>        p(void *);
>        p(void (*)(void));
> }
>
Here is the output from ARM

uint8_t: 1
uint16_t: 2
uint32_t: 4
uint64_t: 8
float: 4
double: 8
void *: 4
void (*)(void): 4

I have CONFIG_AEABI=y

-Sachin Sanap

  reply	other threads:[~2009-06-19  6:06 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <17923cc80905280124t4ed6cf7m87f91e4444146697@mail.gmail.com>
     [not found] ` <4A1EB783.4090801@pandora.be>
2009-05-29  5:51   ` Fwd: ebtables broken for kernel above 2.6.28 sachin sanap
2009-05-29 10:02     ` Jan Engelhardt
2009-05-29 11:14       ` sachin sanap
2009-05-29 16:20         ` Bart De Schuymer
2009-05-29 17:16           ` Jan Engelhardt
2009-06-02 12:20             ` Patrick McHardy
2009-06-05 13:35               ` Jan Engelhardt
2009-06-05 13:38                 ` Patrick McHardy
2009-06-14 17:42         ` Fwd: " Jan Engelhardt
2009-06-19  6:06           ` Sachin Sanap [this message]
2009-06-14 15:47     ` Peter Volkov
2009-06-14 16:54       ` Bart De Schuymer

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=17923cc80906182306t2ae2c7dav3611f61b5c27d0b9@mail.gmail.com \
    --to=sachin.sanap.1gb@gmail.com \
    --cc=bdschuym@pandora.be \
    --cc=buytenh@wantstofly.org \
    --cc=jengelh@medozas.de \
    --cc=kaber@trash.net \
    --cc=netfilter-devel@vger.kernel.org \
    --cc=pva@gentoo.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;
as well as URLs for NNTP newsgroup(s).