Discussion:
[patch 03/12] uml: export getgid for hostfs
(too old to reply)
Blaisorblade
2005-03-24 02:07:11 UTC
Permalink
Export this symbol which is not satisfied currently. The code using it
has been merged, so please export this symbol.
and it's still bogus, and you haven't replied when I mentioned it last
time.
In this moment I need to clean up the missing symbol. If anyone wants to
remove the code using this, then he might post a patch explictly removing it,
and getting it refused probably.

Or at least CC uml-devel when discussing those problems. I'm not currently
able to find on marc.theaimsgroup.com the mail you talk about. Can you please
provide the URL to the discussion? (even on any other archive you like,
obviously).

That said, there are people still using that code, so it should be kept in.

Also, you blocked an important patch (the one adding ->release to
hw_interrupt_type) saying that *perhaps* UML should avoid having any hard
irq, a la S390. You forced so the merge of a very ugly patch manually calling
what should have been UML's release method (i.e. free_irq_by_irq_and_dev) in
every place calling free_irq() (and in fact one was missed at first). Might
you reconsider your position on that issue ? (URL of the discussion below)

http://marc.theaimsgroup.com/?l=linux-kernel&w=2&r=3&s=uml+irq&q=b

The patch adding the generic handling is this one:

http://marc.theaimsgroup.com/?l=linux-kernel&m=109834481320519&w=2
--
Paolo Giarrusso, aka Blaisorblade
Linux registered user n. 292729
http://www.user-mode-linux.org/~blaisorblade

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to ***@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/



-------------------------------------------------------------------------------
Achtung: diese Newsgruppe ist eine unidirektional gegatete Mailingliste.
Antworten nur per Mail an die im Reply-To-Header angegebene Adresse.
Fragen zum Gateway -> ***@inka.de.
-------------------------------------------------------------------------------
Christoph Hellwig
2005-03-29 11:51:42 UTC
Permalink
Post by Blaisorblade
In this moment I need to clean up the missing symbol. If anyone wants to
remove the code using this, then he might post a patch explictly removing it,
and getting it refused probably.
Or at least CC uml-devel when discussing those problems. I'm not currently
able to find on marc.theaimsgroup.com the mail you talk about. Can you please
provide the URL to the discussion? (even on any other archive you like,
obviously).
My unaswered reply to the first submission is at

http://groups-beta.google.com/group/linux.kernel/messages/de9504fe5963ccd1,0c05294c599b22b1,eab26a4ed3f8ff17?thread_id=16c905c7e28e7498&mode=thread&noheader=1&q=uml-export-getgid-for-hostfs#doc_eab26a4ed3f8ff17

(sorry, couldn't find it on marc), it's been Cc'ed to the lists you sent
the patch to.
Post by Blaisorblade
That said, there are people still using that code, so it should be kept in.
But the code is totally bogus, so it should _not_ be kept.
Post by Blaisorblade
Also, you blocked an important patch (the one adding ->release to
hw_interrupt_type) saying that *perhaps* UML should avoid having any hard
irq, a la S390. You forced so the merge of a very ugly patch manually calling
what should have been UML's release method (i.e. free_irq_by_irq_and_dev) in
every place calling free_irq() (and in fact one was missed at first). Might
you reconsider your position on that issue ? (URL of the discussion below)
http://marc.theaimsgroup.com/?l=linux-kernel&w=2&r=3&s=uml+irq&q=b
http://marc.theaimsgroup.com/?l=linux-kernel&m=109834481320519&w=2
I still think it's a really bad idea. But I'm not the irq code maintainer,
it could very well be Ingo overrides me.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to ***@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/



-------------------------------------------------------------------------------
Achtung: diese Newsgruppe ist eine unidirektional gegatete Mailingliste.
Antworten nur per Mail an die im Reply-To-Header angegebene Adresse.
Fragen zum Gateway -> ***@inka.de.
-------------------------------------------------------------------------------
Loading...