All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Luis R. Rodriguez" <mcgrof@do-not-panic.com>
To: Ben Hutchings <ben@decadent.org.uk>
Cc: backports@vger.kernel.org, 703142@bugs.debian.org,
	Camaleón <noelamac@gmail.com>
Subject: Re: Bug#703142: compatibility with alx ?
Date: Wed, 20 Mar 2013 08:50:15 -0700	[thread overview]
Message-ID: <CAB=NE6UZ+RvFHyzUhSwzATnkd9gAGa0EKDQD+U9vxVM8FSvLrA@mail.gmail.com> (raw)
In-Reply-To: <1363790495.31336.51.camel@deadeye.wl.decadent.org.uk>

On Wed, Mar 20, 2013 at 7:41 AM, Ben Hutchings <ben@decadent.org.uk> wrote:
> On Wed, 2013-03-20 at 02:07 -0700, Luis R. Rodriguez wrote:
>> On Sun, Mar 17, 2013 at 2:43 PM, Ben Hutchings <ben@decadent.org.uk> wro=
te:
>> > [Re-sending to the correct list address.]
>> >
>> > On Sun, 2013-03-17 at 16:24 +0100, Camale=C3=B3n wrote:
>> >> El 2013-03-17 a las 14:58 +0000, Ben Hutchings escribi=C3=B3:
>> >>
>> >> > On Sun, 2013-03-17 at 15:46 +0100, Camale=C3=B3n wrote:
>> >>
>> >> (...)
>> >>
>> >> > > Using Debian's stock network driver is not an option for me (full=
 report
>> >> > > available here=C2=B2) so I have to try with the latests drivers b=
ut now that
>> >> > > "compat-drivers" are compiled the generated modules cannot be loa=
ded.
>> >> > >
>> >> > > Is there any by-pass for this?
>> >> > >
>> >> > > =C2=B9http://marc.info/?t=3D136351034300002&r=3D1&w=3D2
>> >> > > =C2=B2http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=3D664767
>> >> >
>> >> > Talk to the compat-drivers developers.
>> >>
>> >> To be sincere, I don't think that's a user's role.
>> >>
>> >> I don't know what's going on with these drivers but if they are not
>> >> supported by Debian at all it would be better for all of us (plain
>> >> users and developers) to simply say it so to avoid wasting time and
>> >> resources.
>> >
>> > I would like to support them, in fact more than that I would like to
>> > integrate them into official packages.  But there is no way we can
>> > support an OOT module that defines symbols that we might need to add f=
or
>> > our own backports.  As it is 'compat' will ironically cause
>> > incompatibility with Debian's own kernel upgrades.
>> >
>> > Compat developers: please add a prefix (not 'compat', that one's alrea=
dy
>> > taken!)
>>
>> We have been using compat_ for a while now to prefix a lot of our
>> symbols without clashes for the 32-64 compat stuff, but sure -- we can
>> use something else to help with any theoretical issues. Surprised
>> Debian of all distributions would frankly have been affected given
>> RHEL / SUSE didn't, but its OK, lets deal with it.
>
> The conflict that just showed up in Debian involved the 'i2c_bit_algo'
> symbol which had no symbol prefix in 'compat'.  We updated the in-tree
> DRM drivers from 3.4.32 and started exporting the symbol from
> i2c-algo-bit itself.
>
> I hadn't noticed that you already used the 'compat_' prefix for some
> exported symbols and I'm not aware of any current conflict with the
> 32-bit compatibility layer, but it seems plausible that it could happen
> in future.

Sure, let me know what you think of the proposed posted changes.

  Luis

  reply	other threads:[~2013-03-20 15:50 UTC|newest]

Thread overview: 23+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20130317144603.GA4850@stt008.linux.site>
     [not found] ` <1363532310.3937.266.camel@deadeye.wl.decadent.org.uk>
     [not found]   ` <20130317152428.GA4940@stt008.linux.site>
     [not found]     ` <1363544257.3937.278.camel@deadeye.wl.decadent.org.uk>
2013-03-17 21:43       ` Bug#703142: compatibility with alx ? Ben Hutchings
2013-03-20  9:07         ` Luis R. Rodriguez
2013-03-20 14:41           ` Ben Hutchings
2013-03-20 15:50             ` Luis R. Rodriguez [this message]
2013-03-20 17:45               ` Ben Hutchings
2013-03-26 15:07                 ` Camaleón
     [not found]     ` <20130317175658.GA8181@elie.Belkin>
     [not found]       ` <20130317182725.GA5811@stt008.linux.site>
     [not found]         ` <20130317183344.GA8786@elie.Belkin>
     [not found]           ` <CAKprTDFhNnGOPHUqg8Wn6Cd170KaeXj7-w-D6wS2Cc38vstt5A@mail.gmail.com>
2013-03-18  8:37             ` Brcmsmac driver woes, possible regression? Jonathan Nieder
2013-03-18 19:38               ` Arend van Spriel
2013-03-18 21:14                 ` Camaleón
2013-03-18 21:25                   ` Jonathan Nieder
2013-03-19  9:43                     ` Camaleón
2013-03-19 16:21                       ` Camaleón
2013-03-19 16:53                         ` Dan Williams
2013-03-19 17:30                           ` Camaleón
2013-03-19 17:59                             ` Dan Williams
2013-03-19 18:11                               ` Camaleón
2013-03-19 18:16                                 ` Dan Williams
2013-03-19 18:30                                   ` Camaleón
2013-03-19 21:55                         ` Arend van Spriel
2013-03-19 22:11                           ` Dan Williams
2013-03-23 15:28                         ` Camaleón
2013-03-24 10:10                           ` Arend van Spriel
2013-03-24 10:37                             ` Camaleón

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='CAB=NE6UZ+RvFHyzUhSwzATnkd9gAGa0EKDQD+U9vxVM8FSvLrA@mail.gmail.com' \
    --to=mcgrof@do-not-panic.com \
    --cc=703142@bugs.debian.org \
    --cc=backports@vger.kernel.org \
    --cc=ben@decadent.org.uk \
    --cc=noelamac@gmail.com \
    /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.