All of lore.kernel.org
 help / color / mirror / Atom feed
From: Arkadiusz Miskiewicz <arekm@pld-linux.org>
To: linux-hotplug@vger.kernel.org
Subject: Re: bashizm in hotplug scripts (including 20040401)
Date: Mon, 05 Apr 2004 17:46:14 +0000	[thread overview]
Message-ID: <200404051946.14605.arekm@pld-linux.org> (raw)
In-Reply-To: <200404020211.45303.arekm@pld-linux.org>

Dnia Monday 05 of April 2004 08:06, Martin Schwenke napisa³:

> The value of PCI_ANY only ever compared against other things - there's
> no useful arithmetic done.  The PCI 2.2 specification only allows
> Vendor ID and Device ID to be 16 bits (in configuration space at
> least).  If there any reason for using the value 0xffffffff, rather
> than just 0xffff?  Greg?
If 16 bits is max in reality then it seems that we don't have to do anything 
because:

PCI_ANY=$((0xffffffff))

would overflow to -1 and 

vendor=$(($vendor));
this will also overflow to -1

if [ $vendor -ne $PCI_ANY -a $vendor -ne $pci_id_vendor ]; then
and we end up with -1 -ne -1 comparsion.

Anyway comment about this should be added in hotplug scripts.

> peace & happiness,
> martin

-- 
Arkadiusz Mi¶kiewicz     CS at FoE, Wroclaw University of Technology
arekm.pld-linux.org, 1024/3DB19BBD, JID: arekm.jabber.org, PLD/Linux


-------------------------------------------------------
This SF.Net email is sponsored by: IBM Linux Tutorials
Free Linux tutorial presented by Daniel Robbins, President and CEO of
GenToo technologies. Learn everything from fundamentals to system
administration.http://ads.osdn.com/?ad_id\x1470&alloc_id638&opÌk
_______________________________________________
Linux-hotplug-devel mailing list  http://linux-hotplug.sourceforge.net
Linux-hotplug-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linux-hotplug-devel

      parent reply	other threads:[~2004-04-05 17:46 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2004-04-02  0:11 bashizm in hotplug scripts (including 20040401) Arkadiusz Miskiewicz
2004-04-02  1:09 ` Martin Schwenke
2004-04-02 13:05 ` Arkadiusz Miskiewicz
2004-04-05  6:06 ` Martin Schwenke
2004-04-05 17:11 ` Greg KH
2004-04-05 17:46 ` Arkadiusz Miskiewicz [this message]

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=200404051946.14605.arekm@pld-linux.org \
    --to=arekm@pld-linux.org \
    --cc=linux-hotplug@vger.kernel.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 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.