kernelnewbies.kernelnewbies.org archive mirror
 help / color / mirror / Atom feed
* objdiff usage
@ 2015-12-02  8:56 Bogicevic Sasa
  2015-12-02  9:57 ` bojan prtvar
  0 siblings, 1 reply; 7+ messages in thread
From: Bogicevic Sasa @ 2015-12-02  8:56 UTC (permalink / raw)
  To: kernelnewbies

Hi all,
I am looking for a guidance on ./scripts/objdiff usage, cant seem to
find a goot example of the scenario where you for example make a bunch of
changes (code style issues) in one folder and need to check if that?changes
don't change the object file. That can speed up code cleaning quite a bi but
all I found online are patches related to objdiff.
                            ?
Thanks,Sasa

^ permalink raw reply	[flat|nested] 7+ messages in thread

* objdiff usage
  2015-12-02  8:56 objdiff usage Bogicevic Sasa
@ 2015-12-02  9:57 ` bojan prtvar
  2015-12-02 10:07   ` Bogicevic Sasa
  0 siblings, 1 reply; 7+ messages in thread
From: bojan prtvar @ 2015-12-02  9:57 UTC (permalink / raw)
  To: kernelnewbies

Hi Sasa,

On Wed, Dec 2, 2015 at 9:56 AM, Bogicevic Sasa <brutallesale@gmail.com> wrote:
> Hi all,
> I am looking for a guidance on ./scripts/objdiff usage, cant seem to
> find a goot example of the scenario where you for example make a bunch of
> changes (code style issues) in one folder and need to check if that?changes
> don't change the object file. That can speed up code cleaning quite a bi but
> all I found online are patches related to objdiff.
>                             ?
> Thanks,Sasa
>


Example usage link can be fund in the objdiff commit thread:
https://lkml.org/lkml/2014/4/7/312

Regards,
Bojan

^ permalink raw reply	[flat|nested] 7+ messages in thread

* objdiff usage
  2015-12-02  9:57 ` bojan prtvar
@ 2015-12-02 10:07   ` Bogicevic Sasa
  2015-12-02 10:31     ` bojan prtvar
  0 siblings, 1 reply; 7+ messages in thread
From: Bogicevic Sasa @ 2015-12-02 10:07 UTC (permalink / raw)
  To: kernelnewbies

On 12/02/2015, bojan prtvar wrote:
>Hi Sasa,
>
>On Wed, Dec 2, 2015 at 9:56 AM, Bogicevic Sasa <brutallesale@gmail.com> wrote:
>> Hi all,
>> I am looking for a guidance on ./scripts/objdiff usage, cant seem to
>> find a goot example of the scenario where you for example make a bunch of
>> changes (code style issues) in one folder and need to check if that?changes
>> don't change the object file. That can speed up code cleaning quite a bi but
>> all I found online are patches related to objdiff.
>>                             ?
>> Thanks,Sasa
>>
>
>
>Example usage link can be fund in the objdiff commit thread:
>https://lkml.org/lkml/2014/4/7/312
>
>Regards,
>Bojan
Hvala Bojane :)
Thanks Bojan,
What about the case where you dont have object files to compare ?
For example driver/pci/hotplug in my case ? How do you prove that your
changes didn't brake something ?

Thanks, Sasa
{
	name: Sasa Bogicevic
	phone: +381:606006200
}

^ permalink raw reply	[flat|nested] 7+ messages in thread

* objdiff usage
  2015-12-02 10:07   ` Bogicevic Sasa
@ 2015-12-02 10:31     ` bojan prtvar
  2015-12-02 10:39       ` Bogicevic Sasa
  0 siblings, 1 reply; 7+ messages in thread
From: bojan prtvar @ 2015-12-02 10:31 UTC (permalink / raw)
  To: kernelnewbies

> What about the case where you dont have object files to compare ?
> For example driver/pci/hotplug in my case ? How do you prove that your
> changes didn't brake something ?

What do you mean by no object files in driver/pci/hotplug ?
Is your HOTPLUG_PCI defined?


Bojan

^ permalink raw reply	[flat|nested] 7+ messages in thread

* objdiff usage
  2015-12-02 10:31     ` bojan prtvar
@ 2015-12-02 10:39       ` Bogicevic Sasa
  2015-12-02 10:56         ` bojan prtvar
  0 siblings, 1 reply; 7+ messages in thread
From: Bogicevic Sasa @ 2015-12-02 10:39 UTC (permalink / raw)
  To: kernelnewbies

On 12/02/2015, bojan prtvar wrote:
>> What about the case where you dont have object files to compare ?
>> For example driver/pci/hotplug in my case ? How do you prove that your
>> changes didn't brake something ?
>
>What do you mean by no object files in driver/pci/hotplug ?
>Is your HOTPLUG_PCI defined?
>
>
>Bojan

Well, when I pull staging tree and go to drivers/pci/hotplug I get c and
h files. Am I supposed to first compile the kernel make changes and
compile again to be anle to compare ?
-- 
{
	name: Sasa Bogicevic
	phone: +381606006200
}

^ permalink raw reply	[flat|nested] 7+ messages in thread

* objdiff usage
  2015-12-02 10:39       ` Bogicevic Sasa
@ 2015-12-02 10:56         ` bojan prtvar
  2015-12-02 11:13           ` Bogicevic Sasa
  0 siblings, 1 reply; 7+ messages in thread
From: bojan prtvar @ 2015-12-02 10:56 UTC (permalink / raw)
  To: kernelnewbies

On Wed, Dec 2, 2015 at 11:39 AM, Bogicevic Sasa <brutallesale@gmail.com> wrote:
> Well, when I pull staging tree and go to drivers/pci/hotplug I get c and
> h files. Am I supposed to first compile the kernel make changes and
> compile again to be anle to compare ?



Something like that. Take a look to the script itself, the usage
example should be quite clear

+# usage example:
+#
+# $ git checkout COMMIT_A
+# $ <your fancy build command here>
+# $ ./scripts/objdiff record path/to/*.o
+#
+# $ git checkout COMMIT_B
+# $ <your fancy build command here>
+# $ ./scripts/objdiff record path/to/*.o
+#
+# $ ./scripts/objdiff diff COMMIT_A COMMIT_B
+# $

Regards,
Bojan

^ permalink raw reply	[flat|nested] 7+ messages in thread

* objdiff usage
  2015-12-02 10:56         ` bojan prtvar
@ 2015-12-02 11:13           ` Bogicevic Sasa
  0 siblings, 0 replies; 7+ messages in thread
From: Bogicevic Sasa @ 2015-12-02 11:13 UTC (permalink / raw)
  To: kernelnewbies

On 12/02/2015, bojan prtvar wrote:
>On Wed, Dec 2, 2015 at 11:39 AM, Bogicevic Sasa <brutallesale@gmail.com> wrote:
>> Well, when I pull staging tree and go to drivers/pci/hotplug I get c and
>> h files. Am I supposed to first compile the kernel make changes and
>> compile again to be anle to compare ?
>
>
>
>Something like that. Take a look to the script itself, the usage
>example should be quite clear
>
>+# usage example:
>+#
>+# $ git checkout COMMIT_A
>+# $ <your fancy build command here>
>+# $ ./scripts/objdiff record path/to/*.o
>+#
>+# $ git checkout COMMIT_B
>+# $ <your fancy build command here>
>+# $ ./scripts/objdiff record path/to/*.o
>+#
>+# $ ./scripts/objdiff diff COMMIT_A COMMIT_B
>+# $
>
>Regards,
>Bojan

Ahhh all clear now, thanks a lot!

Sasa
{
	name: Sasa Bogicevic
	phone: +381606006200
}

^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2015-12-02 11:13 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-12-02  8:56 objdiff usage Bogicevic Sasa
2015-12-02  9:57 ` bojan prtvar
2015-12-02 10:07   ` Bogicevic Sasa
2015-12-02 10:31     ` bojan prtvar
2015-12-02 10:39       ` Bogicevic Sasa
2015-12-02 10:56         ` bojan prtvar
2015-12-02 11:13           ` Bogicevic Sasa

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).