* [uml-devel] Re: [patch 7/7] uml: resolve symbols in back-traces
[not found] <200410272223.i9RMNj921852@mail.osdl.org>
@ 2004-10-28 18:34 ` Blaisorblade
2004-10-28 19:28 ` Chris Wedgwood
0 siblings, 1 reply; 9+ messages in thread
From: Blaisorblade @ 2004-10-28 18:34 UTC (permalink / raw)
To: akpm, user-mode-linux-devel, LKML; +Cc: jdike, cw
On Thursday 28 October 2004 00:27, akpm@osdl.org wrote:
> From: Chris Wedgwood <cw@f00f.org>
> Resolve symbols in back-traces.
> Signed-off-by: Chris Wedgwood <cw@f00f.org>
> Signed-off-by: Andrew Morton <akpm@osdl.org>
> ---
The idea is right - I already had it in my tree, with some little changes (for
instance including the symbol address and so on). So I'll send it reviewed a
bit.
I had not yet sent it because there was more critical stuff and because
sometimes on panic it does not work yet - this seems to be a missing flush
from the UML console drivers :-(, a bit long to solve because currently the
locking is broken (a semaphore is used in process context, nothing in
interrupt context).
And removing the final Emacs comment is not welcome (I don't care, but Jeff
does. If that should be removed, that's a separate problem).
Bye
--
Paolo Giarrusso, aka Blaisorblade
Linux registered user n. 292729
-------------------------------------------------------
This Newsletter Sponsored by: Macrovision
For reliable Linux application installations, use the industry's leading
setup authoring tool, InstallShield X. Learn more and evaluate
today. http://clk.atdmt.com/MSI/go/ins0030000001msi/direct/01/
_______________________________________________
User-mode-linux-devel mailing list
User-mode-linux-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/user-mode-linux-devel
^ permalink raw reply [flat|nested] 9+ messages in thread
* [uml-devel] Re: [patch 7/7] uml: resolve symbols in back-traces
2004-10-28 18:34 ` [uml-devel] Re: [patch 7/7] uml: resolve symbols in back-traces Blaisorblade
@ 2004-10-28 19:28 ` Chris Wedgwood
2004-10-28 21:02 ` Blaisorblade
2004-10-28 21:32 ` Jeff Dike
0 siblings, 2 replies; 9+ messages in thread
From: Chris Wedgwood @ 2004-10-28 19:28 UTC (permalink / raw)
To: Blaisorblade; +Cc: akpm, user-mode-linux-devel, LKML, jdike
On Thu, Oct 28, 2004 at 08:34:21PM +0200, Blaisorblade wrote:
> And removing the final Emacs comment is not welcome (I don't care,
> but Jeff does. If that should be removed, that's a separate
> problem).
the emacs comments are gratuitous and completely pointless, they serve
no useful purpose. fwiw in my .emacs i have:
(defun cw-linux-c-mode ()
"C mode with adjusted defaults for use with the Linux kernel."
(interactive)
(c-mode)
(c-set-style "linux"))
(setq auto-mode-alist
(append '(("wk/linux.*/.*\\.[ch]$" . cw-linux-c-mode))
'(("/usr/src/linux.*/.*\\.[ch]$" . cw-linux-c-mode))
auto-mode-alist))
which actually could be cleaned up a bit (it's been hacked over the
years and never cleaned up suitably) but the idea is pretty simple
--cw
-------------------------------------------------------
This Newsletter Sponsored by: Macrovision
For reliable Linux application installations, use the industry's leading
setup authoring tool, InstallShield X. Learn more and evaluate
today. http://clk.atdmt.com/MSI/go/ins0030000001msi/direct/01/
_______________________________________________
User-mode-linux-devel mailing list
User-mode-linux-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/user-mode-linux-devel
^ permalink raw reply [flat|nested] 9+ messages in thread
* [uml-devel] Re: [patch 7/7] uml: resolve symbols in back-traces
2004-10-28 21:32 ` Jeff Dike
@ 2004-10-28 20:51 ` Chris Wedgwood
2004-10-28 23:44 ` Blaisorblade
2004-10-28 23:42 ` Blaisorblade
1 sibling, 1 reply; 9+ messages in thread
From: Chris Wedgwood @ 2004-10-28 20:51 UTC (permalink / raw)
To: Jeff Dike; +Cc: Blaisorblade, akpm, user-mode-linux-devel, LKML
On Thu, Oct 28, 2004 at 05:32:43PM -0400, Jeff Dike wrote:
> They're not completely pointless, they just cater to an individual's
> development environment, and that sort of stuff should be in the
> environment, and not the code.
unnecessary in the code then, the exception to this perhaps being the
compile-command stuff that was (still is?) in some of the network
drivers as that really is per-file state
-------------------------------------------------------
This Newsletter Sponsored by: Macrovision
For reliable Linux application installations, use the industry's leading
setup authoring tool, InstallShield X. Learn more and evaluate
today. http://clk.atdmt.com/MSI/go/ins0030000001msi/direct/01/
_______________________________________________
User-mode-linux-devel mailing list
User-mode-linux-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/user-mode-linux-devel
^ permalink raw reply [flat|nested] 9+ messages in thread
* [uml-devel] Re: [patch 7/7] uml: resolve symbols in back-traces
2004-10-28 19:28 ` Chris Wedgwood
@ 2004-10-28 21:02 ` Blaisorblade
2004-10-28 21:43 ` Chris Wedgwood
2004-10-28 21:32 ` Jeff Dike
1 sibling, 1 reply; 9+ messages in thread
From: Blaisorblade @ 2004-10-28 21:02 UTC (permalink / raw)
To: Chris Wedgwood; +Cc: akpm, user-mode-linux-devel, LKML, jdike
On Thursday 28 October 2004 21:28, Chris Wedgwood wrote:
> On Thu, Oct 28, 2004 at 08:34:21PM +0200, Blaisorblade wrote:
> > And removing the final Emacs comment is not welcome (I don't care,
> > but Jeff does. If that should be removed, that's a separate
> > problem).
> the emacs comments are gratuitous and completely pointless, they serve
> no useful purpose. fwiw in my .emacs i have:
> (defun cw-linux-c-mode ()
> "C mode with adjusted defaults for use with the Linux kernel."
> (interactive)
> (c-mode)
> (c-set-style "linux"))
> (setq auto-mode-alist
> (append '(("wk/linux.*/.*\\.[ch]$" . cw-linux-c-mode))
> '(("/usr/src/linux.*/.*\\.[ch]$" . cw-linux-c-mode))
> auto-mode-alist))
>
> which actually could be cleaned up a bit (it's been hacked over the
> years and never cleaned up suitably) but the idea is pretty simple
I mostly agree on this, and thought it myself (I *never* use Emacs, only Vim,
but no flames here).
However, let's be kind with Jeff, since he's the UML maintainer. You can be
right, but there's also courtesy.
If Jeff says "OK", then someone can submit a separate patch removing all the
final comments.
Until then, please don't.
And never mix such unrelated changes in a patch.
--
Paolo Giarrusso, aka Blaisorblade
Linux registered user n. 292729
-------------------------------------------------------
This Newsletter Sponsored by: Macrovision
For reliable Linux application installations, use the industry's leading
setup authoring tool, InstallShield X. Learn more and evaluate
today. http://clk.atdmt.com/MSI/go/ins0030000001msi/direct/01/
_______________________________________________
User-mode-linux-devel mailing list
User-mode-linux-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/user-mode-linux-devel
^ permalink raw reply [flat|nested] 9+ messages in thread
* [uml-devel] Re: [patch 7/7] uml: resolve symbols in back-traces
2004-10-28 19:28 ` Chris Wedgwood
2004-10-28 21:02 ` Blaisorblade
@ 2004-10-28 21:32 ` Jeff Dike
2004-10-28 20:51 ` Chris Wedgwood
2004-10-28 23:42 ` Blaisorblade
1 sibling, 2 replies; 9+ messages in thread
From: Jeff Dike @ 2004-10-28 21:32 UTC (permalink / raw)
To: Chris Wedgwood; +Cc: Blaisorblade, akpm, user-mode-linux-devel, LKML
cw@f00f.org said:
> the emacs comments are gratuitous and completely pointless, they serve
> no useful purpose. fwiw in my .emacs i have:
Yeah, that's why I acked that particular change.
They're not completely pointless, they just cater to an individual's development
environment, and that sort of stuff should be in the environment, and not the
code.
Jeff
-------------------------------------------------------
This Newsletter Sponsored by: Macrovision
For reliable Linux application installations, use the industry's leading
setup authoring tool, InstallShield X. Learn more and evaluate
today. http://clk.atdmt.com/MSI/go/ins0030000001msi/direct/01/
_______________________________________________
User-mode-linux-devel mailing list
User-mode-linux-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/user-mode-linux-devel
^ permalink raw reply [flat|nested] 9+ messages in thread
* [uml-devel] Re: [patch 7/7] uml: resolve symbols in back-traces
2004-10-28 21:02 ` Blaisorblade
@ 2004-10-28 21:43 ` Chris Wedgwood
0 siblings, 0 replies; 9+ messages in thread
From: Chris Wedgwood @ 2004-10-28 21:43 UTC (permalink / raw)
To: Blaisorblade; +Cc: akpm, user-mode-linux-devel, LKML, jdike
On Thu, Oct 28, 2004 at 11:02:48PM +0200, Blaisorblade wrote:
> If Jeff says "OK", then someone can submit a separate patch removing
> all the final comments.
he did
> And never mix such unrelated changes in a patch.
idealistically i agree, but doing trivial / obvious cleanups inside
other patches is fine if you ask me
it's not like we are mixing semantically different changes
-------------------------------------------------------
This Newsletter Sponsored by: Macrovision
For reliable Linux application installations, use the industry's leading
setup authoring tool, InstallShield X. Learn more and evaluate
today. http://clk.atdmt.com/MSI/go/ins0030000001msi/direct/01/
_______________________________________________
User-mode-linux-devel mailing list
User-mode-linux-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/user-mode-linux-devel
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [uml-devel] Re: [patch 7/7] uml: resolve symbols in back-traces
2004-10-28 21:32 ` Jeff Dike
2004-10-28 20:51 ` Chris Wedgwood
@ 2004-10-28 23:42 ` Blaisorblade
1 sibling, 0 replies; 9+ messages in thread
From: Blaisorblade @ 2004-10-28 23:42 UTC (permalink / raw)
To: user-mode-linux-devel; +Cc: Jeff Dike, Chris Wedgwood, akpm, LKML
On Thursday 28 October 2004 23:32, Jeff Dike wrote:
> cw@f00f.org said:
> > the emacs comments are gratuitous and completely pointless, they serve
> > no useful purpose. fwiw in my .emacs i have:
>
> Yeah, that's why I acked that particular change.
>
> They're not completely pointless, they just cater to an individual's
> development environment, and that sort of stuff should be in the
> environment, and not the code.
>
> Jeff
Ok. In fact I already agreed, and I was a bit unconfortable with that. However
I'm working on other stuff, so I never worried about that, while seeing you
adding such comments lately. I'm also concerned about not stepping on other
people toes.
--
Paolo Giarrusso, aka Blaisorblade
Linux registered user n. 292729
-------------------------------------------------------
This Newsletter Sponsored by: Macrovision
For reliable Linux application installations, use the industry's leading
setup authoring tool, InstallShield X. Learn more and evaluate
today. http://clk.atdmt.com/MSI/go/ins0030000001msi/direct/01/
_______________________________________________
User-mode-linux-devel mailing list
User-mode-linux-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/user-mode-linux-devel
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [uml-devel] Re: [patch 7/7] uml: resolve symbols in back-traces
2004-10-28 20:51 ` Chris Wedgwood
@ 2004-10-28 23:44 ` Blaisorblade
2004-10-29 0:19 ` Chris Wedgwood
0 siblings, 1 reply; 9+ messages in thread
From: Blaisorblade @ 2004-10-28 23:44 UTC (permalink / raw)
To: user-mode-linux-devel; +Cc: Chris Wedgwood, Jeff Dike, akpm, LKML
On Thursday 28 October 2004 22:51, Chris Wedgwood wrote:
> On Thu, Oct 28, 2004 at 05:32:43PM -0400, Jeff Dike wrote:
> > They're not completely pointless, they just cater to an individual's
> > development environment, and that sort of stuff should be in the
> > environment, and not the code.
>
> unnecessary in the code then, the exception to this perhaps being the
> compile-command stuff that was (still is?) in some of the network
> drivers as that really is per-file state
??? I don't understand you well. If there are compile-commands for emacs,
they're broken too - using make namefile.o ARCH=um is the kernel universal
solution.
--
Paolo Giarrusso, aka Blaisorblade
Linux registered user n. 292729
-------------------------------------------------------
This Newsletter Sponsored by: Macrovision
For reliable Linux application installations, use the industry's leading
setup authoring tool, InstallShield X. Learn more and evaluate
today. http://clk.atdmt.com/MSI/go/ins0030000001msi/direct/01/
_______________________________________________
User-mode-linux-devel mailing list
User-mode-linux-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/user-mode-linux-devel
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [uml-devel] Re: [patch 7/7] uml: resolve symbols in back-traces
2004-10-28 23:44 ` Blaisorblade
@ 2004-10-29 0:19 ` Chris Wedgwood
0 siblings, 0 replies; 9+ messages in thread
From: Chris Wedgwood @ 2004-10-29 0:19 UTC (permalink / raw)
To: Blaisorblade; +Cc: user-mode-linux-devel, Jeff Dike, akpm, LKML
On Fri, Oct 29, 2004 at 01:44:11AM +0200, Blaisorblade wrote:
> ??? I don't understand you well. If there are compile-commands for
> emacs, they're broken too - using make namefile.o ARCH=um is the
> kernel universal solution.
consider a load-able module for uml ... the compile command might be
to build and load the module into a running UML instance... for some
people that's a pretty nice working module where you dont have to
leave your editor to boot/run test things
-------------------------------------------------------
This Newsletter Sponsored by: Macrovision
For reliable Linux application installations, use the industry's leading
setup authoring tool, InstallShield X. Learn more and evaluate
today. http://clk.atdmt.com/MSI/go/ins0030000001msi/direct/01/
_______________________________________________
User-mode-linux-devel mailing list
User-mode-linux-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/user-mode-linux-devel
^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2004-10-29 0:20 UTC | newest]
Thread overview: 9+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <200410272223.i9RMNj921852@mail.osdl.org>
2004-10-28 18:34 ` [uml-devel] Re: [patch 7/7] uml: resolve symbols in back-traces Blaisorblade
2004-10-28 19:28 ` Chris Wedgwood
2004-10-28 21:02 ` Blaisorblade
2004-10-28 21:43 ` Chris Wedgwood
2004-10-28 21:32 ` Jeff Dike
2004-10-28 20:51 ` Chris Wedgwood
2004-10-28 23:44 ` Blaisorblade
2004-10-29 0:19 ` Chris Wedgwood
2004-10-28 23:42 ` Blaisorblade
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox